Skip to content

Conversation

corylanou
Copy link
Collaborator

Summary

  • Adds a new manual workflow (workflow_dispatch) for running cloud integration tests on-demand
  • Allows maintainers to test specific PRs or branches without merging to main
  • Includes only the integration tests that are restricted to main branch in commit.yml

Test Options

The workflow provides options to run:

  • ✅ S3 integration tests (AWS) - defaults to enabled
  • ⬜ Google Cloud Storage integration tests
  • ⬜ Azure Blob Storage integration tests

Tests that already run on every push/PR (S3 Mock, SFTP, unit tests, linting) are excluded since they're covered by the existing commit.yml workflow.

Access Control

By default, only users with write access to the repository can trigger workflow_dispatch workflows. This includes:

  • Repository owners
  • Collaborators with write or admin permissions
  • Organization members with appropriate permissions

No additional configuration is needed to restrict this to maintainers only.

How to Use

  1. Go to Actions tab
  2. Select "Manual Integration Tests" workflow
  3. Click "Run workflow"
  4. Configure:
    • PR number OR branch name (optional)
    • Which cloud integration tests to run
  5. Click "Run workflow"

Test plan

Once merged:

  • Test triggering the workflow for a specific PR
  • Test triggering the workflow for a specific branch
  • Verify S3 integration tests run successfully
  • Verify access control (only maintainers can trigger)

🤖 Generated with Claude Code

corylanou and others added 2 commits August 12, 2025 14:15
This workflow allows maintainers to manually trigger integration tests
that normally only run on the main branch. It includes options for:
- S3 integration tests (AWS)
- Google Cloud Storage integration tests
- Azure Blob Storage integration tests

Tests that already run on every push/PR (S3 Mock, SFTP, unit tests,
linting) are excluded since they're covered by the commit.yml workflow.

The workflow provides flexibility to test cloud storage backends
without merging to main, useful for debugging integration issues
or testing PRs that affect specific storage implementations.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Remove trailing whitespace
- Add newline at end of file

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Add workflow permissions for issues:write and pull-requests:write
  to enable PR commenting
- Create test result directories before uploading artifacts to ensure
  they always exist
- Add try-catch error handling for PR commenting to gracefully handle
  non-existent PRs
- Add continue-on-error to PR comment step to prevent workflow failure

These fixes match the improvements made to other workflows and ensure
the manual integration test workflow is robust and won't fail due to
missing permissions or non-existent artifacts.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@corylanou corylanou requested a review from benbjohnson August 12, 2025 21:09
@corylanou corylanou merged commit 387c59b into main Aug 13, 2025
23 of 24 checks passed
@corylanou corylanou deleted the add-manual-integration-tests branch August 13, 2025 16:09
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