search

altendky / qtrio

QTrio - a library bringing Qt GUIs together with async and await via Trio
https://qtrio.readthedocs.io/
Other
40 stars 4 forks source link

use .unwrap() not .value in tests #249

Closed altendky closed 3 years ago

altendky commented 3 years ago

This makes the error visible in cases where the outcome is not a value at all.

codecov[bot] commented 3 years ago

Codecov Report

Merging #249 (f741db8) into master (2814301) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #249   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           33        33           
  Lines         1960      1960           
  Branches       128       128           
=========================================
  Hits          1960      1960
Impacted Files Coverage Δ
qtrio/_tests/test_core.py 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2814301...f741db8. Read the comment docs.

altendky commented 3 years ago

See the docs at https://outcome.readthedocs.io/en/stable/api.html.