From b7936eed9c80ee67aad5d075e4087cec31b5760a Mon Sep 17 00:00:00 2001
From: Nils Coenen <74965194+NICO-SOLUTIONS@users.noreply.github.com>
Date: Mon, 27 Nov 2023 10:33:11 +0100
Subject: [PATCH] [IMP] moved setUp to setUpClass in TestIrHttp

To avoid creation and re-creation of records for each test run
---
 website_require_login/data/ir_actions.xml     |  1 -
 website_require_login/data/ir_ui_menu.xml     |  1 -
 website_require_login/tests/test_ir_http.py   | 23 ++++++++++---------
 .../views/website_auth_url.xml                |  1 -
 4 files changed, 12 insertions(+), 14 deletions(-)

diff --git a/website_require_login/data/ir_actions.xml b/website_require_login/data/ir_actions.xml
index 886a574c39..ac0413bfba 100644
--- a/website_require_login/data/ir_actions.xml
+++ b/website_require_login/data/ir_actions.xml
@@ -1,4 +1,3 @@
-<?xml version="1.0" encoding="utf-8" ?>
 <odoo>
 
     <record id="action_website_auth_url" model="ir.actions.act_window">
diff --git a/website_require_login/data/ir_ui_menu.xml b/website_require_login/data/ir_ui_menu.xml
index 87d7d56be0..0bc90698bb 100644
--- a/website_require_login/data/ir_ui_menu.xml
+++ b/website_require_login/data/ir_ui_menu.xml
@@ -1,4 +1,3 @@
-<?xml version="1.0" encoding="utf-8" ?>
 <odoo>
 
     <menuitem
diff --git a/website_require_login/tests/test_ir_http.py b/website_require_login/tests/test_ir_http.py
index ed172189b4..c6c2ea18f0 100644
--- a/website_require_login/tests/test_ir_http.py
+++ b/website_require_login/tests/test_ir_http.py
@@ -2,34 +2,35 @@
 
 
 class TestIrHttp(HttpCase):
-    def setUp(self):
-        super().setUp()
-        self.website = self.env["website"].sudo().get_current_website()
-        self.auth_url = self.env["website.auth.url"].create(
-            {"website_id": self.website.id, "path": "/contactus"}
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.website = cls.env["website"].sudo().get_current_website()
+        cls.auth_url = cls.env["website.auth.url"].create(
+            {"website_id": cls.website.id, "path": "/contactus"}
         )
-        self.user = self.env["res.users"].create(
+        cls.user = cls.env["res.users"].create(
             {"name": "Test User", "login": "test_user", "password": "12345"}
         )
-        self.path = "/contactus"
-        self.expected_path = "/web/login?redirect=%s" % self.path
+        cls.path = "/contactus"
+        cls.expected_path = "/web/login?redirect=%s" % cls.path
 
     def test_dispatch_unauthorized(self):
         # Test that a public user cannot access "/auth_path
         self.authenticate(None, None)
-        response = self.url_open(self.path, allow_redirects=False)
+        response = self.url_open(self.__class__.path, allow_redirects=False)
         self.assertEqual(
             response.status_code,
             302,
             "Expected the response status code to be 302 indicating a redirect",
         )
 
-        self.assertIn(self.expected_path, response.headers["Location"])
+        self.assertIn(self.__class__.expected_path, response.headers["Location"])
 
     def test_dispatch_authorized(self):
         # Test that an authorized user can access "/auth_path
         self.authenticate(user="test_user", password="12345")
-        response = self.url_open(self.path)
+        response = self.url_open(self.__class__.path)
         self.assertEqual(
             response.status_code,
             200,
diff --git a/website_require_login/views/website_auth_url.xml b/website_require_login/views/website_auth_url.xml
index 2e354a1621..09d48ce4e5 100644
--- a/website_require_login/views/website_auth_url.xml
+++ b/website_require_login/views/website_auth_url.xml
@@ -1,4 +1,3 @@
-<?xml version="1.0" encoding="utf-8" ?>
 <odoo>
 
     <record id="website_auth_url_tree_view" model="ir.ui.view">