#190 Checking the status of response before assertion count

Failed Responses don't have assertions and the error hides that the
status is not success

Author: thejuan

tests/src/OneLogin/saml2_tests/response_test.py

@@ -1396,10 +1396,10 @@ def testStatusCheckBeforeAssertionCheck(self):
         Tests the status of a response is checked before the assertion count. As failed statuses will have no assertions
         """
         settings = OneLogin_Saml2_Settings(self.loadSettingsJSON())
-        xml_2 = self.file_contents(join(self.data_path, 'responses', 'invalids', 'status_code_responder.xml.base64'))
-        response_2 = OneLogin_Saml2_Response(settings, xml_2)
+        xml = self.file_contents(join(self.data_path, 'responses', 'invalids', 'status_code_responder.xml.base64'))
+        response = OneLogin_Saml2_Response(settings, xml)
         with self.assertRaisesRegexp(OneLogin_Saml2_ValidationError, 'The status code of the Response was not Success, was Responder'):
-            response_2.is_valid(self.get_request_data(), raise_exceptions=True)
+            response.is_valid(self.get_request_data(), raise_exceptions=True)
 
 
 if __name__ == '__main__':