Skip to content

[13.0][FIX] run doctests as part of standard tests - #296

Merged
OCA-git-bot merged 1 commit into
OCA:13.0from
nilshamerlinck:13.0_fix_doctests
Jan 4, 2021
Merged

[13.0][FIX] run doctests as part of standard tests#296
OCA-git-bot merged 1 commit into
OCA:13.0from
nilshamerlinck:13.0_fix_doctests

Conversation

@nilshamerlinck

Copy link
Copy Markdown
Contributor

Odoo v12 introduced test tags.

Since then doctests were not running anymore.

This PR introduces a wrapper for doctest.DocTestCase to get doctests to run again as part of standard tests.

@nilshamerlincknilshamerlinck changed the title [FIX] run doctests as part of standard tests[13.0][FIX] run doctests as part of standard testsJan 3, 2021

@guewenguewen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Uh, good catch!

@lmignonlmignon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @nilshamerlinck

@guewen

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 13.0-ocabot-merge-pr-296-by-guewen-bump-patch, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 5180e51. Thanks a lot for contributing to OCA. ❤️

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@nilshamerlinck@guewen@OCA-git-bot@lmignon