Whenever possible, avoid doing `try/catch` in the test suite #30

DilumAluthge · 2022-05-07T09:22:09Z

This can prevent us from noticing test failures.

codecov · 2022-05-07T09:25:48Z

Codecov Report

Merging #30 (f1503b4) into main (a613238) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #30   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines           26        26           
=========================================
  Hits            26        26

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 a613238...f1503b4. Read the comment docs.

Whenever possible, avoid doing try/catch in the test suite

f1503b4

DilumAluthge requested a review from AshlinHarris May 7, 2022 09:23

DilumAluthge merged commit 43a8f17 into main May 7, 2022

DilumAluthge deleted the dpa/test-try-catch branch May 7, 2022 09:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Whenever possible, avoid doing `try/catch` in the test suite #30

Whenever possible, avoid doing `try/catch` in the test suite #30

DilumAluthge commented May 7, 2022 •

edited

Loading

codecov bot commented May 7, 2022 •

edited

Loading

Whenever possible, avoid doing try/catch in the test suite #30

Whenever possible, avoid doing try/catch in the test suite #30

Conversation

DilumAluthge commented May 7, 2022 • edited Loading

codecov bot commented May 7, 2022 • edited Loading

Codecov Report

Whenever possible, avoid doing `try/catch` in the test suite #30

Whenever possible, avoid doing `try/catch` in the test suite #30

DilumAluthge commented May 7, 2022 •

edited

Loading

codecov bot commented May 7, 2022 •

edited

Loading