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

fix: Fix macOS + node v17 test failures #394

Merged
merged 1 commit into from
Apr 7, 2022

Conversation

joeyparrish
Copy link
Contributor

  • Fix decoder test failures by importing the necessary data and running
    a local server, rather than relying on external sites
  • Tests depending on /proc are now only run on Linux
  • Tests that only set expectations on the behavior of the node http
    server (rather than needle itself) have been deleted

Closes #393

@joeyparrish joeyparrish changed the title fix: Fix macOS test failures fix: Fix node v17 test failures Mar 22, 2022
@joeyparrish joeyparrish changed the title fix: Fix node v17 test failures fix: Fix macOS + node v17 test failures Mar 22, 2022
 - Fix decoder test failures by importing the necessary data and running
   a local server, rather than relying on external sites
 - Tests depending on /proc are now only run on Linux
 - Tests that only set expectations on the behavior of the node http
   server (rather than needle itself) have been deleted

Closes tomas#393
@tomas tomas merged commit 0044513 into tomas:master Apr 7, 2022
@joeyparrish joeyparrish deleted the fix-macos-tests branch April 20, 2022 15:54
joeyparrish added a commit to joeyparrish/less.js that referenced this pull request Apr 20, 2022
The issue was traced upstream to needle, and resolved in:
 - tomas/needle#392
 - tomas/needle#394
 - tomas/needle#396
 - tomas/needle#398

Closes less#3693
iChenLei pushed a commit to less/less.js that referenced this pull request Apr 22, 2022
The issue was traced upstream to needle, and resolved in:
 - tomas/needle#392
 - tomas/needle#394
 - tomas/needle#396
 - tomas/needle#398

Closes #3693
lumburr pushed a commit to lumburr/less.js that referenced this pull request Apr 28, 2022
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.

Test failures on macOS w/ node v17
2 participants