What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fixes some existing unit tests before making new ones.
What is the current behavior? (You can also link to an open issue here)
Currently the logic in the DiscardBadFishTests.cs causes them to fail due NullReferenceExceptions
What is the new behavior? (if this is a feature change)
The Tests now succeed or atleast no longer throws errors
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
…longer work as intended
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) Fixes some existing unit tests before making new ones.
What is the current behavior? (You can also link to an open issue here) Currently the logic in the DiscardBadFishTests.cs causes them to fail due NullReferenceExceptions
What is the new behavior? (if this is a feature change) The Tests now succeed or atleast no longer throws errors
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information: Closes #438