Ensure the instrumentedResponseWriter correctly sets the http

status in the context.

Signed-off-by: Richard Scothern <richard.scothern@gmail.com>
This commit is contained in:
Richard 2015-05-05 14:21:33 -07:00
parent 3a0d6faba1
commit 7f3a57fdbb

View file

@ -93,7 +93,7 @@ func TestURLPrefix(t *testing.T) {
} }
// TestLayerAPI conducts a full of the of the layer api. // TestLayerAPI conducts a full test of the of the layer api.
func TestLayerAPI(t *testing.T) { func TestLayerAPI(t *testing.T) {
// TODO(stevvooe): This test code is complete junk but it should cover the // TODO(stevvooe): This test code is complete junk but it should cover the
// complete flow. This must be broken down and checked against the // complete flow. This must be broken down and checked against the
@ -246,6 +246,16 @@ func TestLayerAPI(t *testing.T) {
t.Fatalf("response body did not pass verification") t.Fatalf("response body did not pass verification")
} }
// ----------------
// Fetch the layer with an invalid digest
badURL := strings.Replace(layerURL, "tarsum", "trsum", 1)
resp, err = http.Get(badURL)
if err != nil {
t.Fatalf("unexpected error fetching layer: %v", err)
}
checkResponse(t, "fetching layer bad digest", resp, http.StatusBadRequest)
// Missing tests: // Missing tests:
// - Upload the same tarsum file under and different repository and // - Upload the same tarsum file under and different repository and
// ensure the content remains uncorrupted. // ensure the content remains uncorrupted.