New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add cypress support for the integration use cases #44
Comments
I can start this task by adding a skeleton setup that allows adding initial coverage to the integration. It would be out of the scope to "automate" it, this first step would help to run tests in a development scenario to avoid manual testing. |
I've started an investigation on this task, and it seems that cypress has known limitations with cross-origin iframes https://docs.cypress.io/guides/guides/web-security#Cross-origin-iframes I did some experiments based on those workarounds and comments from cypress users but don't work well on my tests (too many flaky tests). So, this task may be put on hold until it can be dedicated more time and the cypress community show some progress. I hit what most of the users reported in that thread. |
I crated this draft kiali/kiali#6892 |
The plugin repo should have a cypress suite focused on the integration scenarios.
This effort is different from the regular functional test effort that happens in the main Kiali repo, it shouldn't overlap/duplicate areas of testing.
The scope should be the links between pages and some specific special rendering that happens in the plugin context.
To make it easier, we could follow what other plugins are doing like the KubeVirt one:
https://github.com/kubevirt-ui/kubevirt-plugin/tree/main/cypress
Also, it's out of the scope of this work to include them in any automation system (yet), the ideal would be to have something similar to other plugins, a local script that can launch these tests pointing them to a local cluster with OpenShift + Kiali + Plugin deployed.
PRs that belong to this issue:
Pending Test issues to be implememented:
The text was updated successfully, but these errors were encountered: