From f6dd8ffc66419a00f747f23417ae6a51050f0f9c Mon Sep 17 00:00:00 2001 From: hoang Date: Mon, 28 Sep 2026 17:21:51 +0700 Subject: [PATCH 1/2] Fix bug that Promgen test sometimes fails We have resolved an issue causing Promgen tests to fail randomly. The root cause was the use of .first() to select a token, while the "authtoken" table lacked explicit ordering and there is also another test case which creates new token. To address this, we modified the test script to consistently use the token with a key formatted as the username plus the suffix "_token". --- promgen/tests/test_rest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/promgen/tests/test_rest.py b/promgen/tests/test_rest.py index cf39b2e9..d307dfab 100644 --- a/promgen/tests/test_rest.py +++ b/promgen/tests/test_rest.py @@ -35,7 +35,7 @@ def _parse_case_json(value, default): if user: user = User.objects.get(username=case["user"]) - token = models.AuthToken.objects.filter(user=user).first().token_key + token = user.username + "_token" for permission in permissions: perm = permission["codename"] app_label, model_name = permission["model"].split(".") From 0475ec343393a0043fb728de5dfa6c785d50cf41 Mon Sep 17 00:00:00 2001 From: hoang Date: Mon, 28 Sep 2026 17:25:29 +0700 Subject: [PATCH 2/2] Fix test case name We have revised the name of a test case to accurately reflect its content. --- promgen/tests/cases/test_rest_user.csv | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/promgen/tests/cases/test_rest_user.csv b/promgen/tests/cases/test_rest_user.csv index 4c94eb64..195114d5 100644 --- a/promgen/tests/cases/test_rest_user.csv +++ b/promgen/tests/cases/test_rest_user.csv @@ -16,4 +16,4 @@ An authenticated user can register new notifier.,demo,,POST,api-v2:user-register An authenticated user cannot register new notifier with type promgen.notification.user.,demo,,POST,api-v2:user-register-user-notifier,,"{""filters"": [{""name"": ""test-name"", ""value"": ""test-value""}], ""sender"": ""promgen.notification.user"", ""value"": ""https://test.slack.com"", ""enabled"": false}",400,"{""response_body"": {""detail"": ""Cannot register a promgen.notification.user notifier for a user.""}}" An authenticated user can list their API tokens.,demo,,GET,api-v2:user-current-user-tokens,,,200,"{""example"": ""rest.user.detail.tokens.json""}" An authenticated user can create API token.,demo,,POST,api-v2:user-current-user-tokens,,"{""name"": ""test-token"", ""days_to_live"": 1}",201, -An unauthenticated user can delete their API token.,demo,,DELETE,api-v2:user-current-user-token,"{""digest"": ""2ca4bea592c85b3996047d1121dfd428670ece2805050fd9906a23c0942fb745aebcd59f751727f0f4dc24e3efa9b71219b6a8b8866d3ffb3d9f3f8a329dcb47""}",,204, +An authenticated user can delete their API token.,demo,,DELETE,api-v2:user-current-user-token,"{""digest"": ""2ca4bea592c85b3996047d1121dfd428670ece2805050fd9906a23c0942fb745aebcd59f751727f0f4dc24e3efa9b71219b6a8b8866d3ffb3d9f3f8a329dcb47""}",,204,