Add functions to create modules from a JSON value

[hansl]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.28%. Comparing base (6ddc2b4) to head (c13dddf).
Report is 130 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3804      +/-   ##
==========================================
+ Coverage   47.24%   50.28%   +3.04%     
==========================================
  Files         476      459      -17     
  Lines       46892    44966    -1926     
==========================================
+ Hits        22154    22612     +458     
+ Misses      24738    22354    -2384     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jedel1043]

I really think those functions should be static methods of Module instead, possibly following the steps of ParseJSONModule to be spec-compliant. A Module::json(JsString) -> JsResult<Module> function should be able to cover almost all usages.

[hansl]

I don't mind that, let me make that change.

[jedel1043]

Looks perfect to me!