Small test case fixes for "annalyns infiltration" exercise

Programming Go
1

Hello,
I’ve created two pull requests making small fixes in the test cases for “annalyns-infiltration” exercise.

  1. Fixes error message
    annalyns-infiltration: Fix error message in CanFastAttack test by rahulrao0209 · Pull Request #2895 · exercism/go · GitHub

  2. Removes a duplicate test case
    annalyns-infiltration: Remove duplicate test case from TestCanFreePri… by rahulrao0209 · Pull Request #2896 · exercism/go · GitHub

I’ve tried to adhere to the contributing guidelines, please try taking a look at the pull requests and merge if changes are acceptable.

Thank you!!

1 Like
2

The first one seems okay since the error message should be showing what was passed into CanAttack, what was returned, and what was expected. Currently, the message shows what was passed into CanAttack twice and then what was expected. Therefore that’s confusing, and it probably was just a simple copy and paste error.

For the second one, “Knight and archer are awake. Prisoner is sleeping. Dog is not present” appears on lines 239-245 & 270-277 with the same inputs so yeah that test is duplicated.

The Go maintainers can take this from here. :slight_smile:

3

Not sure why this got flagged as spam.

4

The system got grumpy about you adding too many links to fast. I bumped your trust level up one level, which should help.

1 Like
5

Thanks so much!

6

These have both been merged

1 Like