Skip to content

test: remove redundant mvnw catch#978

Open
999axel999 wants to merge 1 commit into
objectionary:masterfrom
999axel999:zerocracy/970-simplify-mvnw-count
Open

test: remove redundant mvnw catch#978
999axel999 wants to merge 1 commit into
objectionary:masterfrom
999axel999:zerocracy/970-simplify-mvnw-count

Conversation

@999axel999

Copy link
Copy Markdown

Summary

  • Remove the redundant local count try/catch in test/test_mvnw.js.
  • Keep the test behavior unchanged while dropping the dead ENOENT branch.

Closes #970

Tests

  • node ..\objectionary__eoc_fix\node_modules\mocha\bin\mocha.js test\test_mvnw.js --grep "ENOENT race" --timeout 1200000
  • node ..\objectionary__eoc_fix\node_modules\eslint\bin\eslint.js test\test_mvnw.js
  • git diff --check
  • git merge-tree --write-tree origin/master HEAD

Prepared with local automation assistance and reviewed before submission.

@999axel999 999axel999 marked this pull request as ready for review June 7, 2026 02:47

@Thayorns Thayorns left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yegor256 this PR can be closed as a duplicate. The issue is closed already

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

count function incorrectly rethrows ENOENT errors in test/test_mvnw.js

2 participants