From e66461822a444528e7dd630964ce57eb6106607d Mon Sep 17 00:00:00 2001 From: yxzhaao Date: Wed, 22 Jul 2026 18:22:39 +0800 Subject: [PATCH 1/2] test(e2e): declare request identities explicitly --- tests/cli_e2e/docs/docs_create_fetch_test.go | 1 + tests/cli_e2e/okr/okr_shortcuts_test.go | 12 ++++++++++++ 2 files changed, 13 insertions(+) diff --git a/tests/cli_e2e/docs/docs_create_fetch_test.go b/tests/cli_e2e/docs/docs_create_fetch_test.go index 2e1555b72d..e292615abf 100644 --- a/tests/cli_e2e/docs/docs_create_fetch_test.go +++ b/tests/cli_e2e/docs/docs_create_fetch_test.go @@ -44,6 +44,7 @@ func TestDocs_CreateAndFetchWorkflowAsBot(t *testing.T) { "--doc", docToken, "--doc-format", "markdown", }, + DefaultAs: defaultAs, }) require.NoError(t, err) result.AssertExitCode(t, 0) diff --git a/tests/cli_e2e/okr/okr_shortcuts_test.go b/tests/cli_e2e/okr/okr_shortcuts_test.go index a93b4e1349..271e11ea7c 100644 --- a/tests/cli_e2e/okr/okr_shortcuts_test.go +++ b/tests/cli_e2e/okr/okr_shortcuts_test.go @@ -366,6 +366,7 @@ func createTestObjectives(t *testing.T, ctx context.Context, cycleID string, suf "--cycle-id", cycleID, "--input", string(inputJSON), }, + DefaultAs: "user", }) require.NoError(t, err, "failed to create test objectives") result.AssertExitCode(t, 0) @@ -411,6 +412,7 @@ func cleanupLiveTest(t *testing.T, created []liveTestCreated) { "--key-result-id", krID, "--yes", }, + DefaultAs: "user", }) clie2e.ReportCleanupFailure(t, fmt.Sprintf("delete KR %s", krID), result, err) select { @@ -426,6 +428,7 @@ func cleanupLiveTest(t *testing.T, created []liveTestCreated) { "--objective-id", obj.ObjectiveID, "--yes", }, + DefaultAs: "user", }) clie2e.ReportCleanupFailure(t, fmt.Sprintf("delete objective %s", obj.ObjectiveID), result, err) if i > 0 { @@ -447,6 +450,7 @@ func createLiveObjective(t *testing.T, ctx context.Context, cycleID string, suff "--cycle-id", cycleID, "--content", fmt.Sprintf(`{"text":"E2E Single Objective %s","mention":["ou_test"]}`, suffix), }, + DefaultAs: "user", }) require.NoError(t, err, "failed to create live objective") result.AssertExitCode(t, 0) @@ -466,6 +470,7 @@ func createLiveKeyResult(t *testing.T, ctx context.Context, objectiveID string, "--objective-id", objectiveID, "--content", fmt.Sprintf(`{"text":"E2E Single KR %s","mention":["ou_test"]}`, suffix), }, + DefaultAs: "user", }) require.NoError(t, err, "failed to create live key result") result.AssertExitCode(t, 0) @@ -499,6 +504,7 @@ func TestOKR_BatchCreateLive(t *testing.T) { "okr", "+cycle-detail", "--cycle-id", cycleID, }, + DefaultAs: "user", }) require.NoError(t, err) result.AssertExitCode(t, 0) @@ -544,6 +550,7 @@ func TestOKR_CreateLive_Objective(t *testing.T) { "okr", "+cycle-detail", "--cycle-id", cycleID, }, + DefaultAs: "user", }) require.NoError(t, err) result.AssertExitCode(t, 0) @@ -581,6 +588,7 @@ func TestOKR_CreateLive_KeyResultUnderExistingObjective(t *testing.T) { "okr", "+cycle-detail", "--cycle-id", cycleID, }, + DefaultAs: "user", }) require.NoError(t, err) result.AssertExitCode(t, 0) @@ -640,6 +648,7 @@ func TestOKR_ReorderLive(t *testing.T) { "--level", "objective", "--ops", string(opsJSON), }, + DefaultAs: "user", }) require.NoError(t, err) result.AssertExitCode(t, 0) @@ -650,6 +659,7 @@ func TestOKR_ReorderLive(t *testing.T) { "okr", "+cycle-detail", "--cycle-id", cycleID, }, + DefaultAs: "user", }) require.NoError(t, err) result.AssertExitCode(t, 0) @@ -702,6 +712,7 @@ func TestOKR_WeightLive(t *testing.T) { "--level", "objective", "--weights", string(weightsJSON), }, + DefaultAs: "user", }) require.NoError(t, err) result.AssertExitCode(t, 0) @@ -712,6 +723,7 @@ func TestOKR_WeightLive(t *testing.T) { "okr", "+cycle-detail", "--cycle-id", cycleID, }, + DefaultAs: "user", }) require.NoError(t, err) result.AssertExitCode(t, 0) From 95d3a6bc17a3ec22eaa422c067925e02105b18d2 Mon Sep 17 00:00:00 2001 From: yxzhaao Date: Wed, 22 Jul 2026 19:09:42 +0800 Subject: [PATCH 2/2] test(e2e): skip base workflow without bot credentials --- tests/cli_e2e/base/base_record_batch_update_workflow_test.go | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/cli_e2e/base/base_record_batch_update_workflow_test.go b/tests/cli_e2e/base/base_record_batch_update_workflow_test.go index 9b375925ff..5dbcbfc697 100644 --- a/tests/cli_e2e/base/base_record_batch_update_workflow_test.go +++ b/tests/cli_e2e/base/base_record_batch_update_workflow_test.go @@ -15,6 +15,7 @@ import ( ) func TestBaseRecordBatchUpdatePerRecordWorkflow(t *testing.T) { + clie2e.SkipWithoutTenantAccessToken(t) parentT := t ctx, cancel := context.WithTimeout(context.Background(), 4*time.Minute) t.Cleanup(cancel)