Rename new to parsed on partial request

Author: kyleect

src/models/partial_request.rs

@@ -26,7 +26,7 @@ impl fmt::Display for PartialHttpRequest {
 }
 
 impl PartialHttpRequest {
-    pub fn new(
+    pub fn parsed(
         message: &str,
         method: Option<Range<usize>>,
         uri: Option<Range<usize>>,
@@ -195,7 +195,7 @@ where
     F: Fn(&str) -> FirstLineParts,
 {
     if input.trim().is_empty() {
-        return Ok(PartialHttpRequest::new(
+        return Ok(PartialHttpRequest::parsed(
             input,
             None,
             None,
@@ -220,7 +220,7 @@ where
 
     let body_span = get_span_extent_from_spans(body_spans);
 
-    Ok(PartialHttpRequest::new(
+    Ok(PartialHttpRequest::parsed(
         input,
         method,
         uri,
@@ -300,31 +300,31 @@ mod tests {
     #[test]
     #[should_panic]
     fn verifies_out_of_bounds_method_span() {
-        PartialHttpRequest::new("", Some(1..2), None, None, vec![], None);
+        PartialHttpRequest::parsed("", Some(1..2), None, None, vec![], None);
     }
 
     #[test]
     #[should_panic]
     fn verifies_inverted_method_span() {
-        PartialHttpRequest::new("", Some(2..1), None, None, vec![], None);
+        PartialHttpRequest::parsed("", Some(2..1), None, None, vec![], None);
     }
 
     #[test]
     #[should_panic]
     fn verifies_out_of_bounds_uri_span() {
-        PartialHttpRequest::new("", None, Some(1..2), None, vec![], None);
+        PartialHttpRequest::parsed("", None, Some(1..2), None, vec![], None);
     }
 
     #[test]
     #[should_panic]
     fn verifies_inverted_uri_span() {
-        PartialHttpRequest::new("", None, Some(2..1), None, vec![], None);
+        PartialHttpRequest::parsed("", None, Some(2..1), None, vec![], None);
     }
 
     #[test]
     #[should_panic]
     fn verifies_method_span_overlaps_uri_span() {
-        PartialHttpRequest::new(
+        PartialHttpRequest::parsed(
             "GET https://example.com",
             Some(0..3),
             Some(2..10),
@@ -337,45 +337,45 @@ mod tests {
     #[test]
     #[should_panic]
     fn verifies_out_of_bounds_http_version_span() {
-        PartialHttpRequest::new("", None, None, Some(1..2), vec![], None);
+        PartialHttpRequest::parsed("", None, None, Some(1..2), vec![], None);
     }
 
     #[test]
     #[should_panic]
     fn verifies_inverted_http_version_span() {
-        PartialHttpRequest::new("", None, None, Some(2..1), vec![], None);
+        PartialHttpRequest::parsed("", None, None, Some(2..1), vec![], None);
     }
 
     #[test]
     #[should_panic]
     fn verifies_out_of_bounds_header_span() {
-        PartialHttpRequest::new("", None, None, None, vec![1..2], None);
+        PartialHttpRequest::parsed("", None, None, None, vec![1..2], None);
     }
 
     #[test]
     #[should_panic]
     fn verifies_inverted_header_span() {
-        PartialHttpRequest::new("", None, None, None, vec![2..1], None);
+        PartialHttpRequest::parsed("", None, None, None, vec![2..1], None);
     }
 
     #[test]
     #[should_panic]
     fn verifies_out_of_bounds_body_span() {
-        PartialHttpRequest::new("", None, None, None, vec![], Some(1..2));
+        PartialHttpRequest::parsed("", None, None, None, vec![], Some(1..2));
     }
 
     #[test]
     #[should_panic]
     fn verifies_inverted_body_span() {
-        PartialHttpRequest::new("", None, None, None, vec![], Some(2..1));
+        PartialHttpRequest::parsed("", None, None, None, vec![], Some(2..1));
     }
 
     #[test]
     fn implements_default() {
         let partial = PartialHttpRequest::default();
 
         assert_eq!(
-            PartialHttpRequest::new(
+            PartialHttpRequest::parsed(
                 "GET https://example.com HTTP/1.1",
                 Some(0..3),
                 Some(4..23),

src/models/request.rs

@@ -124,7 +124,7 @@ mod from_partial_request_tests {
 
     #[test]
     fn from_partial_request_get() {
-        let partial_request = PartialHttpRequest::new(
+        let partial_request = PartialHttpRequest::parsed(
             r#"
         GET https://example.com HTTP/1.1
         x-api-key: abc123

tests/parse_tests.rs

@@ -13,7 +13,7 @@ fn parse_empty_request() {
     let partial = PartialHttpRequest::from_str(&content).expect("should be parsable");
 
     assert_eq!(
-        PartialHttpRequest::new(&content, None, None, None, vec![], None),
+        PartialHttpRequest::parsed(&content, None, None, None, vec![], None),
         partial
     );
 }
@@ -26,7 +26,7 @@ fn parse_whitespace_request() {
     let partial = PartialHttpRequest::from_str(&content).expect("should be parsable");
 
     assert_eq!(
-        PartialHttpRequest::new(&content, None, None, None, vec![], None),
+        PartialHttpRequest::parsed(&content, None, None, None, vec![], None),
         partial
     );
 }
@@ -39,7 +39,7 @@ fn parse_get_request() {
     let partial = PartialHttpRequest::from_str(&content).expect("should be parsable");
 
     assert_eq!(
-        PartialHttpRequest::new(
+        PartialHttpRequest::parsed(
             include_str!("../tests/fixtures/get.request"),
             Some(0..3),
             Some(4..23),
@@ -72,7 +72,7 @@ fn parse_get_without_http_version_request() {
     let partial = PartialHttpRequest::from_str(&content).expect("should be parsable");
 
     assert_eq!(
-        PartialHttpRequest::new(
+        PartialHttpRequest::parsed(
             include_str!("../tests/fixtures/get_without_http_version.request"),
             Some(0..3),
             Some(4..23),
@@ -105,7 +105,7 @@ fn parse_get_with_headers_request() {
     let partial = PartialHttpRequest::from_str(&content).expect("should be parsable");
 
     assert_eq!(
-        PartialHttpRequest::new(
+        PartialHttpRequest::parsed(
             include_str!("../tests/fixtures/get_with_headers.request"),
             Some(0..3),
             Some(4..23),
@@ -144,7 +144,7 @@ fn parse_post_with_headers_and_body_request() {
     let body = Some(53..64);
 
     assert_eq!(
-        PartialHttpRequest::new(&content, method, uri, http_version, headers, body),
+        PartialHttpRequest::parsed(&content, method, uri, http_version, headers, body),
         partial
     );
 
@@ -170,7 +170,7 @@ fn parse_post_with_body_request() {
     let partial = PartialHttpRequest::from_str(&content).expect("should be parsable");
 
     assert_eq!(
-        PartialHttpRequest::new(
+        PartialHttpRequest::parsed(
             include_str!("../tests/fixtures/post_with_body.request"),
             Some(0..4),
             Some(5..24),
@@ -203,7 +203,7 @@ fn parse_get_with_multiple_spaces_request() {
     let partial = PartialHttpRequest::from_str(&content).expect("should be parsable");
 
     assert_eq!(
-        PartialHttpRequest::new(
+        PartialHttpRequest::parsed(
             &content,
             Some(0..3),
             Some(5..24),