Skip to content

Commit 79a5e4f

Browse files
committed
test: extract TEST_PASSWORD constant in test_views.py
1 parent 124d3f1 commit 79a5e4f

1 file changed

Lines changed: 12 additions & 10 deletions

File tree

netbox_interface_name_rules/tests/test_views.py

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@
1313

1414
User = get_user_model()
1515

16+
TEST_PASSWORD = "testpass123" # noqa: S105 - test credential only
17+
1618

1719
class ViewTestBase(TestCase):
1820
"""Base class that creates a superuser and logs in."""
@@ -22,7 +24,7 @@ def setUpTestData(cls):
2224
"""Create superuser and basic InterfaceNameRule for view tests."""
2325
cls.superuser = User.objects.create_superuser(
2426
username="viewtestuser",
25-
password="testpass123",
27+
password=TEST_PASSWORD,
2628
email="viewtest@example.com",
2729
)
2830
manufacturer = Manufacturer.objects.create(name="ViewMfg", slug="viewmfg")
@@ -47,7 +49,7 @@ def setUpTestData(cls):
4749

4850
def setUp(self):
4951
"""Log in the superuser before each test."""
50-
self.client.login(username="viewtestuser", password="testpass123")
52+
self.client.login(username="viewtestuser", password=TEST_PASSWORD)
5153

5254

5355
class RuleListViewTest(ViewTestBase):
@@ -129,8 +131,8 @@ def test_toggle_ajax_returns_json(self):
129131
def test_toggle_ajax_no_permission_returns_403(self):
130132
"""Authenticated user without change permission gets 403 on AJAX toggle."""
131133
# Create a regular user without permissions
132-
User.objects.create_user(username="noperm_user", password="testpass123")
133-
self.client.login(username="noperm_user", password="testpass123")
134+
User.objects.create_user(username="noperm_user", password=TEST_PASSWORD)
135+
self.client.login(username="noperm_user", password=TEST_PASSWORD)
134136
url = self._toggle_url(self.rule.pk)
135137
response = self.client.post(url, HTTP_X_REQUESTED_WITH="XMLHttpRequest")
136138
self.assertEqual(response.status_code, 403)
@@ -342,7 +344,7 @@ class ViewTestBase2(TestCase):
342344
def setUpTestData(cls):
343345
cls.superuser = User.objects.create_superuser(
344346
username="covextrauser",
345-
password="testpass123",
347+
password=TEST_PASSWORD,
346348
email="covextra@example.com",
347349
)
348350
manufacturer = Manufacturer.objects.create(name="CovXMfg", slug="covxmfg")
@@ -359,7 +361,7 @@ def setUpTestData(cls):
359361
)
360362

361363
def setUp(self):
362-
self.client.login(username="covextrauser", password="testpass123")
364+
self.client.login(username="covextrauser", password=TEST_PASSWORD)
363365

364366

365367
class RuleTestViewGetWithRuleIdTest(ViewTestBase2):
@@ -426,8 +428,8 @@ def test_post_background_action_enqueues_job(self):
426428

427429
def test_post_no_change_permission_raises_forbidden(self):
428430
"""POST apply without dcim.change_interface permission raises PermissionDenied (line 382-383)."""
429-
User.objects.create_user(username="noperm_apply_user", password="testpass123")
430-
self.client.login(username="noperm_apply_user", password="testpass123")
431+
User.objects.create_user(username="noperm_apply_user", password=TEST_PASSWORD)
432+
self.client.login(username="noperm_apply_user", password=TEST_PASSWORD)
431433
url = self._url(self.rule.pk)
432434
response = self.client.post(url, {"action": "apply"})
433435
self.assertEqual(response.status_code, 403)
@@ -449,8 +451,8 @@ def test_toggle_non_ajax_no_permission_raises_403(self):
449451
raises PermissionDenied when the user lacks change_interfacenamerule, which
450452
Django converts to a 403 response.
451453
"""
452-
User.objects.create_user(username="noperm_tog_user2", password="testpass123")
453-
self.client.login(username="noperm_tog_user2", password="testpass123")
454+
User.objects.create_user(username="noperm_tog_user2", password=TEST_PASSWORD)
455+
self.client.login(username="noperm_tog_user2", password=TEST_PASSWORD)
454456
url = self._toggle_url(self.rule.pk)
455457
response = self.client.post(url)
456458
self.assertEqual(response.status_code, 403)

0 commit comments

Comments
 (0)