Skip to content

PMM-14012: remove Portal code - #1057

Merged
fabio-silva merged 6 commits into
mainfrom
PMM-14012-remove-portal
Feb 26, 2026
Merged

PMM-14012: remove Portal code#1057
fabio-silva merged 6 commits into
mainfrom
PMM-14012-remove-portal

Conversation

@fabio-silva

Copy link
Copy Markdown
Contributor

Copilot AI 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.

Pull request overview

This PR removes all Portal-related code and tests from the PMM UI testing framework, as the Portal integration feature is being deprecated.

Changes:

  • Removed Portal integration tests and related test infrastructure
  • Removed Portal API helpers and page objects
  • Cleaned up configuration files and environment variables
  • Removed Portal-specific template sources and test scenarios

Reviewed changes

Copilot reviewed 49 out of 49 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tests/telemetry_test.js Removed telemetry test that verified data landing on Portal
tests/pages/perconaPlatformPage/* Removed Percona Platform page objects and version-specific implementations
tests/pages/organizationTicketsPage.js Removed page object for organization tickets functionality
tests/pages/organizationEntitlementsPage.js Removed page object for organization entitlements
tests/pages/environmentOverviewPage.js Removed environment overview page object
tests/pages/api/portalAPI.js Removed Portal API helper with methods for ServiceNow, Okta, and Portal operations
tests/pages/api/platformAPI.js Removed Platform API helper with telemetry methods
tests/ia/ruleTemplates_test.js Removed test scenario verifying Portal template behavior with telemetry disabled
tests/ia/pages/ruleTemplatesPage.js Removed 'saas' (Percona Platform) from template sources
tests/helper/linksHelper.js Removed Portal profile URLs
tests/dockerConfiguration/publicAddressVariable_test.js Removed unused portalAPI parameter from After hook
tests/configuration/verifyPMMSettingsPageFunctionality_test.js Removed test verifying public address auto-set on Portal connection
tests/configuration/pages/pmmSettingsPage.js Removed Portal-related page elements and references
tests/advisors/stt/databaseChecksTiers_test.js Removed comprehensive Portal user tiers test
playwright-tests/tests/portal/* Removed all Playwright Portal tests including setup, cleanup, and connection tests
playwright-tests/playwright.config.ts Removed Portal test project configuration
playwright-tests/pages/pmm-settings/percona-platform.page.ts Removed Percona Platform settings page
playwright-tests/pages/platformPages/* Removed all Platform-specific pages (tickets, entitlements, environment overview)
playwright-tests/pages/login*.page.ts Removed Portal login methods and page
playwright-tests/pages/common.page.ts Removed Portal-related UI elements
playwright-tests/package.json Removed Portal-specific npm script
playwright-tests/helpers/* Removed Portal user types, helper methods, and enums
playwright-tests/docs/setup-env-portal.md Removed Portal test setup documentation
playwright-tests/api/* Removed Portal and Okta API implementations
codeceptConfigHelper.js Removed Portal API and page references, simplified test chunking
docker-compose*.yml Removed Portal-related environment variables
env.list Removed PORTAL_BASE_URL variable
README.md Removed Portal-related test tags documentation
.github/workflows/* Removed Portal test workflows and simplified project selection

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread playwright-tests/helpers/test-helper.ts
Comment thread .github/workflows/pmm-ui-tests.yml
@fabio-silva
fabio-silva merged commit 180d971 into main Feb 26, 2026
23 of 29 checks passed
@fabio-silva
fabio-silva deleted the PMM-14012-remove-portal branch February 26, 2026 11:21
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.

4 participants