restricted-site-access icon indicating copy to clipboard operation
restricted-site-access copied to clipboard
verified publisher icon 10up

Add E2E tests for single site environment

Open Sidsector9 opened this issue 3 years ago • 3 comments

Is your enhancement related to a problem? Please describe.

The plugin doesn't work exactly the same between multisite and single site environments. For example $wp->request is defined in the multisite env much earlier compared to single site env.

At the moment, we run Cypress E2E tests only on a multisite env. We need to cover the single site env as well to make the plugin more robust.

A possible solution would be to set up a separate E2E GitHub action for single site.

Designs

No response

Describe alternatives you've considered

No response

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

Sidsector9 avatar Apr 18 '23 11:04 Sidsector9

@Sidsector9 I think it is a configuration setup. The pipeline in github action is something I haven't set up, and I need to check it from scratch, so I think it would be nice if you could work on it.

sudip-md avatar Jun 14 '24 06:06 sudip-md

Hi @jeffpaul @vikrampm1 I have tested the existing E2E test cases, and only one test case (add-invalid-addresses.test.js) failed. The failure is due to the absence of validation on the plugin side. I also tested this scenario manually and was able to add an invalid address without any restriction.

Image

cc : @Sidsector9 @iamdharmesh

sudip-md avatar Aug 13 '25 08:08 sudip-md

The failure is due to the absence of validation on the plugin side.

@sudip-md are you able to help update the tests to properly cover that?

jeffpaul avatar Sep 04 '25 19:09 jeffpaul