maui icon indicating copy to clipboard operation
maui copied to clipboard

[Testing] Rework Appium Runner so that it goes directly to tests instead of navigates

Open jsuarezruiz opened this issue 1 year ago • 9 comments

Description of Change

Rework Appium Runner so that it goes directly to tests instead of navigates.

image

Issues Fixed

Fixes #20614

jsuarezruiz avatar Feb 19 '24 12:02 jsuarezruiz

/azp run

jsuarezruiz avatar Mar 13 '24 07:03 jsuarezruiz

Azure Pipelines successfully started running 3 pipeline(s).

azure-pipelines[bot] avatar Mar 13 '24 07:03 azure-pipelines[bot]

/rebase

PureWeen avatar Mar 15 '24 18:03 PureWeen

By changing MainPage and resetting after each test, golden tests require updating the reference snapshots. In many cases, the difference is the NavigationBar, back button etc.

jsuarezruiz avatar Mar 19 '24 15:03 jsuarezruiz

/azp run

jsuarezruiz avatar Mar 20 '24 12:03 jsuarezruiz

Azure Pipelines successfully started running 3 pipeline(s).

azure-pipelines[bot] avatar Mar 20 '24 12:03 azure-pipelines[bot]

I don't think we need to navigate to the second page to "set the main page"

Can we just add a feature to this page? If someone types into that box and clicks "Go to tests cases" it'll check the text in the entry box and try to match it to an existing test? and then it'll swap the main page?

image

Applied changes to write the bug description and navigate directly to it from the MainPage. This avoid:

  • Tap the Button to navigate to the issues page
  • Navigate to the issues list page.

Is not to much, but the change reduce some milliseconds in every test.

jsuarezruiz avatar Mar 26 '24 11:03 jsuarezruiz

Maybe instead of the checkbox we can just add an entry? Just trying to reduce the number of things we have to interact with at startup?

I wish we could more easily "reset the app" without having to call "Resetapp" hmmm

It looks like there's a few tests failing as well in CI

We can remove it but, then, we will always reset, right?. With the CheckBox we can mark and decide if we want to reset or not.

jsuarezruiz avatar Apr 04 '24 14:04 jsuarezruiz

Maybe instead of the checkbox we can just add an entry? Just trying to reduce the number of things we have to interact with at startup? I wish we could more easily "reset the app" without having to call "Resetapp" hmmm It looks like there's a few tests failing as well in CI

We can remove it but, then, we will always reset, right?. With the CheckBox we can mark and decide if we want to reset or not.

For now let's just always reset.

If there are two text boxes then the second text box can always be used to swap the main page If a test doesn't want to swap the main page then it can use normal navigation

PureWeen avatar Apr 05 '24 02:04 PureWeen

Working through this one a bit.

Just wanted to add a request changes comment here so it's not merged yet.

Let me know if you have feedback, request changes etc :)

jsuarezruiz avatar Apr 25 '24 13:04 jsuarezruiz