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: Develop out FIDO tests with pytest #191

Closed
wants to merge 7 commits into from

Conversation

ross-spencer
Copy link

Demonstrating use of pytest with Carl's updates in #190

Connected to #189

carlwilson and others added 5 commits May 15, 2020 00:33
- added simple test, with no assert, for file identification; and
- added similar for stream identification which demonstrates hang.
- fixed termination condition in `blocking_read`.
Bring in a CSV reader to consume the FIDO output and make various
assertions about the result's validity.
@ross-spencer ross-spencer force-pushed the dev/develop-fido-test-w-pytest branch from f5adcce to 3b9e73f Compare May 25, 2020 03:18
@carlwilson carlwilson changed the base branch from rc/1.6 to main September 2, 2024 15:23
@carlwilson
Copy link
Member

Closed as contained in #226

@carlwilson carlwilson closed this Nov 14, 2024
Copy link

codecov bot commented Nov 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 34.00%. Comparing base (cd85271) to head (a9e4fdc).
Report is 39 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #191       +/-   ##
===========================================
+ Coverage   23.42%   34.00%   +10.58%     
===========================================
  Files          11       11               
  Lines        1490     1491        +1     
===========================================
+ Hits          349      507      +158     
+ Misses       1141      984      -157     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

2 participants