no need to pass headers when creating resources

Author: piyush-jaiswal

tests/test_category.py

@@ -75,12 +75,14 @@ def test_get_all_categories(self, create_category):
         assert "B" in names
 
     def test_update_category(self, create_authenticated_headers, create_category):
-        headers = create_authenticated_headers()
-        response = create_category("OldName", headers=headers)
+        response = create_category("OldName")
         data = response.get_json()
         cat_id = data["id"]
+
         update_resp = self.client.put(
-            f"/categories/{cat_id}", json={"name": "NewName"}, headers=headers
+            f"/categories/{cat_id}",
+            json={"name": "NewName"},
+            headers=create_authenticated_headers(),
         )
 
         assert update_resp.status_code == 200
@@ -112,11 +114,13 @@ def test_update_category_duplicate_name(
         self._verify_category_in_db("NewName")
 
     def test_delete_category(self, create_authenticated_headers, create_category):
-        headers = create_authenticated_headers()
-        response = create_category("ToDelete", headers=headers)
+        response = create_category("ToDelete")
         data = response.get_json()
         cat_id = data["id"]
-        delete_resp = self.client.delete(f"/categories/{cat_id}", headers=headers)
+
+        delete_resp = self.client.delete(
+            f"/categories/{cat_id}", headers=create_authenticated_headers()
+        )
 
         assert delete_resp.status_code == 204
         get_resp = self.client.get(f"/categories/{cat_id}")

tests/test_product.py

@@ -71,12 +71,14 @@ def test_get_all_products(self, create_product):
         assert "B" in names
 
     def test_update_product(self, create_authenticated_headers, create_product):
-        headers = create_authenticated_headers()
-        response = create_product("OldProduct", "OldDesc", headers=headers)
+        response = create_product("OldProduct", "OldDesc")
         data = response.get_json()
         p_id = data["id"]
+
         update_resp = self.client.put(
-            f"/product/{p_id}/update", json={"name": "NewProduct", "description": "NewDesc"}, headers=headers
+            f"/product/{p_id}/update",
+            json={"name": "NewProduct", "description": "NewDesc"},
+            headers=create_authenticated_headers(),
         )
 
         assert update_resp.status_code == 201
@@ -88,11 +90,13 @@ def test_update_product(self, create_authenticated_headers, create_product):
         self._verify_product_in_db("OldProduct", should_exist=False)
 
     def test_delete_product(self, create_authenticated_headers, create_product):
-        headers = create_authenticated_headers()
-        response = create_product("ToDelete", "desc", headers=headers)
+        response = create_product("ToDelete", "desc")
         data = response.get_json()
         p_id = data["id"]
-        delete_resp = self.client.delete(f"/product/{p_id}", headers=headers)
+
+        delete_resp = self.client.delete(
+            f"/product/{p_id}", headers=create_authenticated_headers()
+        )
 
         assert delete_resp.status_code == 200
         get_resp = self.client.get(f"/product/{p_id}")

tests/test_subcategory.py

@@ -75,12 +75,14 @@ def test_get_all_subcategories(self, create_subcategory):
         assert "B" in names
 
     def test_update_subcategory(self, create_authenticated_headers, create_subcategory):
-        headers = create_authenticated_headers()
-        response = create_subcategory("OldSubcat", headers=headers)
+        response = create_subcategory("OldSubcat")
         data = response.get_json()
         sc_id = data["id"]
+
         update_resp = self.client.put(
-            f"/subcategories/{sc_id}", json={"name": "NewSubcat"}, headers=headers
+            f"/subcategories/{sc_id}",
+            json={"name": "NewSubcat"},
+            headers=create_authenticated_headers(),
         )
 
         assert update_resp.status_code == 200
@@ -111,11 +113,13 @@ def test_update_subcategory_duplicate_name(
         self._verify_subcategory_in_db("NewSubcat")
 
     def test_delete_subcategory(self, create_authenticated_headers, create_subcategory):
-        headers = create_authenticated_headers()
-        response = create_subcategory("ToDelete", headers=headers)
+        response = create_subcategory("ToDelete")
         data = response.get_json()
         sc_id = data["id"]
-        delete_resp = self.client.delete(f"/subcategories/{sc_id}", headers=headers)
+
+        delete_resp = self.client.delete(
+            f"/subcategories/{sc_id}", headers=create_authenticated_headers()
+        )
 
         assert delete_resp.status_code == 204
         get_resp = self.client.get(f"/subcategories/{sc_id}")