Skip to content

Commit 484fdd9

Browse files
committed
fix: Resolve test 19_github_api ownership errors in CI
- Change github_commits to TEMP TABLE to prevent persistence - Add explicit DROP TABLE github_commits in cleanup - Add --clean flag to CI E2E tests to clear cached data directories Fixes ownership error "must be owner of table github_commits" that occurred due to persistent tables in cached ~/.pgrx data directories.
1 parent c64eba8 commit 484fdd9

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,11 +128,11 @@ jobs:
128128

129129
- name: Run E2E tests (shared_preload_libraries enforcement)
130130
id: e2e_no_preload
131-
run: ./scripts/test-e2e-local.sh --no-preload
131+
run: ./scripts/test-e2e-local.sh --clean --no-preload
132132

133133
- name: Run E2E tests
134134
id: e2e_tests
135-
run: ./scripts/test-e2e-local.sh --pg-version ${{ matrix.pg_version }}
135+
run: ./scripts/test-e2e-local.sh --clean --pg-version ${{ matrix.pg_version }}
136136

137137
- name: Upload PostgreSQL logs on E2E failure
138138
if: failure()

tests/e2e/sql/19_github_api.sql

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
-- ============================================================================
88

99
DROP TABLE IF EXISTS github_commits;
10-
CREATE TABLE github_commits (
10+
CREATE TEMP TABLE github_commits (
1111
id SERIAL PRIMARY KEY,
1212
sha TEXT UNIQUE,
1313
author TEXT,
@@ -102,6 +102,7 @@ SELECT sha, author, committed_at, LEFT(message, 50) AS message FROM github_commi
102102

103103
-- Cleanup
104104
DROP TABLE _test_github;
105+
DROP TABLE github_commits;
105106
SELECT df.clearvars();
106107

107108
SELECT 'TEST PASSED' AS result;

0 commit comments

Comments
 (0)