Hi Kushagra,

please apply: (settings X flags on the .py file)
diff --git a/testenv/Test-reject-secure-cookie.py b/testenv/Test-reject-
secure-cookie.py
old mode 100644
new mode 100755

With the attached patch you can continue testing your code (Test--rejected-
log.py still fails here).

@Darshit Please review the attached patch (It's just a work-for-me patch)


Regards, Tim

On Sunday 14 February 2016 07:41:08 Kushagra Singh wrote:
> I'm unable to pin point the reason why its not failing. Checking the
> hook file, an error is definitely raised when the contents don't
> match. That shows up in the log as well. Where exactly should I be
> looking?
> 
> Kushagra
>From 703dc9cb76dcbd7a1685aef39b12b07d9fa3dcdf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tim=20R=C3=BChsen?= <[email protected]>
Date: Wed, 9 Mar 2016 11:03:08 +0100
Subject: [PATCH] Fix return value of python HTTP tests

* testenv/test/http_test.py: Return value from HTTP.begin() method
---
 testenv/test/http_test.py | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/testenv/test/http_test.py b/testenv/test/http_test.py
index 5a13d4f..50398dd 100644
--- a/testenv/test/http_test.py
+++ b/testenv/test/http_test.py
@@ -39,12 +39,13 @@ class HTTPTest(BaseTest):
             # If any exception occurs, self.__exit__ will be immediately called.
             # We must call the parent method in the end in order to verify
             # whether the tests succeeded or not.
+            self.tests_passed = False
             if self.ready:
                 self.do_test()
+                self.tests_passed = True
                 print_green("Test Passed.")
-            else:
-                self.tests_passed = False
-            super(HTTPTest, self).begin()
+
+        return 0 if self.tests_passed else 100
 
     def instantiate_server_by(self, protocol):
         server = {HTTP: HTTPd,
-- 
2.7.0

Reply via email to