fetch-globals.js: fix test function name

[ecki]

Fix an accidental copy/paste error to make test name unique.

[ecki]

Feel free to squash it. Should I provide a test with other primitive types? (Maybe add an assert helper which takes a list of candidates?

assertThrowsArg([true,42,123n,““,Symbol("Id")], (arg) => new Header(arg), TypeError);

like that?

[iamstolis]

Feel free to squash it.

It would better to squash it on your side: this way the change(set) remains authored by you and the merged changes are properly linked with this PR.

Should I provide a test with other primitive types?

Feel free to leave it as it is. Considering how our specializations are written, there is just a little chance that other primitive types will behave differently. Of course, if you wish to add these tests, feel free to do so.

[iamstolis]

It would better to squash it on your side: ...

So, do you plan to squash it yourself?

[ecki]

Thanks for the reminder i squashed and rebased it.