Skip to content

[eSignet-UI Automation]: Remove ui-test folder and update push-trigger.yml for release-1.8.x - #1863

Merged
Prafulrakhade merged 2 commits into
mosip:release-1.8.xfrom
damodarguru:release-1.8.x
May 13, 2026
Merged

Prafulrakhade merged 2 commits into
mosip:release-1.8.xfrom
damodarguru:release-1.8.x

Conversation

@damodarguru

@damodarguru damodarguru commented May 13, 2026 •

Copy link
Copy Markdown
Contributor

[eSignet-UI Automation]: Remove ui-test folder and update push-trigger.yml for release-1.8.x

Summary by CodeRabbit

Release Notes

  • Chores
    • Removed the UI automation test framework and all associated test infrastructure, including test configurations, workflows, and feature files.

…sh-trigger.yml for release-1.8.x

Signed-off-by: damodarguru <damodar.g@cyberpwn.com>
…sh-trigger.yml for release-1.8.x

Signed-off-by: damodarguru <damodar.g@cyberpwn.com>
@coderabbitai

coderabbitai Bot commented May 13, 2026 •

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 484153d3-1963-4d33-ac2a-b98305ab55a5

📥 Commits

Reviewing files that changed from the base of the PR and between 9adc485 and 3b63e22.

📒 Files selected for processing (34)
  • .github/workflows/push-trigger.yml
  • ui-test/.gitignore
  • ui-test/Dockerfile
  • ui-test/README.md
  • ui-test/browserstack.yml
  • ui-test/dependency-reduced-pom.xml
  • ui-test/entrypoint.sh
  • ui-test/pom.xml
  • ui-test/src/main/java/base/BasePage.java
  • ui-test/src/main/java/base/BaseTest.java
  • ui-test/src/main/java/constants/ESignetConstants.java
  • ui-test/src/main/java/constants/UiConstants.java
  • ui-test/src/main/java/pages/LoginOptionsPage.java
  • ui-test/src/main/java/pages/MultiLanguagePage.java
  • ui-test/src/main/java/pages/SignUpPage.java
  • ui-test/src/main/java/runners/Runner.java
  • ui-test/src/main/java/stepdefinitions/LoginOptionsStepDefinition.java
  • ui-test/src/main/java/stepdefinitions/LoginWithInjiStepDefinition.java
  • ui-test/src/main/java/stepdefinitions/MultiLanguageStepDef.java
  • ui-test/src/main/java/stepdefinitions/SignUpStepDef.java
  • ui-test/src/main/java/utils/BaseTestUtil.java
  • ui-test/src/main/java/utils/EsignetConfigManager.java
  • ui-test/src/main/java/utils/ExtentReportManager.java
  • ui-test/src/main/java/utils/LanguageUtil.java
  • ui-test/src/main/java/utils/ScreenshotUtil.java
  • ui-test/src/main/java/utils/WaitUtil.java
  • ui-test/src/main/resources/config.properties
  • ui-test/src/main/resources/extent.properties
  • ui-test/src/main/resources/featurefiles/LoginOptions.feature
  • ui-test/src/main/resources/featurefiles/LoginWithInji.feature
  • ui-test/src/main/resources/featurefiles/MultiLanguage.feature
  • ui-test/src/main/resources/featurefiles/Sample.feature
  • ui-test/src/main/resources/featurefiles/SignUp.feature
  • ui-test/testNgXmlFiles/TestNg.xml
💤 Files with no reviewable changes (33)
  • ui-test/Dockerfile
  • ui-test/src/main/resources/featurefiles/MultiLanguage.feature
  • ui-test/src/main/resources/featurefiles/SignUp.feature
  • ui-test/src/main/resources/featurefiles/LoginOptions.feature
  • ui-test/testNgXmlFiles/TestNg.xml
  • ui-test/src/main/java/stepdefinitions/MultiLanguageStepDef.java
  • ui-test/.gitignore
  • ui-test/src/main/resources/featurefiles/LoginWithInji.feature
  • ui-test/src/main/java/constants/UiConstants.java
  • ui-test/README.md
  • ui-test/src/main/java/stepdefinitions/LoginWithInjiStepDefinition.java
  • ui-test/browserstack.yml
  • ui-test/src/main/java/base/BasePage.java
  • ui-test/src/main/java/utils/ScreenshotUtil.java
  • ui-test/src/main/resources/featurefiles/Sample.feature
  • ui-test/src/main/java/pages/SignUpPage.java
  • ui-test/src/main/java/utils/WaitUtil.java
  • ui-test/src/main/java/utils/ExtentReportManager.java
  • ui-test/src/main/resources/extent.properties
  • ui-test/src/main/java/stepdefinitions/LoginOptionsStepDefinition.java
  • ui-test/entrypoint.sh
  • ui-test/src/main/resources/config.properties
  • ui-test/src/main/java/constants/ESignetConstants.java
  • ui-test/pom.xml
  • ui-test/src/main/java/pages/MultiLanguagePage.java
  • ui-test/src/main/java/utils/BaseTestUtil.java
  • ui-test/src/main/java/base/BaseTest.java
  • ui-test/src/main/java/runners/Runner.java
  • ui-test/dependency-reduced-pom.xml
  • ui-test/src/main/java/utils/EsignetConfigManager.java
  • ui-test/src/main/java/utils/LanguageUtil.java
  • ui-test/src/main/java/pages/LoginOptionsPage.java
  • ui-test/src/main/java/stepdefinitions/SignUpStepDef.java

Walkthrough

The PR removes the entire ui-test automation framework module and its corresponding CI/CD pipeline integration from the eSignet project. This includes test source code, configuration, build artifacts, deployment workflows, and all supporting Cucumber feature files and TestNG suite definitions.

Changes

UI Test Module Removal

Layer / File(s) Summary
CI/CD workflow removal
.github/workflows/push-trigger.yml
UI test job definitions (build-maven-uitest-esignet, build-uitest-esignet-local, build-dockers-uitest-esignet) are removed from the push-trigger workflow, eliminating the Maven build, artifact upload, and Docker image build stages for UI tests.
Build and configuration removal
ui-test/pom.xml, ui-test/dependency-reduced-pom.xml, ui-test/Dockerfile, ui-test/entrypoint.sh, ui-test/.gitignore
Maven POM files, Docker build instructions, shell entry point, and git ignore patterns for the UI test module are deleted.
Test framework base classes and utilities
ui-test/src/main/java/base/*, ui-test/src/main/java/utils/*
Base Selenium/Cucumber test infrastructure removed: BaseTest (Selenium WebDriver management, Extent reporting, Cucumber hooks), BasePage (Page Object base), and utility classes (BaseTestUtil, EsignetConfigManager, ExtentReportManager, LanguageUtil, ScreenshotUtil, WaitUtil) that provided BrowserStack integration, config loading, reporting, and wait handling.
Page objects and step definitions
ui-test/src/main/java/pages/*, ui-test/src/main/java/stepdefinitions/*, ui-test/src/main/java/constants/*
Page Object Model classes (LoginOptionsPage, MultiLanguagePage, SignUpPage), Cucumber step definitions (LoginOptionsStepDefinition, LoginWithInjiStepDefinition, MultiLanguageStepDef, SignUpStepDef), and constants containers (ESignetConstants, UiConstants) are deleted.
Test specifications and configuration
ui-test/src/main/resources/config.properties, ui-test/src/main/resources/extent.properties, ui-test/src/main/resources/featurefiles/*, ui-test/testNgXmlFiles/TestNg.xml, ui-test/README.md, ui-test/browserstack.yml
Cucumber feature files (LoginOptions.feature, LoginWithInji.feature, MultiLanguage.feature, SignUp.feature, Sample.feature), TestNG suite XML, property configuration files, BrowserStack credentials config, and README documentation are removed.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Poem

🐰 A test suite bids adieu,
Files vanish, clean and true,
No more Selenium dreams,
UI tests burst at the seams—
Farewell to browser and brew! 🎭

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: removing the ui-test folder and updating the push-trigger.yml workflow file for the release-1.8.x branch.
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

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

3 participants