Skip to content

Add tests for config#2976

Open
dvermd wants to merge 1 commit into
polybar:masterfrom
dvermd:config_test
Open

Add tests for config#2976
dvermd wants to merge 1 commit into
polybar:masterfrom
dvermd:config_test

Conversation

@dvermd

@dvermd dvermd commented Jun 10, 2023

Copy link
Copy Markdown
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Other: Add tests for config class

Description

Testing dereference* functions is not part of this PR. Only public methods are tested.

Related Issues & Documents

Documentation (check all applicable)

  • This PR requires changes to the Wiki documentation (describe the changes)
  • This PR requires changes to the documentation inside the git repo (please add them to the PR).
  • Does not require documentation changes

@codecov

codecov Bot commented Jun 10, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 14.17%. Comparing base (2cd0809) to head (4bd3cfa).
Report is 70 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2976      +/-   ##
==========================================
+ Coverage   12.69%   14.17%   +1.47%     
==========================================
  Files         160      160              
  Lines       12530    12532       +2     
==========================================
+ Hits         1591     1776     +185     
+ Misses      10939    10756     -183     
Flag Coverage Δ
unittests 14.17% <ø> (+1.47%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant