Skip to content

ci: introduce integration test on GitHub Actions #355

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Aug 15, 2025
Merged

ci: introduce integration test on GitHub Actions #355

merged 1 commit into from
Aug 15, 2025
+48 −2

Conversation

Songmu
Copy link
Collaborator

@Songmu Songmu commented Aug 13, 2025 •
edited
Loading

This PR enables integration tests to run on GitHub Actions using the service account functionality (#359) and folder ID specification feature (#354).

Workflow Execution Strategy

To avoid duplicate executions, manage costs, and address security concerns, the integration tests will run via:

  • Manual trigger: workflow_dispatch for on-demand execution
  • Weekly schedule: Mondays, early morning 5:36 JST (Sunday 20:36 UTC)
  • Label trigger: When the "integration-test" label is added to a PR
    • This enables testing for PRs from forked repositories

Implementation Details

Authentication:

  • Uses Workload Identity Federation for secure, keyless authentication
  • No service account keys are stored in the repository
  • Configuration values are hardcoded as they are not sensitive:
    • workload_identity_provider
    • service_account
    • DECK_TEST_FOLDER_ID

Infrastructure Setup (Complete)

I've provisioned the following resources in my personal Google Workspace:

  • Test Shared Drive
    • Added your (@k1LoW) Gmail account as a member for content management
  • Google Cloud Project with:
    • Google Slides API and Drive API enabled
    • Service account
    • Workload Identity Pool configured
    • GitHub identity provider linked to the service account

Testing

The workflow has been tested and is ready for use. We can manually trigger it via the Actions tab or add the "integration-test" label to any PR to run the tests.

@Songmu Songmu force-pushed the ghaction-integration branch from 85f648b to 3f42ad0 Compare August 13, 2025 18:57
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 3f42ad0 to 3968801 Compare August 13, 2025 19:00
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 3968801 to 3a8930e Compare August 14, 2025 02:04
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 3a8930e to 38eaf88 Compare August 14, 2025 02:17
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 38eaf88 to 0d769af Compare August 14, 2025 02:21
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 0d769af to 980060d Compare August 14, 2025 02:27
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 980060d to 92fc0ac Compare August 14, 2025 12:15
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 92fc0ac to 1215818 Compare August 14, 2025 12:22
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 1215818 to 47aa628 Compare August 14, 2025 12:26
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 47aa628 to 890fd27 Compare August 14, 2025 14:45
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu mentioned this pull request Aug 15, 2025
Merged
@Songmu Songmu force-pushed the ghaction-integration branch from a657d81 to ce69977 Compare August 15, 2025 08:07
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu changed the title WIP: refine integration test to run on GitHub Actions chore: introduce integration test on GitHub Actions Aug 15, 2025
@Songmu Songmu force-pushed the ghaction-integration branch from ce69977 to adcc9c9 Compare August 15, 2025 08:30
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu changed the title chore: introduce integration test on GitHub Actions ci: introduce integration test on GitHub Actions Aug 15, 2025
@Songmu Songmu force-pushed the ghaction-integration branch from adcc9c9 to 88e473d Compare August 15, 2025 10:48
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 88e473d to 5ee3775 Compare August 15, 2025 11:01
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from 5ee3775 to cfbd143 Compare August 15, 2025 11:29
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Songmu Songmu force-pushed the ghaction-integration branch from cfbd143 to 0c1f643 Compare August 15, 2025 11:58
@github-actions GitHub Actions
Copy link
Contributor

Code Metrics Report

main (b9a2be8) #355 (456069f) +/-
Coverage 31.7% 31.7% 0.0%
Code to Test Ratio 1:0.8 1:0.8 +0.0
Test Execution Time 9s 11s +2s
Details 
  |                     | main (b9a2be8) | #355 (456069f) | +/-  |
  |---------------------|----------------|----------------|------|
  | Coverage            |          31.7% |          31.7% | 0.0% |
  |   Files             |             26 |             26 |    0 |
  |   Lines             |           3063 |           3063 |    0 |
  |   Covered           |            972 |            972 |    0 |
+ | Code to Test Ratio  |          1:0.8 |          1:0.8 | +0.0 |
  |   Code              |           6289 |           6289 |    0 |
+ |   Test              |           5476 |           5481 |   +5 |
- | Test Execution Time |             9s |            11s |  +2s |

Reported by octocov

@Songmu Songmu requested a review from k1LoW August 15, 2025 12:02
@Songmu Songmu marked this pull request as ready for review August 15, 2025 12:02
k1LoW
k1LoW approved these changes Aug 15, 2025
View reviewed changes
Copy link
Owner

@k1LoW k1LoW left a comment

Choose a reason for hiding this comment

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

It's amazing.

@Songmu Songmu merged commit c976511 into main Aug 15, 2025
1 check passed
@Songmu Songmu deleted the ghaction-integration branch August 15, 2025 15:27
@github-actions github-actions bot mentioned this pull request Aug 15, 2025
Merged
@BrewTestBot BrewTestBot mentioned this pull request Aug 16, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@k1LoW k1LoW k1LoW approved these changes

Assignees
No one assigned
Labels
integration-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Songmu @k1LoW