Skip to content

add a test for the mod helper#478

Open
c-tonneslan wants to merge 1 commit into
Masterminds:masterfrom
c-tonneslan:test/add-mod-test
Open

add a test for the mod helper#478
c-tonneslan wants to merge 1 commit into
Masterminds:masterfrom
c-tonneslan:test/add-mod-test

Conversation

@c-tonneslan

Copy link
Copy Markdown

Closes #458.

`mod` has been documented in `docs/math.md` since the math docs were written but never covered by tests, so a regression would have slipped through. `TestMod` exercises a couple of representative inputs to match the surrounding `TestDiv` / `TestMul` style and pins down the operand order (the piped value is the right-hand operand).

mod has been documented since the math docs were written but never
covered by tests, so a regression would have slipped through. Drop
in a TestMod that exercises a couple of representative inputs to
match the surrounding TestDiv/TestMul style and pin down the
operand order (the piped value is the right-hand operand).

Closes Masterminds#458.

Signed-off-by: Charlie Tonneslan <cst0520@gmail.com>
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.

Missing Modulo tests

1 participant