--- a/test/test_actions.py	Sun Feb 29 00:35:55 2004 +0000
+++ b/test/test_actions.py	Sun Feb 29 00:41:51 2004 +0000
@@ -6,7 +6,7 @@
 from roundup import hyperdb
 from roundup.date import Date, Interval
 from roundup.cgi.actions import *
-from roundup.cgi.exceptions import Redirect, Unauthorised
+from roundup.cgi.exceptions import Redirect, Unauthorised, SeriousError
 
 class MockNull:
     def __init__(self, **kwargs):
@@ -38,14 +38,20 @@
         self.client.form = self.form
 
 class ShowActionTestCase(ActionTestCase):
-    def assertRaisesMessage(self, exception, callable, message, *args, **kwargs):
+    _nocheck = "don't compare exception values (the exception is enough)"
+    def assertRaisesMessage(self, exception, callable, message, *args,
+            **kwargs):
         try:
             callable(*args, **kwargs)
         except exception, msg:
+            if message is self._nocheck:
+                return
             self.assertEqual(str(msg), message)
         else:
-            if hasattr(excClass,'__name__'): excName = excClass.__name__
-            else: excName = str(excClass)
+            if hasattr(excClass,'__name__'):
+                excName = excClass.__name__
+            else:
+                excName = str(excClass)
             raise self.failureException, excName
 
     def testShowAction(self):
@@ -55,11 +61,18 @@
         self.assertRaises(ValueError, action.handle)
 
         self.form.value.append(MiniFieldStorage('@type', 'issue'))
-        self.assertRaisesMessage(Redirect, action.handle, 'BASE/issue')
+        self.assertRaisesMessage(SeriousError, action.handle, self._nocheck)
 
         self.form.value.append(MiniFieldStorage('@number', '1'))
         self.assertRaisesMessage(Redirect, action.handle, 'BASE/issue1')
 
+    def testShowActionNoType(self):
+        action = ShowAction(self.client)
+        self.assertRaises(ValueError, action.handle)
+        self.form.value.append(MiniFieldStorage('@number', '1'))
+        self.assertRaisesMessage(ValueError, action.handle,
+            'No type specified')
+
 class RetireActionTestCase(ActionTestCase):
     def testRetireAction(self):
         self.client.db.security.hasPermission = true