Add some more testcases

Author: lyc8503

Lib/http/server.py

@@ -951,7 +951,7 @@ def parse_range(self):
         range_header = self.headers.get('range')
         if not range_header:
             return None
-        m = re.match(RANGE_REGEX_PATTERN, range_header)
+        m = RANGE_REGEX_PATTERN.match(range_header)
         if not m:
             return None
 

Lib/test/test_httpservers.py

@@ -544,6 +544,7 @@ def test_range_get(self):
         self.assertEqual(response.getheader('accept-ranges'), 'bytes')
         self.check_status_and_reason(response, HTTPStatus.OK, data=self.data)
 
+        # valid ranges
         response = self.request(self.base_url + '/test', headers={'Range': 'bytes=3-12'})
         self.assertEqual(response.getheader('content-range'), 'bytes 3-12/30')
         self.assertEqual(response.getheader('content-length'), '10')
@@ -559,6 +560,18 @@ def test_range_get(self):
         self.assertEqual(response.getheader('content-length'), '5')
         self.check_status_and_reason(response, HTTPStatus.PARTIAL_CONTENT, data=self.data[25:])
 
+        response = self.request(self.base_url + '/test', headers={'Range': 'bytes=29-29'})
+        self.assertEqual(response.getheader('content-range'), 'bytes 29-29/30')
+        self.assertEqual(response.getheader('content-length'), '1')
+        self.check_status_and_reason(response, HTTPStatus.PARTIAL_CONTENT, data=self.data[29:])
+
+        # end > file size
+        response = self.request(self.base_url + '/test', headers={'Range': 'bytes=25-100'})
+        self.assertEqual(response.getheader('content-range'), 'bytes 25-29/30')
+        self.assertEqual(response.getheader('content-length'), '5')
+        self.check_status_and_reason(response, HTTPStatus.PARTIAL_CONTENT, data=self.data[25:])
+
+        # invalid ranges
         response = self.request(self.base_url + '/test', headers={'Range': 'bytes=100-200'})
         self.check_status_and_reason(response, HTTPStatus.REQUESTED_RANGE_NOT_SATISFIABLE)
 
@@ -568,6 +581,9 @@ def test_range_get(self):
         response = self.request(self.base_url + '/test', headers={'Range': 'bytes=wrong format'})
         self.check_status_and_reason(response, HTTPStatus.OK, data=self.data)
 
+        response = self.request(self.base_url + '/test', headers={'Range': 'bytes=-'})
+        self.check_status_and_reason(response, HTTPStatus.OK, data=self.data)
+
     def test_head(self):
         response = self.request(
             self.base_url + '/test', method='HEAD')