Skip to content

BasicMathsTestSuite has some duplicated tests for logn section #334

Description

@YYDan

Description

logn_test has a pre-existing copy-paste bug in that asserts #34#36 are byte-identical in behaviour to #19#21 — logn(-2, _vReal/_vInt/_vInt64).

They sit under a // negative values heading where every other assert negates the value (logn(5, -_vReal)), so #34#36 were clearly meant to be logn(-2, -_vReal) etc. but instead just repeat the earlier negative-base cases.

Expected Change

That second set of tests is fixed to actually test what they claim to

Script Name

BasicMathsTestSuite

Fact/Assert Name

See above

How reliably does this issue happen?

Always

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions