From e12f135b018832966bfbee6c3af727d6fa1ec7a4 Mon Sep 17 00:00:00 2001
From: Edwin Gonzales <edwin@newlogic.com>
Date: Fri, 14 Feb 2025 15:23:37 +0800
Subject: [PATCH] [FIX] spp_change_request_base: Fix issues with the tests.

---
 .../tests/test_change_requests.py                  | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/spp_change_request_base/tests/test_change_requests.py b/spp_change_request_base/tests/test_change_requests.py
index 5e3c9b579..c457b1414 100644
--- a/spp_change_request_base/tests/test_change_requests.py
+++ b/spp_change_request_base/tests/test_change_requests.py
@@ -10,7 +10,7 @@ def setUpClass(cls):
         super().setUpClass()
         cls._test_change_request = cls._create_change_request()
 
-    def test_01_create(self):
+    def test_01_assign_to_user(self):
         self.assertEqual(
             self._test_change_request.assign_to_id,
             self.env.user,
@@ -33,7 +33,7 @@ def test_03_unlink(self):
             "Draft change request should unlinkable by its creator!",
         )
 
-    def test_05_assign_to_user(self):
+    def test_04_assign_to_admin_user(self):
         admin = self.env.ref("base.user_admin")
         self._test_change_request.assign_to_user(admin)
         self.assertEqual(
@@ -45,7 +45,7 @@ def test_05_assign_to_user(self):
         with self.assertRaisesRegex(UserError, "^.*not have any validation sequence defined.$"):
             self._test_change_request.assign_to_user(self.env.user)
 
-    def test_07_open_request_detail(self):
+    def test_05_open_request_detail(self):
         res = self._test_change_request.open_request_detail()
         self.assertListEqual(
             [res.get("type"), res.get("tag"), res.get("params", {}).get("type")],
@@ -53,12 +53,12 @@ def test_07_open_request_detail(self):
             "Request Type ID does not exist, client should display error notification!",
         )
 
-    def test_08_cancel_error(self):
+    def test_06_cancel_error(self):
         with self.assertRaisesRegex(UserError, "^.*request to be cancelled must be in draft.*$"):
             self._test_change_request.state = "validated"
             self._test_change_request._cancel(self._test_change_request)
 
-    def test_09_cancel(self):
+    def test_07_cancel(self):
         self.assertListEqual(
             [
                 self._test_change_request.state,
@@ -83,12 +83,12 @@ def test_09_cancel(self):
             "Cancelled CR should have date cancelled info.!",
         )
 
-    def test_10_check_user_error(self):
+    def test_8_check_user_error(self):
         self._test_change_request.assign_to_id = None
         with self.assertRaisesRegex(UserError, "^.*no user assigned.*$"):
             self._test_change_request._check_user(process="Apply")
 
-    def test_11_check_user(self):
+    def test_9_check_user(self):
         with self.assertRaisesRegex(UserError, "^You are not allowed.*$"):
             self._test_change_request.with_user(2)._check_user(process="Apply")
         self.assertTrue(