Skip to content

Fix Streamlit app path in readiness test - #8

Merged
DOKOS-TAYOS merged 5 commits into
mainfrom
agent/fix-streamlit-app-path
Aug 7, 2026
Merged

DOKOS-TAYOS merged 5 commits into
mainfrom
agent/fix-streamlit-app-path

Conversation

@DOKOS-TAYOS

Copy link
Copy Markdown
Owner

What changed

  • Resolve the repository root from tests/test_repository_readiness.py using Path(__file__).
  • Pass the resulting absolute app.py path to AppTest.from_file.

Why

The readiness test passed "app.py" as a relative path. Streamlit resolved that path relative to the test file and attempted to load tests/app.py, causing the GitHub Actions jobs for the Dependabot PRs to fail with FileNotFoundError.

Scope

This changes only the path resolution in the existing Streamlit readiness test. No application behavior or dependencies are modified.

Validation

The branch is one commit ahead of main and modifies only tests/test_repository_readiness.py. GitHub Actions will run the existing Ruff formatting/lint checks and pytest matrix on this PR.

@DOKOS-TAYOS
DOKOS-TAYOS marked this pull request as ready for review August 7, 2026 03:32
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@cursor

cursor Bot commented Aug 7, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@DOKOS-TAYOS DOKOS-TAYOS closed this Aug 7, 2026
@DOKOS-TAYOS DOKOS-TAYOS reopened this Aug 7, 2026
@DOKOS-TAYOS
DOKOS-TAYOS merged commit 9ec9a33 into main Aug 7, 2026
3 checks passed
@DOKOS-TAYOS
DOKOS-TAYOS deleted the agent/fix-streamlit-app-path branch August 8, 2026 18:21
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