You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Having explored how to incorporate unit and integration testing to the current knetminer client-side architecture using Vitest, which is almost similar to Jest. It was discovered the file type text/javascript attributed to Knetminer Js files is currently outdated and discouraged by HTML5 specification as discussed in this SO thread making it impossible to use assertion libraries as most are compatible with CommonJs and ESM JS module system.
As a result, I propose that knetminer's Javascript files be converted to type modules using ESM specification so that it units can be imported to where it's needed and the issue specified in #766 can be resolved. Also, ESM modules are currently supported across web browsers as specified by MDN javascript module guides making it the best approach to incorporate unit testing to knetminer client-side js files.
The text was updated successfully, but these errors were encountered:
This ticket is related to #766
Having explored how to incorporate unit and integration testing to the current knetminer client-side architecture using Vitest, which is almost similar to Jest. It was discovered the file type
text/javascript
attributed to Knetminer Js files is currently outdated and discouraged by HTML5 specification as discussed in this SO thread making it impossible to use assertion libraries as most are compatible with CommonJs and ESM JS module system.As a result, I propose that knetminer's Javascript files be converted to type
modules
using ESM specification so that it units can be imported to where it's needed and the issue specified in #766 can be resolved. Also, ESM modules are currently supported across web browsers as specified by MDN javascript module guides making it the best approach to incorporate unit testing to knetminer client-side js files.The text was updated successfully, but these errors were encountered: