Commits on Sep 10, 2024

1. Fix dynamic import to work with jest …

   When using jest this code gets transpiled to `require` and the file URL syntax of the import is not supported and breaks the testing environment.  This change only uses file URL import syntax on windows machines.  This fixes our tests and is still supported on linux and mac operating systems.
   
   I think windows is definitely in the minority here and we should not optimize for it.  Furthermore, I believe using a UNC path would be a better solution outright but I cannot test because I don't have a window machine at hand.
   
   Why not just use jest's `--experimental-vm-modules` flag? Glad you asked.  Because of this [jest bug](jestjs/jest#11434 (comment)) which is caused by this [node bug](nodejs/node#37648) which is ultimately caused by this [v8 bug](https://issues.chromium.org/issues/40784051)

   

   coryasilva committed Sep 10, 2024
   Configuration menu

   • View commit details
   • Copy full SHA for 9c175fc
   • Browse repository at this point

   Copy the full SHA

   9c175fc View commit details

   Browse the repository at this point in the history