test: test intended code-paths #5186

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

arhart commented Mar 31, 2013

The tests did not agree with the test comments. Tests first and second
were both testing the !state.reading case. Now second tests the
state.reading && state.length case.

Fixes joyent/node#5183

@arhart arhart test: test intended code-paths
The tests did not agree with the test comments. Tests first and second
were both testing the !state.reading case. Now second tests the
state.reading && state.length case.

Fixes nodejs/node-v0.x-archive#5183
933f1f1

isaacs commented Apr 1, 2013

Landed on db8ce89. Thanks!

isaacs closed this Apr 1, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment