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.

Sign up for GitHub

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

Jump to bottom

Rename function test_connect_helpers.test_synapse #88

Closed
heplesser opened this issue Aug 19, 2015 · 0 comments · Fixed by #91
Closed

Rename function test_connect_helpers.test_synapse #88

heplesser opened this issue Aug 19, 2015 · 0 comments · Fixed by #91

Comments

@heplesser
Copy link
Contributor

Function test_connect_helpers.test_synapse is a helper function, it does not implement a test. We therefore need to change the name to something not containing test, so that nosetests won't run it as a test.

This has lead to failing tests for some time (see eg Travis build 141.8), which were not reported due to #89.
@heplesser heplesser mentioned this issue Aug 19, 2015
Closed
heplesser added a commit to heplesser/nest-simulator that referenced this issue Aug 19, 2015
@heplesser
Renamed helper function test_synapse to check_synapse to avoid it…
3119fa7 
… being picked up as a test by nosetests. This should fix nest#88.
This was referenced Aug 19, 2015
Merged
Merged
@jougs jougs closed this as completed in #91 Aug 19, 2015
janhahne pushed a commit to janhahne/nest-simulator that referenced this issue Jun 1, 2018
@suku248
Merge pull request nest#88 from jakobj/fix/sp_sorts_connections
943648e 
Do not sort connection in structural plasticity update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Renamed helper function test_synapse to check_synapse to avoid it… heplesser/nest-simulator
1 participant
@heplesser