Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test on Mac in CI #4

Closed
wants to merge 13 commits into from
Closed

Test on Mac in CI #4

wants to merge 13 commits into from

Conversation

tshino
Copy link
Owner

@tshino tshino commented Nov 7, 2021

No description provided.

@tshino
Copy link
Owner Author

tshino commented Nov 9, 2021

observations

From some point on the main branch, tests on Mac in CI started to fail continuously. The logs tell us almost nothing but the exit code:
image
saying:

> npm run lint
Error: Process completed with exit code 249.

Above npm run lint is pretest script that runs before test script. So I removed pretest script to skip lint. But CI is still reporting the same error:

> node ./test/runTest.js
Error: Process completed with exit code 249.

Above node ./test/runTest.js is test script.
Then, I guessed that maybe the big package.json was the cause, so I removed all the keybindings definitions in the package.json to find out the cause.
Then, the error disappeared.

@tshino
Copy link
Owner Author

tshino commented Nov 13, 2021

observations part 2

Based on the hypothesis that the large package.json is the cause, I tried a binary search.
image

In summary, roughly 220KB of package.json is ok, but 227KB of package.json is not.

% of keybindings included size of package.json test
0% 4,673 pass
50% 121,358 pass
75% 179,733 pass
87% 206,426 pass
94% 220,638 pass
97% 227,328 fail
100% 234,242 fail

All these failures are the 'exit code 249'.

@tshino tshino closed this Jan 1, 2022
@tshino tshino deleted the test-on-macos branch March 4, 2022 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant