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(file-list): Use path.posix to fix tests when run on Windows #1728

Merged
merged 1 commit into from
Nov 29, 2015

Conversation

Volune
Copy link
Contributor

@Volune Volune commented Nov 28, 2015

Have file-list.js use path.posix instead of path itself, so the tests can expect a '/' separator on any platform.

Closes #1028

@GitCop
Copy link

GitCop commented Nov 28, 2015

There were the following issues with your Pull Request

  • Commit: 8172e3e
    • Your commit message body contains a line that is longer than 80 characters

Guidelines are available at http://karma-runner.github.io/0.13/dev/git-commit-msg.html


This message was auto-generated by https://gitcop.com

@Volune Volune force-pushed the fix-tests-on-windows branch 2 times, most recently from c0a8d56 to f87a651 Compare November 28, 2015 11:58
Have file-list.js use path.posix instead of path itself,
so the tests can expect a '/' separator on any platform.

Node 0.10 doesn't provide path.posix and will still fail on Windows.

Closes karma-runner#1028
@dignifiedquire
Copy link
Member

Thanks a lot! :octocat:

dignifiedquire added a commit that referenced this pull request Nov 29, 2015
test(file-list): Use path.posix to fix tests when run on Windows
@dignifiedquire dignifiedquire merged commit 8f61eb2 into karma-runner:master Nov 29, 2015
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.

3 participants