Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Integration tests #358

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Integration tests #358

wants to merge 5 commits into from

Conversation

bconn98
Copy link
Collaborator

@bconn98 bconn98 commented Mar 3, 2024

Add integration level tests exercising the full configuration format

Covers toml, json, and yaml Additionally added an intentionally malformed configuration & test.

@bconn98 bconn98 requested review from estk and gadunga as code owners March 3, 2024 02:28
@codecov-commenter
Copy link

codecov-commenter commented Mar 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.69%. Comparing base (8ab1b34) to head (1260ca0).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #358      +/-   ##
==========================================
+ Coverage   63.39%   67.69%   +4.30%     
==========================================
  Files          24       24              
  Lines        1557     1560       +3     
==========================================
+ Hits          987     1056      +69     
+ Misses        570      504      -66     

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

tests/color_control.rs Outdated Show resolved Hide resolved
@bconn98 bconn98 force-pushed the integration_tests branch from 8bbfba2 to 1260ca0 Compare March 4, 2024 03:34
@estk estk enabled auto-merge (squash) April 1, 2024 17:01
@estk
Copy link
Owner

estk commented Jul 9, 2024

blocked on #367

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants