search

8thlight / cob_spec

A fitnesse suite for a web server
24 stars 33 forks source link

Add SimpleHead test #30

Closed Nayshins closed 8 years ago

Nayshins commented 9 years ago

This branch adds a simple head request test. The test sends a Head request for /file1, and checks if a content length header is returned without sending the response body.

trptcolin commented 9 years ago

This looks great and is a big improvement over the existing HEAD tests.

The one issue is that your server doesn't pass all the cob_spec tests in the add-simple-head branch. Your server passes 43915978af27075e0c3596b1443b0be185b238ba, but not this branch, which includes #25. Update your server to pass all the tests in this add-simple-head branch, and I'll verify and merge.

KevinLiddle commented 8 years ago

Closing due to old age, and the fact that I just merged a newer HEAD test PR a few minutes ago.