Skip to content

Update wso2ipw for ICP smoke test - #1718

Open
manuranga wants to merge 2 commits into
5.0.xfrom
fix/wso2ipw-icp-navigation
Open

Update wso2ipw for ICP smoke test#1718
manuranga wants to merge 2 commits into
5.0.xfrom
fix/wso2ipw-icp-navigation

Conversation

@manuranga

@manuranga manuranga commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Updates the smoke-test workflow to use the fixed wso2ipw tag for ICP navigation.\n\nThe failed 5.0.x run timed out on a stale guest selector:\n\n getByRole('button', { name: /HelloWorld/ })\n\nIn the current 5.0.x UI the resource breadcrumb uses Artifacts > HTTP Service > resource, so no guest button named HelloWorld exists. wso2ipw v0.1.10 navigates back via the host Open Overview action, which was verified locally in a Lima amd64 Ubuntu VM to get past the original timeout and reach the ICP monitoring checkbox.\n\nNote: npm publishing of v0.1.10 is currently blocked by npm token permissions in manuranga/wso2ipw, so this PR pins the immutable GitHub tag tarball for validation.

Summary by CodeRabbit

  • Chores
    • Updated Linux and Windows smoke-test workflows to install the wso2ipw tool from the GitHub release tarball for tag v0.1.10 (instead of the previously pinned npm version).

@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8f15fe45-70bb-4d69-a5cc-11c8da5b9b31

📥 Commits

Reviewing files that changed from the base of the PR and between 1384b67 and b1084df.

📒 Files selected for processing (1)
  • .github/workflows/smoke-test.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/smoke-test.yml

Walkthrough

The Install wso2ipw step in the smoke-test workflow is updated for both the Linux and Windows jobs: instead of installing the npm package wso2ipw@0.1.8, each job now installs wso2ipw from the GitHub release tarball for tag v0.1.10.

Changes

wso2ipw Version Bump in Smoke-Test Workflow

Layer / File(s) Summary
wso2ipw install step update (Linux + Windows)
.github/workflows/smoke-test.yml
Both the Linux (line 128) and Windows (line 227) jobs replace npm install -g wso2ipw@0.1.8 with the GitHub release tarball URL for v0.1.10.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description lacks most required template sections such as Purpose, Goals, Approach, User stories, Release notes, Documentation, Training, Certification, Marketing, Automation tests, Security checks, and others. Fill out the required template sections including Purpose, Goals, Approach, Release note, Documentation, Security checks, and Test environment to meet repository standards.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: updating wso2ipw version in the ICP smoke test workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/wso2ipw-icp-navigation

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

2 participants