fix(#93): removed `seq` atom + added flag for runtime semver

maxonfjvipon commented 5 days ago

Closes: #93

PR-Codex overview

The focus of this PR is to update test functions to include a done callback for asynchronous testing and handle exceptions in the test command execution.

Detailed summary

Added done callback to test functions for asynchronous testing
Handled exceptions in test command execution
Added runtimeVersion option for specifying eo2js-runtime version
Refactored pckg function to accept an options object
Updated package.json generation based on program options

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

maxonfjvipon commented 5 days ago

@rultor merge

rultor commented 5 days ago

@rultor merge

@maxonfjvipon OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 5 days ago

@rultor merge

@maxonfjvipon Oops, I failed. You can see the full log here (spent 20min)

    at checkExecSyncError (node:child_process:885:11)
    at execSync (node:child_process:957:15)
    at test (/home/r/repo/eo2js/src/commands/test.js:12:3)
    at Command.<anonymous> (/home/r/repo/eo2js/src/eo2js.js:71:5)
    at Command.listener [as _actionHandler] (/home/r/repo/eo2js/node_modules/commander/lib/command.js:542:17)
    at /home/r/repo/eo2js/node_modules/commander/lib/command.js:1502:14
    at Command._chainOrCall (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1386:12)
    at Command._parseCommand (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1501:27)
    at /home/r/repo/eo2js/node_modules/commander/lib/command.js:1265:27
    at Command._chainOrCall (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1386:12)
    at Command._dispatchSubcommand (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1261:25)
    at Command._parseCommand (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1457:19)
    at Command.parse (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1064:10)
    at Object.<anonymous> (/home/r/repo/eo2js/src/eo2js.js:75:11)
    at Module._compile (node:internal/modules/cjs/loader:1254:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
    at Module.load (node:internal/modules/cjs/loader:1117:32)
    at Module._load (node:internal/modules/cjs/loader:958:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:23:47

  \u001b[31m  1) should execute all eo-runtime tests\u001b[0m

\u001b[0m  integration test\u001b[0m
Recompiling stylesheets...
Recompiled 0 stylesheets
npm WARN deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm WARN deprecated glob@8.1.0: Glob versions prior to v9 are no longer supported
  \u001b[32m  ✓\u001b[0m\u001b[90m should execute simple unit test\u001b[0m\u001b[31m (16033ms)\u001b[0m
  \u001b[32m  ✓\u001b[0m\u001b[90m should dataize simple program\u001b[0m\u001b[31m (15574ms)\u001b[0m

\u001b[92m \u001b[0m\u001b[32m 29 passing\u001b[0m\u001b[90m (9m)\u001b[0m
\u001b[31m  1 failing\u001b[0m

\u001b[0m  1) runtime tests
       should execute all eo-runtime tests:
\u001b[0m\u001b[31m     Command failed: node /home/r/repo/eo2js/src/eo2js.js test -t /home/r/repo/eo2js/temp/runtime-tests/target -p project -d /home/r/repo/eo2js-runtime  --exclude org/eolang/rust-tests.test.js
npm WARN deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm WARN deprecated glob@8.1.0: Glob versions prior to v9 are no longer supported
Command failed: node ./node_modules/mocha/bin/mocha.js . --recursive --exclude "__main__.js" --exclude "node_modules/**" --exclude "org/eolang/rust-tests.test.js"
\u001b[0m\u001b[90m
  Error: Command failed: node /home/r/repo/eo2js/src/eo2js.js test -t /home/r/repo/eo2js/temp/runtime-tests/target -p project -d /home/r/repo/eo2js-runtime  --exclude org/eolang/rust-tests.test.js
  npm WARN deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
  npm WARN deprecated glob@8.1.0: Glob versions prior to v9 are no longer supported

      at checkExecSyncError (node:child_process:885:11)
      at execSync (node:child_process:957:15)
      at execNode (test/helpers.js:20:12)
      at runSync (test/helpers.js:42:10)
      at Context.<anonymous> (test/it/runtime-tests.test.js:84:17)
      at process.processImmediate (node:internal/timers:476:21)
\u001b[0m

\u001b[33mWarning: \u0007 Use --force to continue.\u001b[39m

\u001b[31mAborted due to warnings.\u001b[39m
container 5ed66bad744bfd9881ed896dbd1f01b9916a161fd65a69b909b729602f421cc7 is dead
Fri Jun 28 10:03:16 UTC 2024

maxonfjvipon commented 4 days ago

@rultor merge

rultor commented 4 days ago

@rultor merge

@maxonfjvipon OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 4 days ago

@rultor merge

@maxonfjvipon Oops, I failed. You can see the full log here (spent 20min)

    at execSync (node:child_process:957:15)
    at test (/home/r/repo/eo2js/src/commands/test.js:13:5)
    at Command.<anonymous> (/home/r/repo/eo2js/src/eo2js.js:71:5)
    at Command.listener [as _actionHandler] (/home/r/repo/eo2js/node_modules/commander/lib/command.js:542:17)
    at /home/r/repo/eo2js/node_modules/commander/lib/command.js:1502:14
    at Command._chainOrCall (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1386:12)
    at Command._parseCommand (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1501:27)
    at /home/r/repo/eo2js/node_modules/commander/lib/command.js:1265:27
    at Command._chainOrCall (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1386:12)
    at Command._dispatchSubcommand (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1261:25)
    at Command._parseCommand (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1457:19)
    at Command.parse (/home/r/repo/eo2js/node_modules/commander/lib/command.js:1064:10)
    at Object.<anonymous> (/home/r/repo/eo2js/src/eo2js.js:75:11)
    at Module._compile (node:internal/modules/cjs/loader:1254:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
    at Module.load (node:internal/modules/cjs/loader:1117:32)
    at Module._load (node:internal/modules/cjs/loader:958:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:23:47

  \u001b[31m  1) should execute all eo-runtime tests\u001b[0m

\u001b[0m  integration test\u001b[0m
Recompiling stylesheets...
Recompiled 0 stylesheets
npm WARN deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm WARN deprecated glob@8.1.0: Glob versions prior to v9 are no longer supported
  \u001b[32m  ✓\u001b[0m\u001b[90m should execute simple unit test\u001b[0m\u001b[31m (16716ms)\u001b[0m
  \u001b[32m  ✓\u001b[0m\u001b[90m should dataize simple program\u001b[0m\u001b[31m (15882ms)\u001b[0m

\u001b[92m \u001b[0m\u001b[32m 29 passing\u001b[0m\u001b[90m (9m)\u001b[0m
\u001b[31m  1 failing\u001b[0m

\u001b[0m  1) runtime tests
       should execute all eo-runtime tests:
\u001b[0m\u001b[31m     Command failed: node /home/r/repo/eo2js/src/eo2js.js test -t /home/r/repo/eo2js/temp/runtime-tests/target -p project -d /home/r/repo/eo2js-runtime  --exclude org/eolang/rust-tests.test.js
npm WARN deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm WARN deprecated glob@8.1.0: Glob versions prior to v9 are no longer supported
Failed to execute mochajs Command failed: node ./node_modules/mocha/bin/mocha.js . --recursive --exclude "__main__.js" --exclude "node_modules/**" --exclude "org/eolang/rust-tests.test.js"
Command failed: node ./node_modules/mocha/bin/mocha.js . --recursive --exclude "__main__.js" --exclude "node_modules/**" --exclude "org/eolang/rust-tests.test.js"
\u001b[0m\u001b[90m
  Error: Command failed: node /home/r/repo/eo2js/src/eo2js.js test -t /home/r/repo/eo2js/temp/runtime-tests/target -p project -d /home/r/repo/eo2js-runtime  --exclude org/eolang/rust-tests.test.js
  npm WARN deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
  npm WARN deprecated glob@8.1.0: Glob versions prior to v9 are no longer supported

      at checkExecSyncError (node:child_process:885:11)
      at execSync (node:child_process:957:15)
      at execNode (test/helpers.js:20:12)
      at runSync (test/helpers.js:42:10)
      at Context.<anonymous> (test/it/runtime-tests.test.js:84:17)
      at process.processImmediate (node:internal/timers:476:21)
\u001b[0m

\u001b[33mWarning: \u0007 Use --force to continue.\u001b[39m

\u001b[31mAborted due to warnings.\u001b[39m
container 299e36c1acc4470220d0898dd1a99cad5e152af5312917145e9a9c8e983d7e5f is dead
Fri Jun 28 13:08:31 UTC 2024

maxonfjvipon commented 2 days ago

@rultor merge

rultor commented 2 days ago

@rultor merge

@maxonfjvipon OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 2 days ago

@rultor merge

@maxonfjvipon Done! FYI, the full log is here (took me 30min)

objectionary / eo2js

fix(#93): removed `seq` atom + added flag for runtime semver #94

PR-Codex overview

Detailed summary