Challenge 6: Word Frequency Counter - Test Case 3

[pc386]

There is one test case for Challenge 6: Word Frequency Counter which I disagree with.

=== RUN   TestCountWordFrequency/Test_Case_#3
    solution_test.go:84: Expected {
          "go": 4,
          "learn": 1,
          "lets": 1,
          "programming": 1,
        } but got {
          "go": 4,
          "learn": 1,
          "let's": 1,
          "programming": 1,
        }

As you can see, the test expects lets instead of let's. But in my opinion, they are not the same words. I think it should stay let's.

[dlwyatt]

Not only that, the problem statement says words are defined as sequences of letters and digits. Strictly speaking that ' should be a separator between "let" and "s" if we're following those rules. I tried the AI code review for giggles and it even considered it a flaw that I made this test case pass by first not splitting on apostrophes, then removing them from the word map keys later. 😆