Skip to content

[FIX] test_owmosaic: Cleanup/join threads on test tear down#3040

Merged
astaric merged 1 commit intobiolab:masterfrom
ales-erjavec:fixes/test-owmosaic-join
May 24, 2018
Merged

[FIX] test_owmosaic: Cleanup/join threads on test tear down#3040
astaric merged 1 commit intobiolab:masterfrom
ales-erjavec:fixes/test-owmosaic-join

Conversation

@ales-erjavec
Copy link
Copy Markdown
Contributor

Issue

VizRank tests for OWMosaic leave threads running after test completion influencing other tests via reduced performance and resulting TimeoutErrors.

For instance, run in bash:

$ python -m unittest  Orange.widgets.visualize.tests.test_owmosaic.MosaicVizRankTests{,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,}
Description of changes

Cleanup/join threads on test tear down

Includes
  • Code changes
  • Tests
  • Documentation

@codecov-io
Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (master@a8ea66b). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #3040   +/-   ##
=========================================
  Coverage          ?   82.31%           
=========================================
  Files             ?      335           
  Lines             ?    57758           
  Branches          ?        0           
=========================================
  Hits              ?    47541           
  Misses            ?    10217           
  Partials          ?        0

@astaric
Copy link
Copy Markdown
Member

astaric commented May 24, 2018

Nice catch. How did you figure it out?

@astaric astaric merged commit 3a68781 into biolab:master May 24, 2018
@ales-erjavec ales-erjavec deleted the fixes/test-owmosaic-join branch May 29, 2018 07:10
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