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

add tests for #15584 #15619

Merged
merged 7 commits into from
Oct 18, 2020
Merged

add tests for #15584 #15619

merged 7 commits into from
Oct 18, 2020

Conversation

n5m
Copy link
Contributor

@n5m n5m commented Oct 18, 2020

Adding test cases for #15584

@n5m
Copy link
Contributor Author

n5m commented Oct 18, 2020

builds.sr.ht: freebsd.yml failed because

.tasks/setup: line 9: git: command not found

@n5m n5m closed this Oct 18, 2020
@n5m n5m reopened this Oct 18, 2020
@ghost
Copy link

ghost commented Oct 18, 2020

@n5m it will be fixed by #15613

@Varriount Varriount closed this Oct 18, 2020
@Varriount Varriount reopened this Oct 18, 2020
joining with other test cases (e.g., a test case that invokes `RLock.initRLock()`) may cause this test to compile, when the goal is to test if these lines can be compiled in isolation
@n5m
Copy link
Contributor Author

n5m commented Oct 18, 2020

Tests have correctly been detected.

2020-10-18T16:06:51.0791190Z PASS: tests/stdlib/trlocks.nim C                                   ( 0.72 sec)
2020-10-18T16:07:16.0433168Z PASS: tests/stdlib/tuserlocks.nim C                                ( 0.88 sec)

@n5m n5m marked this pull request as ready for review October 18, 2020 16:25
@Varriount Varriount merged commit 78cbb52 into nim-lang:devel Oct 18, 2020
narimiran pushed a commit that referenced this pull request Oct 21, 2020
* add test for #15584

* Rename userlocks.nim to trlocks.nim

* add bug comment

* improve cmd

* reference bugfix, not bug

* add test that runs rlocks

* disallow joining

joining with other test cases (e.g., a test case that invokes `RLock.initRLock()`) may cause this test to compile, when the goal is to test if these lines can be compiled in isolation

(cherry picked from commit 78cbb52)
mildred pushed a commit to mildred/Nim that referenced this pull request Jan 11, 2021
* add test for nim-lang#15584

* Rename userlocks.nim to trlocks.nim

* add bug comment

* improve cmd

* reference bugfix, not bug

* add test that runs rlocks

* disallow joining

joining with other test cases (e.g., a test case that invokes `RLock.initRLock()`) may cause this test to compile, when the goal is to test if these lines can be compiled in isolation
irdassis pushed a commit to irdassis/Nim that referenced this pull request Mar 16, 2021
* add test for nim-lang#15584

* Rename userlocks.nim to trlocks.nim

* add bug comment

* improve cmd

* reference bugfix, not bug

* add test that runs rlocks

* disallow joining

joining with other test cases (e.g., a test case that invokes `RLock.initRLock()`) may cause this test to compile, when the goal is to test if these lines can be compiled in isolation
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