Write idiomatic Go table-driven tests — tests knowledge of Go testing conventions
Write a Go test file for the following function:
```go
// WordCount returns a map of each unique word in s to the number
// of times it appears. Words are case-insensitive and separated by
// whitespace. Punctuation attached to words is stripped.
func WordCount(s string) map[string]int
```
Requirements:
- Package: `wordcount` (test file: `wordcount_test.go`)
- Use Go's standard `testing` package — no third-party libraries
- Use a table-driven test with a `[]struct{ name, input string; want map[string]int }` slice
- Include at least 6 test cases: empty string, single word, mixed case, punctuation, repeated words, multi-word sentence
- Use `t.Run(tc.name, ...)` for subtests so failures are identifiable
- Use `reflect.DeepEqual` or `maps.Equal` to compare maps
- Add a benchmark `BenchmarkWordCount` that runs on a realistic sentence
Do the 6+ test cases cover empty input, case-insensitivity, punctuation stripping, and repeated words with correct expected counts? Is the map comparison correct (not == on maps)? Does the benchmark compile and follow Go benchmark conventions (b.N loop)?
Is the table-driven pattern used correctly (slice of structs, not parallel variables)? Are subtests used with t.Run? Is the test file in the right package (wordcount or wordcount_test)? Is the code idiomatic Go (no unnecessary imports, proper error handling)?
Are test case names descriptive ('empty string', 'mixed case punctuation')? Is there a comment on the test function explaining what it covers? Is the benchmark named and commented correctly?
# Test cases are logically correct (verified word counts, case-insensitivity, punctuation stripping) and use reflect.DeepEqual properly, but the map literal isn't gofmt-aligned, an unnecessary b.ResetTimer() is added with no setup work, and a non-standard unicode hyphen appears in a test name.
inspect full trace →# Solid table-driven tests with correct maps.Equal comparison and proper benchmark, but extra cases (contractions, apostrophes) rely on unverified assumptions about punctuation handling, and code lacks explanatory comments on the test/benchmark functions themselves.
inspect full trace →# Test cases and benchmark are well-designed and exceed the requirement, but the file incorrectly declares package wordcount_test and reimplements a stub WordCount that always returns nil, meaning the delivered file will fail most subtests unless manually edited to remove the stub and fix the package, contradicting the stated 'package: wordcount' requirement.
inspect full trace →# Test logic and case coverage are solid and benchmark is correct, but the code contains a fatal compile error (`t *test.T` instead of `*testing.T`) that would prevent the file from building; also lacks a dedicated comment on TestWordCount explaining its coverage.
inspect full trace →# The test file will not compile because 'reflect' is imported but never actually referenced (only mentioned in a comment), causing a Go compile error; additionally the apostrophe-handling test case ('It's') mismatches the model's own regex-based implementation output, meaning even after fixing the import the test would fail — despite good table-driven structure, many descriptive test names, and a solid benchmark.
inspect full trace →