From 9b4c74f74a458bd468b36ba5e13627d7c1151359 Mon Sep 17 00:00:00 2001 From: Joe Li Date: Mon, 13 Apr 2026 15:05:44 -0700 Subject: [PATCH] fix(playwright): use superset shell instead of python -c in bashlib Replace inline `python -c` block with `superset shell` heredoc, which has the Flask app context pre-loaded. Drops the create_app()/app_context() boilerplate and addresses review feedback. Co-Authored-By: Claude Opus 4.6 (1M context) --- .github/workflows/bashlib.sh | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/bashlib.sh b/.github/workflows/bashlib.sh index 76f44d28f1b..dd4b152de37 100644 --- a/.github/workflows/bashlib.sh +++ b/.github/workflows/bashlib.sh @@ -129,6 +129,8 @@ playwright_testdata() { superset init # Enable DML on the examples database so Playwright tests can create/drop # temporary tables via SQL Lab without depending on external data sources. + # Uses `superset shell` (Flask shell with app context pre-loaded) rather + # than `python -c` to avoid the create_app()/app_context() boilerplate. superset shell <<'PYEOF' import sys from superset.extensions import db