-
-
Notifications
You must be signed in to change notification settings - Fork 3
[BUG] Incorrectly passing tests #149
Copy link
Copy link
Open
Labels
ASAPShould be done As Soon As Possible.Should be done As Soon As Possible.bugSomething isn't workingSomething isn't workingenhancementNew feature or requestNew feature or requesthelp wantedExtra attention is neededExtra attention is needed
Description
Activity
Metadata
Metadata
Assignees
Labels
ASAPShould be done As Soon As Possible.Should be done As Soon As Possible.bugSomething isn't workingSomething isn't workingenhancementNew feature or requestNew feature or requesthelp wantedExtra attention is neededExtra attention is needed
Describe the bug
This tool was probably coded (initially) to check if the expected output is part of the actual output (or the other way around). This means they can differ but the test still pass because the expected output is contained in the actual output. Right after the mvp version of this tool was finished, it already was being used expecting the expected and actual outputs to be the same for a test to pass (instead of contained in each other). In other words, this behavior is now an unhelpful bug.
To Reproduce
TODO
Expected behavior
A test should only pass if all the lines are the same, in the same order with all characters equal. The only ignored characters should be:
eg:
is the same as:
but different from
Screenshots
TODO
Desktop (please complete the following information):
Additional context
Tests may incorrectly pass when they shouldn't if the user prints additional characters before or after the block containing the main content.