From 014bf73db82758e98ac40cf9ffd1bfc44681fad0 Mon Sep 17 00:00:00 2001 From: Earl Warren Date: Wed, 27 Aug 2025 08:32:47 +0200 Subject: [PATCH] chore: prefix all Actions related integration tests with TestActions (#9035) So they can conveniently be run all together with: `make 'test-sqlite#TestActions'` Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/9035 Reviewed-by: Gusted Co-authored-by: Earl Warren Co-committed-by: Earl Warren --- tests/integration/actions_commit_status_test.go | 2 +- tests/integration/actions_job_test.go | 4 ++-- tests/integration/actions_log_test.go | 2 +- tests/integration/actions_notifications_test.go | 2 +- tests/integration/actions_route_test.go | 2 +- tests/integration/actions_trigger_test.go | 12 ++++++------ tests/integration/actions_variables_test.go | 2 +- tests/integration/api_admin_actions_test.go | 2 +- tests/integration/api_org_actions_test.go | 2 +- tests/integration/api_repo_actions_test.go | 8 ++++---- tests/integration/api_user_actions_test.go | 2 +- tests/integration/cmd_forgejo_actions_test.go | 2 +- 12 files changed, 21 insertions(+), 21 deletions(-) diff --git a/tests/integration/actions_commit_status_test.go b/tests/integration/actions_commit_status_test.go index fdc12205fe..7c61904bc6 100644 --- a/tests/integration/actions_commit_status_test.go +++ b/tests/integration/actions_commit_status_test.go @@ -51,7 +51,7 @@ func TestActionsAutomerge(t *testing.T) { ) } -func TestForcePushCommitStatus(t *testing.T) { +func TestActionsForcePushCommitStatus(t *testing.T) { defer unittest.OverrideFixtures("tests/integration/fixtures/TestForcePushCommitStatus/")() defer tests.PrepareTestEnv(t)() diff --git a/tests/integration/actions_job_test.go b/tests/integration/actions_job_test.go index 412215086e..cd3c95b4a1 100644 --- a/tests/integration/actions_job_test.go +++ b/tests/integration/actions_job_test.go @@ -27,7 +27,7 @@ import ( "github.com/stretchr/testify/require" ) -func TestJobWithNeeds(t *testing.T) { +func TestActionsJobWithNeeds(t *testing.T) { if !setting.Database.Type.IsSQLite3() { t.Skip() } @@ -174,7 +174,7 @@ jobs: }) } -func TestJobNeedsMatrix(t *testing.T) { +func TestActionsJobNeedsMatrix(t *testing.T) { if !setting.Database.Type.IsSQLite3() { t.Skip() } diff --git a/tests/integration/actions_log_test.go b/tests/integration/actions_log_test.go index 2c2e2e88b3..0443dc9cfb 100644 --- a/tests/integration/actions_log_test.go +++ b/tests/integration/actions_log_test.go @@ -25,7 +25,7 @@ import ( "google.golang.org/protobuf/types/known/timestamppb" ) -func TestDownloadTaskLogs(t *testing.T) { +func TestActionsDownloadTaskLogs(t *testing.T) { if !setting.Database.Type.IsSQLite3() { t.Skip() } diff --git a/tests/integration/actions_notifications_test.go b/tests/integration/actions_notifications_test.go index e47cb64b83..ed59c92500 100644 --- a/tests/integration/actions_notifications_test.go +++ b/tests/integration/actions_notifications_test.go @@ -18,7 +18,7 @@ import ( "github.com/stretchr/testify/assert" ) -func TestActionNotifications(t *testing.T) { +func TestActionsNotifications(t *testing.T) { if !setting.Database.Type.IsSQLite3() { t.Skip() } diff --git a/tests/integration/actions_route_test.go b/tests/integration/actions_route_test.go index 2641fb6a04..521b379064 100644 --- a/tests/integration/actions_route_test.go +++ b/tests/integration/actions_route_test.go @@ -147,7 +147,7 @@ func TestActionsWebRouteLatestRun(t *testing.T) { }) } -func TestActionsArtifactDeletion(t *testing.T) { +func TestActionsWebRouteArtifactDeletion(t *testing.T) { onGiteaRun(t, func(t *testing.T, u *url.URL) { user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2}) diff --git a/tests/integration/actions_trigger_test.go b/tests/integration/actions_trigger_test.go index 0e61e05547..d13c666020 100644 --- a/tests/integration/actions_trigger_test.go +++ b/tests/integration/actions_trigger_test.go @@ -39,7 +39,7 @@ import ( "github.com/stretchr/testify/require" ) -func TestPullRequestCommitStatus(t *testing.T) { +func TestActionsPullRequestCommitStatus(t *testing.T) { onGiteaRun(t, func(t *testing.T, u *url.URL) { user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2}) // owner of the base repo session := loginUser(t, "user2") @@ -350,7 +350,7 @@ jobs: }) } -func TestPullRequestWithInvalidWorkflow(t *testing.T) { +func TestActionsPullRequestWithInvalidWorkflow(t *testing.T) { onGiteaRun(t, func(t *testing.T, u *url.URL) { user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2}) // owner of the base repo session := loginUser(t, "user2") @@ -431,7 +431,7 @@ runs-on: docker }) } -func TestPullRequestTargetEvent(t *testing.T) { +func TestActionsPullRequestTargetEvent(t *testing.T) { onGiteaRun(t, func(t *testing.T, u *url.URL) { user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2}) // owner of the base repo org3 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 3}) // owner of the forked repo @@ -588,7 +588,7 @@ func TestPullRequestTargetEvent(t *testing.T) { }) } -func TestSkipCI(t *testing.T) { +func TestActionsSkipCI(t *testing.T) { onGiteaRun(t, func(t *testing.T, u *url.URL) { session := loginUser(t, "user2") user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2}) @@ -679,7 +679,7 @@ func TestSkipCI(t *testing.T) { }) } -func TestCreateDeleteRefEvent(t *testing.T) { +func TestActionsCreateDeleteRefEvent(t *testing.T) { onGiteaRun(t, func(t *testing.T, u *url.URL) { user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2}) @@ -795,7 +795,7 @@ func TestCreateDeleteRefEvent(t *testing.T) { }) } -func TestWorkflowDispatchEvent(t *testing.T) { +func TestActionsWorkflowDispatchEvent(t *testing.T) { onGiteaRun(t, func(t *testing.T, u *url.URL) { user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2}) diff --git a/tests/integration/actions_variables_test.go b/tests/integration/actions_variables_test.go index 2442710ea1..782d167502 100644 --- a/tests/integration/actions_variables_test.go +++ b/tests/integration/actions_variables_test.go @@ -18,7 +18,7 @@ import ( "github.com/stretchr/testify/assert" ) -func TestActionVariablesModification(t *testing.T) { +func TestActionsVariablesModification(t *testing.T) { defer unittest.OverrideFixtures("tests/integration/fixtures/TestActionVariablesModification")() defer tests.PrepareTestEnv(t)() diff --git a/tests/integration/api_admin_actions_test.go b/tests/integration/api_admin_actions_test.go index 8e87d98ef0..a8a9281ff5 100644 --- a/tests/integration/api_admin_actions_test.go +++ b/tests/integration/api_admin_actions_test.go @@ -17,7 +17,7 @@ import ( "github.com/stretchr/testify/assert" ) -func TestAPISearchActionJobs_GlobalRunner(t *testing.T) { +func TestActionsAPISearchActionJobs_GlobalRunner(t *testing.T) { defer tests.PrepareTestEnv(t)() job := unittest.AssertExistsAndLoadBean(t, &actions_model.ActionRunJob{ID: 393}) diff --git a/tests/integration/api_org_actions_test.go b/tests/integration/api_org_actions_test.go index a5497130f9..e76ef2f9be 100644 --- a/tests/integration/api_org_actions_test.go +++ b/tests/integration/api_org_actions_test.go @@ -17,7 +17,7 @@ import ( "github.com/stretchr/testify/assert" ) -func TestAPISearchActionJobs_OrgRunner(t *testing.T) { +func TestActionsAPISearchActionJobs_OrgRunner(t *testing.T) { defer tests.PrepareTestEnv(t)() session := loginUser(t, "user1") diff --git a/tests/integration/api_repo_actions_test.go b/tests/integration/api_repo_actions_test.go index af2aa10cfd..348ff45ed9 100644 --- a/tests/integration/api_repo_actions_test.go +++ b/tests/integration/api_repo_actions_test.go @@ -23,7 +23,7 @@ import ( "github.com/stretchr/testify/assert" ) -func TestAPISearchActionJobs_RepoRunner(t *testing.T) { +func TestActionsAPISearchActionJobs_RepoRunner(t *testing.T) { defer tests.PrepareTestEnv(t)() repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1}) @@ -47,7 +47,7 @@ func TestAPISearchActionJobs_RepoRunner(t *testing.T) { assert.Equal(t, job.ID, jobs[0].ID) } -func TestAPIWorkflowDispatchReturnInfo(t *testing.T) { +func TestActionsAPIWorkflowDispatchReturnInfo(t *testing.T) { onGiteaRun(t, func(t *testing.T, u *url.URL) { workflowName := "dispatch.yml" user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2}) @@ -102,7 +102,7 @@ jobs: }) } -func TestAPIGetListActionRun(t *testing.T) { +func TestActionsAPIGetListActionRun(t *testing.T) { defer tests.PrepareTestEnv(t)() var ( runIDs = []int64{892, 893, 894} @@ -185,7 +185,7 @@ func TestAPIGetListActionRun(t *testing.T) { } } -func TestAPIGetActionRun(t *testing.T) { +func TestActionsAPIGetActionRun(t *testing.T) { defer tests.PrepareTestEnv(t)() repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 63}) diff --git a/tests/integration/api_user_actions_test.go b/tests/integration/api_user_actions_test.go index f8d5bb500e..7f65c543fa 100644 --- a/tests/integration/api_user_actions_test.go +++ b/tests/integration/api_user_actions_test.go @@ -17,7 +17,7 @@ import ( "github.com/stretchr/testify/assert" ) -func TestAPISearchActionJobs_UserRunner(t *testing.T) { +func TestActionsAPISearchActionJobs_UserRunner(t *testing.T) { defer tests.PrepareTestEnv(t)() normalUsername := "user2" diff --git a/tests/integration/cmd_forgejo_actions_test.go b/tests/integration/cmd_forgejo_actions_test.go index 01696d3171..9d7db42c2e 100644 --- a/tests/integration/cmd_forgejo_actions_test.go +++ b/tests/integration/cmd_forgejo_actions_test.go @@ -20,7 +20,7 @@ import ( "github.com/stretchr/testify/require" ) -func Test_CmdForgejo_Actions(t *testing.T) { +func TestActions_CmdForgejo_Actions(t *testing.T) { onGiteaRun(t, func(*testing.T, *url.URL) { token, err := runMainApp("forgejo-cli", "actions", "generate-runner-token") require.NoError(t, err)