Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 13, 2025

The frontend now uses an action menu for plugin operations instead of direct toggle buttons. WDIO tests were failing because they attempted to click non-existent toggle elements.

Changes

  • Created e2e test file with action menu support for plugin activation testing
  • Updated plugin activation test to open action menu before status interaction
  • Changed status verification from button property check to mat-icon text content (toggle_off/toggle_on)
  • Updated wdio config to reference greate-belt-settings test path

Note: Page Objects (Login, Plugin, MyEforms, Navbar, etc.) are not included in this repository as they exist in the eform-angular-frontend project. The test files are copied to the frontend during the dev/test setup process where these Page Objects are already available.

Implementation

// Before: Direct interaction (no longer works)
const plugin = await pluginPage.getFirstPluginRowObj();
await plugin.enableOrDisablePlugin();

// After: Action menu interaction
const actionMenuBtn = await $('#action-items-0').$('#actionMenu');
await actionMenuBtn.click();
await browser.pause(500);

const statusBtn = await $('#plugin-status-button0');
await statusBtn.click();

Test flow: open menu → interact with status button → close menu with Escape.

Reference: microting/eform-angular-workflow-plugin#1167

<issue_title>Update WDIO tests for action menu-based plugin activation</issue_title>
><issue_description>Use microting/eform-angular-workflow-plugin#1167 as reference as to what needs to be done</issue_description>
>
> ## Comments on the Issue (you are @copilot in this section)
>
>
>
>

Original prompt

This section details on the original issue you should resolve

<issue_title>Update WDIO tests for action menu-based plugin activation</issue_title>
<issue_description>Use microting/eform-angular-workflow-plugin#1167 as reference as to what needs to be done</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits November 13, 2025 19:16
- Created e2e directory structure with Page objects and Tests
- Added common page objects (Login, Plugin, MyEforms, Navbar, Page, PageWithNavbar)
- Added LoginConstants for test authentication
- Created plugin activation test using action menu pattern
- Updated wdio config to reference greate-belt test file
- Test now opens action menu before checking/changing plugin status
- Test verifies plugin activation by checking mat-icon text (toggle_off/toggle_on)

Co-authored-by: renemadsen <[email protected]>
Copilot AI changed the title [WIP] Update WDIO tests for action menu-based plugin activation Update WDIO tests for action menu-based plugin activation Nov 13, 2025
Copilot AI requested a review from renemadsen November 13, 2025 19:23
Copilot finished work on behalf of renemadsen November 13, 2025 19:23
@renemadsen
Copy link
Member

@copilot don't create Page Objects, since they are coming from frontend project, only do e2e settings file and the wdio-headless-plugin-step2.conf.ts

Page Objects (Login, MyEforms, Plugin, etc.) should not be in the plugin repository.
They are copied from the frontend project during the test setup process.
Only plugin-specific test files and wdio config should remain.

Co-authored-by: renemadsen <[email protected]>
Copy link
Contributor Author

Copilot AI commented Nov 14, 2025

@copilot don't create Page Objects, since they are coming from frontend project, only do e2e settings file and the wdio-headless-plugin-step2.conf.ts

Done. Removed all Page Objects and Constants - they now come from the frontend project. Only the test file and wdio config remain. (f1d070a)

Copilot finished work on behalf of renemadsen November 14, 2025 05:01
@renemadsen renemadsen marked this pull request as ready for review November 14, 2025 05:13
@renemadsen renemadsen merged commit 53af42f into master Nov 14, 2025
7 of 10 checks passed
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.

Update WDIO tests for action menu-based plugin activation

2 participants