Skip to content

test(edalizer): cover parse_args and parameters - #793

Open
tymonx wants to merge 1 commit into
olofk:mainfrom
tymonx:feature/edalize-tests
Open

test(edalizer): cover parse_args and parameters#793
tymonx wants to merge 1 commit into
olofk:mainfrom
tymonx:feature/edalize-tests

Conversation

@tymonx

@tymonx tymonx commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

This PR:

  • Provides additional unit tests for the fusesoc.edalizer.Edalizer class
  • Improves code coverage from 55% to 78% for the fusesoc.edalizer module only
  • Tests all variants of passing command line arguments from FuseSoC to Edalize backend
  • Tests all variants of handling parameters

To run all tests related with the Edalizer class:

pytest ./tests/test_edalizer.py --cov fusesoc.edalizer

Previous coverage:

Name                  Stmts   Miss  Cover
-----------------------------------------
fusesoc/edalizer.py     387    173    55%
-----------------------------------------
TOTAL                   387    173    55%

Coverage from this PR:


Name                  Stmts   Miss  Cover
-----------------------------------------
fusesoc/edalizer.py     387     85    78%
-----------------------------------------
TOTAL                   387     85    78%

@olofk

olofk commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Excellent. Test coverage has been lagging behind. Been merging a lot of PRs lately though, so there's a merge conflict now. Could you rebase against latest main and resolve that first?

@tymonx
tymonx force-pushed the feature/edalize-tests branch from d5c61f5 to 1b2b256 Compare August 11, 2026 09:11
@tymonx

tymonx commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

Excellent. Test coverage has been lagging behind. Been merging a lot of PRs lately though, so there's a merge conflict now. Could you rebase against latest main and resolve that first?

@olofk Done 👍

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