Skip to content

test host in tests constants jes v3 - #420

Merged
1000TurquoisePogs merged 4 commits into
v3.x/masterfrom
users/zlatnikp/testHostv3
Sep 11, 2026
Merged

1000TurquoisePogs merged 4 commits into
v3.x/masterfrom
users/zlatnikp/testHostv3

Conversation

@pavelzlatnik

Copy link
Copy Markdown
Collaborator

This PR addresses Issue that test host url was part of utils.
Created mock which is part of host constants in tests to support test scenario.

PR Type

  • Bug fix
  • Feature
  • Other (Please indicate)

PR Checklist

  • PR completes npm run preCommit without error
  • Relevant Test cases have been added (Unit and or FVT)
  • Relevant update to CHANGELOG.md
  • PR from forked repo? Ensure Allow edits by maintaners is set.

…development code from utils

Co-Authored-by: Gemini Pro
Modified AI generated code: Yes
Dev Name: Pavel Zlatnik

Signed-off-by: Pavel Zlatnik <pavel.zlatnik@broadcom.com>
Signed-off-by: Pavel Zlatnik <pavel.zlatnik@broadcom.com>
Signed-off-by: Pavel Zlatnik <pavel.zlatnik@broadcom.com>
Signed-off-by: Pavel Zlatnik <pavel.zlatnik@broadcom.com>
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@1000TurquoisePogs 1000TurquoisePogs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

replacement for #415

@1000TurquoisePogs

Copy link
Copy Markdown
Member

Build and test succeed, but due to mainframe maintenance pax fails, so I must force-merge.

@1000TurquoisePogs
1000TurquoisePogs merged commit b11814b into v3.x/master Sep 11, 2026
9 of 11 checks passed
@1000TurquoisePogs
1000TurquoisePogs deleted the users/zlatnikp/testHostv3 branch September 11, 2026 18:02

@Martin-Zeithaml Martin-Zeithaml left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed after the merge, lgtm.

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.

3 participants