Skip to content

Use consistent "skip test" label #1370

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

Closed
wants to merge 1 commit into from
Closed

Conversation

joshgoebel
Copy link
Contributor

@joshgoebel joshgoebel commented Sep 24, 2021

This seems to consistently fail in the online test runner... (but it's not skipped)... is there a reason for this discrepancy or is this a bug?
Screen Shot 2021-09-23 at 10 47 33 PM

This seems to consistently fail in the online test runner... (but it's not skipped)... is there a reason for this discrepancy or is this a bug?
@SleeplessByte
Copy link
Member

Yes, this is on purpose. test.skip tests are supposed to be skipped because they can't run online.

The typescript-test-runner was recently updated to actually not fail skipped tests, but the javascript-test-runner isn't updated yet.

@SleeplessByte
Copy link
Member

PakkuDon pushed a commit to PakkuDon/exercism-javascript that referenced this pull request Oct 22, 2021
Co-authored-by: Derk-Jan Karrenbeld <derk-jan+github@karrenbeld.info>
Co-authored-by: Erik Schierboom <erik_schierboom@hotmail.com>
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.

2 participants