diff test/test_actions.py @ 6196:56854f96d805

Cover a couple of more paths in Retire and Restore.
author John Rouillard <rouilj@ieee.org>
date Fri, 12 Jun 2020 17:31:41 -0400
parents 8ef8aa3d485c
children 45ba6b71f1cf
line wrap: on
line diff
--- a/test/test_actions.py	Thu Jun 11 23:10:25 2020 -0400
+++ b/test/test_actions.py	Fri Jun 12 17:31:41 2020 -0400
@@ -88,8 +88,22 @@
         self.client._ok_message = []
         RetireAction(self.client).handle()
         self.assertTrue(len(self.client._ok_message) == 1)
+        self.assertTrue(not self.client.nodeid == None)
+
+        self.client.template = 'index'
+        self.client._ok_message = []
+        RetireAction(self.client).handle()
+        self.assertTrue(len(self.client._ok_message) == 1)
+        self.assertTrue(self.client.nodeid == None)
+
+        self.client.env={}
+        self.client.env['REQUEST_METHOD'] = 'GET'
+        self.client._ok_message = []
+        self.assertRaises(Reject, RetireAction(self.client).execute)
+
 
     def testNoPermission(self):
+        self.client.classname='user'
         self.assertRaises(Unauthorised, RetireAction(self.client).execute)
 
     def testDontRetireAdminOrAnonymous(self):
@@ -113,6 +127,18 @@
         self.client._ok_message = []
         RestoreAction(self.client).handle()
         self.assertTrue(len(self.client._ok_message) == 1)
+        self.assertTrue(not self.client.nodeid == None)
+
+        self.client.template = 'index'
+        self.client._ok_message = []
+        RestoreAction(self.client).handle()
+        self.assertTrue(len(self.client._ok_message) == 1)
+        self.assertTrue(self.client.nodeid == None)
+
+        self.client.env={}
+        self.client.env['REQUEST_METHOD'] = 'GET'
+        self.client._ok_message = []
+        self.assertRaises(Reject, RestoreAction(self.client).execute)
 
     def testNoPermission(self):
         self.assertRaises(Unauthorised, RestoreAction(self.client).execute)

Roundup Issue Tracker: http://roundup-tracker.org/