Add tests for text modules#56812
Conversation
|
Might consider adding a test that the request has a Also, you could add a test that importing a module as JS, from a server which responds with valid JS file with a content type of |
Done. There are some test for the JSON module fetch headers already at
Done; added those to |
|
New tests look great, thanks. |
annevk
left a comment
There was a problem hiding this comment.
These look good to me. I didn't verify every line though.
zcorpan
left a comment
There was a problem hiding this comment.
Test coverage looks good. There are merge conflicts though.
This is to support the spec changes in:
I've effectively tried to duplicate and appropriately modify all the tests for JSON modules that I was able to find.