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

Android gallery fixes #15118

Merged
merged 20 commits into from
May 19, 2023
Merged

Android gallery fixes #15118

merged 20 commits into from
May 19, 2023

Conversation

PureWeen
Copy link
Member

@PureWeen PureWeen commented May 16, 2023

Description of Change

  • Fixes a number of issues just related to running Xamarin.UITest with MAUI.
  • Adds 3 attributes for ignoring tests. These inherit from IgnoreAttribute on the platform they are failing from and then just from Category where they aren't
    • FailsOnMaui
    • FailsOnMauiAndroid
    • FailsOnMauiiOS

@github-actions
Copy link
Contributor

Thank you for your pull request. We are auto-formatting your source code to follow our code guidelines.

@Eilon Eilon added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label May 18, 2023
@PureWeen PureWeen requested a review from rmarinho May 18, 2023 17:51
@PureWeen PureWeen marked this pull request as ready for review May 18, 2023 17:51
@github-actions
Copy link
Contributor

Thank you for your pull request. We are auto-formatting your source code to follow our code guidelines.

@PureWeen PureWeen merged commit 4c1a400 into main May 19, 2023
@PureWeen PureWeen deleted the android_gallery_fixes branch May 19, 2023 14:45
rmarinho pushed a commit that referenced this pull request May 30, 2023
* - fix up back button

* - remove CarouselPage tests

* - add missing shim extension for grid

* - ignore carouselview tests

* - add better testing categories

* - wire up stubbed TestApp for nunit

* - remove material entry tests

* - remove relative layout tests

* - categorize TabbedPageTests

* - fix build errors

* - ignore cell tests

* Auto-format source code

* - add ignore attributes for android

* - add rest of ignores

* Auto-format source code

* - additional ignores

* - for now ignore on both platforms

* - add ignore

* - add ignore

* - remove platform tests from SLN

---------

Co-authored-by: GitHub Actions Autoformatter <autoformat@example.com>
@PureWeen PureWeen added the backport/suggested The PR author or issue review has suggested that the change should be backported. label Jun 2, 2023
@PureWeen
Copy link
Member Author

PureWeen commented Jun 2, 2023

Added backport/suggested

If we get the legacy gallery running on net7, I can run the current set of failing tests and see if anything is green on net7 but red on net8.

@hartez hartez removed the backport/suggested The PR author or issue review has suggested that the change should be backported. label Jun 7, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions platform/android 🤖 t/housekeeping ♻︎
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants