Skip to content

CI: Run the TypedPython app fixture and fail when its tests skip - #120

Merged
b-re-w merged 3 commits into
developfrom
feat/tp-app-ci
Oct 3, 2026
Merged

b-re-w merged 3 commits into
developfrom
feat/tp-app-ci

Conversation

@b-re-w

@b-re-w b-re-w commented Oct 3, 2026

Copy link
Copy Markdown
Member

Leader decision on #119's follow-up.

  • typedpython.yml: new app-fixture job (macos-latest). It creates a venv with pyrefly 1.3.2, runs :ksp-fixtures:app:desktopTest --rerun -PtypedpythonPython=<venv> after deleting old results, then fails if TypedPythonKotlinCallTest has no results or any skipped test. The workflow now also triggers on ksp-fixtures/app/**.
  • ksp-fixtures/app/build.gradle.kts: when the bridge is not built, desktopTest logs one line, TypedPython bridge tests will be SKIPPED: <reason>. A local skip stays allowed. The test JVM gets PYTHONDONTWRITEBYTECODE=1, so no __pycache__ is left in src/desktopTest/typedpython.
  • Includes Test: Always run the TypedPython leak test now that #98 is fixed #119's branch (the leak test no longer skips); merge Test: Always run the TypedPython leak test now that #98 is fixed #119 first.

Local verification (macOS arm64, JDK 21; the check step was run as written in the workflow):

@b-re-w
b-re-w merged commit 0ff7a07 into develop Oct 3, 2026
5 of 9 checks passed
@b-re-w
b-re-w deleted the feat/tp-app-ci branch October 3, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant