-
Notifications
You must be signed in to change notification settings - Fork 1
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
Biodiverse tests #3
Comments
Can you list the failure diagnostics? It should note which tests fail higher up in the log. |
Test # 11
|
Test # 28
|
Test # 30
|
All the warnings about uninitialised values in Env.pm, split and regexes were fixed a few days ago in shawnlaffan/biodiverse@795717d Updating to that commit will (should) fix the t/30-Progress.t failures. I can add a new tag if needed. The failures in t/11-BaseData-import-feature-data.t look to be related to unicode file name tests. They won't affect anything so long as we keep to ascii characters in file names. I need to look into it further, though. The failing test in t/28-Randomisation.t seems to be difficult to get working properly on some systems. It is skipped under BSD systems and WSL on windows, and looks like it also needs to be be skipped under docker. The test was introduced for shawnlaffan/biodiverse#746 |
Some of the errors in t/11-BaseData-import-feature-data.t will be fixed by shawnlaffan/biodiverse@87b4b98 |
Great! Thank you!
Probably for now we can use a hash ID of the latest fix (or just the head of the master branch) and we'll specify stable version later. |
I've fixed the Now almost all tests (including
I'm not a Windows user, but I thought that WSL2 uses Docker as a backend. And the most common distro is Ubuntu as well. |
That's good news.
The tweak to WSL was for version 1. Running it under WSL2 the test passes, despite being marked as todo. I've modified the test to use different permissions as they might have been too stringent before. Can you run this one-liner in the shell and report the result?
|
The output is:
|
For my version the output is: linux |
Thanks for those. I don't think we can differentiate docker linux this way. I've modified the test in shawnlaffan/biodiverse@cc1f70a |
Currently these tests are failing (v.0.01, 814ef52):
The text was updated successfully, but these errors were encountered: