Too Long; Didn't Read
Using strings is bad practice — use integers instead of strings. This makes the test take a longer time to understand meaning and slow down. The name of the test might help, but as with other documentation, it will eventually change and be out-of-sync with the code. Irrelevant values on the other hand have the value `"irrelevant"`, which tells me I can safely ignore these in this test. Instead of using strings, use a meaningful number and a meaningful name.
Share Your Thoughts