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

[FIX] Fix tests #1698

Merged
merged 4 commits into from
Nov 2, 2016
Merged

[FIX] Fix tests #1698

merged 4 commits into from
Nov 2, 2016

Conversation

ales-erjavec
Copy link
Contributor

  • Fix tests so they run with PyQt5, without sql backend and on Windows.
  • Fix QtWebEngine availability test in canvasmain.py

@ales-erjavec
Copy link
Contributor Author

These were originally part of #1693

@codecov-io
Copy link

codecov-io commented Oct 28, 2016

Current coverage is 88.86% (diff: 100%)

Merging #1698 into master will not change coverage

@@             master      #1698   diff @@
==========================================
  Files            82         82          
  Lines          8802       8802          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           7822       7822          
  Misses          980        980          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 2fb5812...d78d5dc

* Use full local file url (with file: schema).
  On Qt5 QUrl("/abs/path") is not considered to be a local file
* Use `os.path.samepath` to check path equality (windows)
Avoid an "ARPACK error 3: No shifts could be applied ..." when running
tests on Windows.
@astaric astaric merged commit 3d865bc into biolab:master Nov 2, 2016
@astaric astaric modified the milestone: 3.3.9 Nov 28, 2016
@ales-erjavec ales-erjavec deleted the test-fixes branch January 27, 2017 10:40
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