refactor HTTPResponse.Compare

pkg/drivers/response.go

@@ -30,20 +30,17 @@ func (resp *HTTPResponse) Compare(other core.Value) int64 {
 		return Compare(HTTPResponseType, other.Type())
 	}
 
-	otherObj := values.Parse(other)
+	otherResp := other.(*HTTPResponse)
-	respObj := values.Parse(resp)
 
-	// HTTPHeaders has it's own Compare
+	comp := resp.Headers.Compare(otherResp.Headers)
-	respObj.(*values.Object).Set(
+	if comp != 0 {
-		values.NewString("Headers"),
+		return comp
-		resp.Headers,
+	}
-	)
-	otherObj.(*values.Object).Set(
-		values.NewString("Headers"),
-		other.(*HTTPResponse).Headers,
-	)
 
-	return respObj.Compare(otherObj)
+	// it makes no sense to compare Status strings
+	// because they are always equal if StatusCode's are equal
+	return values.NewInt(resp.StatusCode).
+		Compare(values.NewInt(resp.StatusCode))
 }
 
 func (resp *HTTPResponse) Unwrap() interface{} {