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

test: test the transformation extensions #160

Merged
merged 1 commit into from
Jul 1, 2023
Merged

Conversation

jnikula
Copy link
Owner

@jnikula jnikula commented Jul 1, 2023

Use the built-in extensions to handle javadoc and napoleon when running tests via Sphinx, and use the conversions directly when running the parser tests. This extends test coverage on the recommended ways to handle syntax transformations.

Before:

Name Stmts Miss Cover

src/hawkmoth/ext/javadoc/init.py 12 12 0%
src/hawkmoth/ext/napoleon/init.py 11 11 0%
src/hawkmoth/ext/transformations/init.py 17 2 88%

TOTAL 850 103 88%

After:

Name Stmts Miss Cover

src/hawkmoth/ext/javadoc/init.py 12 0 100%
src/hawkmoth/ext/napoleon/init.py 11 0 100%
src/hawkmoth/ext/transformations/init.py 17 8 53%

TOTAL 850 86 90%

@jnikula jnikula requested a review from BrunoMSantos July 1, 2023 09:41
Use the built-in extensions to handle javadoc and napoleon when running
tests via Sphinx, and use the conversions directly when running the
parser tests. This extends test coverage on the recommended ways to
handle syntax transformations.

Before:

Name                                           Stmts   Miss  Cover
------------------------------------------------------------------
src/hawkmoth/ext/javadoc/__init__.py              12     12     0%
src/hawkmoth/ext/napoleon/__init__.py             11     11     0%
src/hawkmoth/ext/transformations/__init__.py      17      2    88%
------------------------------------------------------------------
TOTAL                                            850    103    88%

After:

Name                                           Stmts   Miss  Cover
------------------------------------------------------------------
src/hawkmoth/ext/javadoc/__init__.py              12      0   100%
src/hawkmoth/ext/napoleon/__init__.py             11      0   100%
src/hawkmoth/ext/transformations/__init__.py      17      8    53%
------------------------------------------------------------------
TOTAL                                            850     86    90%
@jnikula jnikula force-pushed the test-use-extensions branch from b940d75 to ab75062 Compare July 1, 2023 11:09
@BrunoMSantos
Copy link
Collaborator

Nice one! Looks good to me ;)

@jnikula jnikula merged commit 4d29d5b into master Jul 1, 2023
@jnikula jnikula deleted the test-use-extensions branch July 4, 2023 08:59
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