Skip to content

ci: fix testcase discovery path in test-uipath workflow#107

Merged
radu-mocanu merged 1 commit intomainfrom
fix/test-uipath-workflow-path
Apr 23, 2026
Merged

ci: fix testcase discovery path in test-uipath workflow#107
radu-mocanu merged 1 commit intomainfrom
fix/test-uipath-workflow-path

Conversation

@radu-mocanu
Copy link
Copy Markdown
Collaborator

@radu-mocanu radu-mocanu commented Apr 23, 2026

Summary

  • point testcase discovery and per-testcase working directory at `uipath-python/packages/uipath/testcases/` instead of the non-existent repo-root `testcases/`

Why

the integration test job was failing before running anything because `find testcases` at the `uipath-python` repo root returned nothing, the matrix fanned out an empty-string testcase entry, and the container tried to chdir into a path that does not exist. the testcases live under `packages/uipath/testcases/` after the mono-repo refactor

@radu-mocanu radu-mocanu requested a review from a team as a code owner April 23, 2026 11:05
Copilot AI review requested due to automatic review settings April 23, 2026 11:05
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the GitHub Actions integration-test discovery and execution paths in the test-uipath workflow to point at the actual UiPath Python testcase location under uipath-python/packages/uipath/testcases/, preventing failures due to missing directories.

Changes:

  • Update testcase discovery step to run from uipath-python/packages/uipath so find testcases resolves correctly.
  • Update per-testcase execution working directory to uipath-python/packages/uipath/testcases/${{ matrix.testcase }}.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/test-uipath.yml
@radu-mocanu radu-mocanu merged commit b465907 into main Apr 23, 2026
58 of 60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants