Skip to content

Mutation testing trial#89

Draft
presidente20 wants to merge 3 commits into
devfrom
mutation-testing-trial
Draft

Mutation testing trial#89
presidente20 wants to merge 3 commits into
devfrom
mutation-testing-trial

Conversation

@presidente20

Copy link
Copy Markdown
Contributor

This PR contains a short trial of mutation testing including:

  • a markdown note summarising the trial (main content of the PR)
  • code changes to demonstrate configuration of the mutmut tool
  • code changes to fix mutants generated in a couple of examples

This is a Draft PR and not intended to be merged. If successful this could result in two PRs, one to add mutmut.py as a package and initial config and another for the test_number.py and test_convert.py modules containing changes to kill mutants.

If there is interest I can provide a demo the tool in use.

@nbarlowATI nbarlowATI 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.

Very interesting, and does sound like something we should run on as much of the codebase as feasible, at least once (and fix the resulting issues).
Hopefully as the codebase stabilises, we wouldn't break things as often, so we could just run it occasionally....
I agree with the strategy of testing it first on Starling though.

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