Skip to content
This repository was archived by the owner on Jun 16, 2026. It is now read-only.

test: add 13 read-only smoke scenarios across all services#10

Merged
TommyLike merged 3 commits into
mainfrom
feat/more-smoke-tests
Jun 6, 2026
Merged

test: add 13 read-only smoke scenarios across all services#10
TommyLike merged 3 commits into
mainfrom
feat/more-smoke-tests

Conversation

@TommyLike

Copy link
Copy Markdown
Collaborator

Summary

Add 13 read-only smoke test scenarios, expanding coverage from 9 to 22 scenarios (7 skipped).

New Scenarios

Service Scenario Status
gitcode repos-get
gitcode pulls-get
gitcode issues-comments
gitcode users-list-user
gitcode users-get
github users-get
github users-list-user
github pulls-list
github pulls-get
forum topics-list skipped (unstable)
forum topics-get skipped (unstable)
jenkins builds-get skipped (needs config)
jenkins queue-list skipped (needs config)

Smoke Result

19 passed, 0 failed, 7 skipped  |  Total: 18.56s

Coverage Matrix

Service Scenarios Active Skipped
gitcode 9 9 0
github 7 7 0
eur 3 3 0
jenkins 3 0 3
forum 3 0 3
etherpad 1 0 1
Total 26 19 7

Generated with Claude Code

GitCode (+5): repos-get, pulls-get, issues-comments, users-list-user, users-get
GitHub (+4):  users-get, users-list-user, pulls-list, pulls-get
Forum (+2):   topics-list, topics-get (skipped, forum unstable)
Jenkins (+2): builds-get, queue-list (skipped, needs configured instance)

Smoke result: 19 passed, 0 failed, 7 skipped

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@opensourceways-bot

Copy link
Copy Markdown

Welcome To opensourceways Community

Hey @TommyLike , thanks for your contribution to the community.

Bot Usage Manual

I'm the Bot here serving you. You can find the instructions on how to interact with me at Here . That means you can comment below every pull request or issue to trigger Bot Commands.

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

TommyLike, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

Linking Issue Notice

@TommyLike , the pull request must be linked to at least one issue.
If an issue has already been linked, but the needs-issue label remains, you can remove the label by commenting /check-issue .

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces several test scenario configuration files in YAML format for the forum, gitcode, github, and jenkins services. These scenarios define command-line arguments, execution timeouts, and various assertions (such as exit codes, stdout/stderr checks, and JSON key validations) to verify the behavior of different service endpoints. Some scenarios are configured to be skipped due to environment instability or setup requirements. There are no review comments, and I have no feedback to provide on these changes.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

TommyLike, thanks for your pull request. All authors of the commits have signed the CLA. 👍

The GitHub smoke scenarios were running without authentication because
SMOKE_GITHUB_TOKEN was not passed from secrets to the workflow env.
Public endpoints worked without auth, but GET /user (users-list-user)
required authentication and failed with exit code 1.

Also add SMOKE_GITHUB_OWNER and SMOKE_GITHUB_REPO for consistency.
@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

TommyLike, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@TommyLike TommyLike merged commit 4db071a into main Jun 6, 2026
7 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants