Skip to content

improve coverage#2575

Merged
staticdev merged 1 commit into
PyCQA:mainfrom
Khabib73:tests/improve-coverage-literal
Jul 4, 2026
Merged

improve coverage#2575
staticdev merged 1 commit into
PyCQA:mainfrom
Khabib73:tests/improve-coverage-literal

Conversation

@Khabib73

@Khabib73 Khabib73 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Improve coverage in literal.py from 95% to 97%.

Added 2 tests case:

  • test_value_assignment_dict
  • test_value_assignment_unique_tuple

Before:

Name                        Stmts   Miss Branch BrPart  Cover   Missing
-----------------------------------------------------------------------
isort/literal.py               64      3     14      1    95%   66, 90, 115

After:

Name                        Stmts   Miss Branch BrPart  Cover   Missing
-----------------------------------------------------------------------
isort/literal.py               64      1     14      1    97%   66

@codecov

codecov Bot commented Jul 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.32%. Comparing base (a0cc97f) to head (cef9dcc).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2575      +/-   ##
==========================================
+ Coverage   99.25%   99.32%   +0.06%     
==========================================
  Files          41       41              
  Lines        3108     3108              
  Branches      674      674              
==========================================
+ Hits         3085     3087       +2     
+ Misses         13       11       -2     
  Partials       10       10              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Khabib73 Khabib73 force-pushed the tests/improve-coverage-literal branch from 6c67a24 to cef9dcc Compare July 4, 2026 08:23
@Khabib73

Khabib73 commented Jul 4, 2026

Copy link
Copy Markdown
Contributor Author

@DanielNoord Hi! Rebased on latest master, CI is green, ready to merge :)

@staticdev staticdev left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks a lot @Khabib73

@staticdev staticdev added this pull request to the merge queue Jul 4, 2026
Merged via the queue into PyCQA:main with commit 6dcdb84 Jul 4, 2026
26 checks passed
@Khabib73

Khabib73 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor Author

Thanks!

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