Doc string tests do not fail on syntax error

[cknitt]

Currently the doc string tests are showing some syntax errors:

  Syntax error in code block in docstring
  Stdlib_DataView.resi:8:52-9:8

   6 │ let dv = DataView.fromBuffer(ArrayBuffer.make(16))
   7 │ dv->DateView.setFloat32(0, 3.14)
   8 │ dv->DateView.setFloat32(4, 3.14, ~littleEndian=true
   9 │ DataView.getFloat32(dv, 0) == 3.14
  10 │ DataView.getFloat32(dv, 4, ~littleEndian=true) == 3.14

  Did you forget a `,` here?



  Syntax error in code block in docstring
  Stdlib_DataView.resi:7:25-31

  5 │
  6 │ let dv = DataView.fromBuffer(ArrayBuffer.make(16))
  7 │ DataView.setUint32(dv, 0 123456)
  8 │ DataView.setUint32(dv, 4, 123456, ~littleEndian=true)
  9 │ dv->DateView.getUint32(0) == 123456

  Did you forget a `,` here?

but still complete successfully (no error in CI).

[cknitt]

@htoooth Would you like to have a look at this?

First, fixing the test scripts so that CI fails for these errors.
Second, fixing the errors themselves.

[zth]

IIRC nothing with the new docstrings stuff was actually added to CI.

[cknitt]

It runs as part of node scripts/test.js -all.