Skip to content
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 design note of flow testing extensions & GUI testing #70

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KazuhiroItoh
Copy link
Contributor

This PR adds plugin extensions and GUI testing function to the flow testing design notes.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Feb 10, 2022

CLA Signed

The committers are authorized under a signed CLA.

@knolleary
Copy link
Member

Is this a proposal to add UI testing to the existing flow tester plugin that is still being developed, or to create another plugin that extends the flow tester?

I am concerned that adding UI testing to the Flow Tester plugin would make it much larger and more complicated. For most users that want to test flows, this extra size and complexity would not be needed.

I believe we still have a lot of work on the Flow Tester to do - (which I acknowledge it held up by me at the moment) so I would also be concerned about trying to add more complexity before the core is done.

If this was proposed to be a second plugin that can be installed alongside the core Flow Tester to add-in UI testing capabilities, I think that would be better.

@KazuhiroItoh
Copy link
Contributor Author

Sorry for the late reply.

If this was proposed to be a second plugin that can be installed alongside the core Flow Tester to add-in UI testing capabilities, I think that would be better.

Yes, it is a proposal as a plugin that can add functions to the Flow Tester plugin.
Users can use the UI testing feature by installing the UI testing add-on for the Flow Tester as needed.

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.

2 participants