Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

.expect(status) should assert only status #40

Merged
merged 1 commit into from Dec 7, 2012

Conversation

Projects
None yet
3 participants
Contributor

eldargab commented Dec 2, 2012

Otherwise we have a gotcha:

.expect(200)
.end(function (err, res) {
  if (err) return done(err)  // currently yields expected body to be undefined
})
Contributor

eldargab commented Dec 6, 2012

Could you review this or #41?

Contributor

fengmk2 commented Dec 7, 2012

@visionmedia Please review this. It cause by my last pull request. I'm sorry guys...

@tj tj added a commit that referenced this pull request Dec 7, 2012

@tj tj Merge pull request #40 from eldargab/fix
.expect(status) should assert only status
490d246

@tj tj merged commit 490d246 into visionmedia:master Dec 7, 2012

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