HED module incorrectly reports HED_VERSION_NOT_DEFINED warning on HED tests

[happy5214]

The tests have improperly nested HED data in three sidecars (the HED level is the third level, where it should be the second level). This is causing https://github.com/hed-standard/hed-javascript/blob/916236a58b9912cb4fefc0e79aa9c28d0f35cedb/bids/types/json.js#L107 to (correctly) throw an IssueError for what amounts to a SIDECAR_INVALID HED error. However, the try block starting at

bids-validator/bids-validator/validators/hed.js

Line 34 in 5646d9d

try {

incorrectly assumes all caught errors are missing schema errors, leading to improper issue reporting.

The tests need to be fixed to correct the level of HED in the three bad test sidecars, and hed.js needs to be re-coded to find another way to deal with a null return from the validation function.

Originally posted by @happy5214 in #2167 (comment)

[happy5214]

Please assign this to me. I plan to work on this in the next day or two.