It gives meaning to the numbers. For example we use 1 for a location then
use it later to check what we expect is in location 1. Making it a variable
let's the code be easier to read.
On Jan 18, 2014 5:24 PM, "Taylor Wrobel" <notifications@github.com> wrote:

> In
> SoundStream-test/src/com/thelastcrusade/soundstream/net/message/MessengerTest.java:
>
> > @@ -53,6 +54,92 @@
> >      //TODO: add tests for partial messages, to make sure we handle the case where data isn't all there yet
> >
> >      @Test
> > +    public void testCancelMessage() throws Exception {
>
> I disagree that there's a need to make variables for numbers in tests that
> are not special. If anything we could make an "assert queue is empty"
> method that does the check with 0, but more than that I find just pedantic
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/TheLastCrusade/SoundStream/pull/330/files#r8990140>
> .
>