test: Improved code coverage
This commit is contained in:
		
							
								
								
									
										2
									
								
								cover.sh
									
									
									
									
									
								
							
							
						
						
									
										2
									
								
								cover.sh
									
									
									
									
									
								
							@@ -1,3 +1,3 @@
 | 
				
			|||||||
#!/usr/bin/env bash
 | 
					#!/usr/bin/env bash
 | 
				
			||||||
 | 
					
 | 
				
			||||||
nosetests --with-coverage --cover-package=falcon
 | 
					nosetests --with-coverage --cover-package=falcon --cover-min-percentage=90
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -48,6 +48,12 @@ class TestHeaders(helpers.TestSuite):
 | 
				
			|||||||
        content_length_header = ('Content-Length', content_length)
 | 
					        content_length_header = ('Content-Length', content_length)
 | 
				
			||||||
        self.assertThat(headers, Contains(content_length_header))
 | 
					        self.assertThat(headers, Contains(content_length_header))
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    def test_default_value(self):
 | 
				
			||||||
 | 
					        self._simulate_request(self.test_route)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					        value = self.resource.req.get_header('X-Not-Found', '876')
 | 
				
			||||||
 | 
					        self.assertEquals(value, '876')
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    def test_prefer_host_header(self):
 | 
					    def test_prefer_host_header(self):
 | 
				
			||||||
        self._simulate_request(self.test_route)
 | 
					        self._simulate_request(self.test_route)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -38,7 +38,6 @@ class UnauthorizedResource:
 | 
				
			|||||||
                                      'Missing or invalid token header.',
 | 
					                                      'Missing or invalid token header.',
 | 
				
			||||||
                                      'Token')
 | 
					                                      'Token')
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					 | 
				
			||||||
class NotFoundResource:
 | 
					class NotFoundResource:
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    def on_get(self, req, resp):
 | 
					    def on_get(self, req, resp):
 | 
				
			||||||
@@ -50,6 +49,13 @@ class MethodNotAllowedResource:
 | 
				
			|||||||
        raise falcon.HTTPMethodNotAllowed(['PUT'])
 | 
					        raise falcon.HTTPMethodNotAllowed(['PUT'])
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					class InternalServerErrorResource:
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    def on_get(self, req, resp):
 | 
				
			||||||
 | 
					        raise falcon.HTTPInternalServerError('Excuse Us', 'Something went'
 | 
				
			||||||
 | 
					                                             'boink!')
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					
 | 
				
			||||||
class TestHTTPError(helpers.TestSuite):
 | 
					class TestHTTPError(helpers.TestSuite):
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    def prepare(self):
 | 
					    def prepare(self):
 | 
				
			||||||
@@ -168,3 +174,9 @@ class TestHTTPError(helpers.TestSuite):
 | 
				
			|||||||
        self.assertEqual(self.srmock.status, falcon.HTTP_405)
 | 
					        self.assertEqual(self.srmock.status, falcon.HTTP_405)
 | 
				
			||||||
        self.assertEqual(body, [])
 | 
					        self.assertEqual(body, [])
 | 
				
			||||||
        self.assertIn(('Allow', 'PUT'), self.srmock.headers)
 | 
					        self.assertIn(('Allow', 'PUT'), self.srmock.headers)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    def test_500(self):
 | 
				
			||||||
 | 
					        self.api.add_route('/500', InternalServerErrorResource())
 | 
				
			||||||
 | 
					        body = self._simulate_request('/500')
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					        self.assertEqual(self.srmock.status, falcon.HTTP_500)
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user