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

Baseline should be update to not depend anymore on smalltalk/Unicode #56

Merged
merged 3 commits into from Apr 6, 2020

Conversation

akgrant43
Copy link
Contributor

Load Unicode from github repository (was SmalltalkHub).

Fixes: #55

- Reduce the sleep time in testReadingFromStdoutAfterCommandFinishesDoesNotBlocksVM to 1 second.   There's no value in holding up the tests for 15 seconds and only depending on the process being finished.
- #numberOfOpenFiles only cound FIFO (pipes) as the tests are about redirecting stdio and counting all files isn't reliable (causes periodic failures on the CI)
- #getStreamsInfoForRunningTest update comments for Pharo 8
@akgrant43
Copy link
Contributor Author

Fixed one flakey test with 1a5c026

The other test failure appears to be unrelated to this PR.

Modify the command as per https://docs.travis-ci.com/user/common-build-problems/
in particular don't try and run it as a separate command.
@akgrant43
Copy link
Contributor Author

02a5dfe is successful in that it now times out after 1 hour instead of 10 minutes.

But the test is randomly hanging (not related to this PR).

@akgrant43 akgrant43 merged commit cad939d into pharo-contributions:master Apr 6, 2020
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.

Baseline should be update to not depend anymore on smalltalk/Unicode
1 participant