Skip to content

Commit

Permalink
[Testing] Enable Catalyst UI Tests with Screenshot Verification - 2 (#…
Browse files Browse the repository at this point in the history
…26874)

* 6 issues enable the mac for snapshots

* modified issues

* updated changes

* Update Issue12213.cs

* Added mac images and fixed one flaky tests

* Added mac image and fix for one flaky test

---------

Co-authored-by: LogishaSelvarajSF4525 <[email protected]>
  • Loading branch information
1 parent 2f51dd9 commit 46096ff
Show file tree
Hide file tree
Showing 27 changed files with 29 additions and 27 deletions.
Binary file not shown.
6 changes: 5 additions & 1 deletion src/Controls/tests/TestCases.HostApp/Issues/Issue12213.cs
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,11 @@ protected override void Init()

private void TapGestureRecognizer_Tapped(object sender, TappedEventArgs e)
{
DisplayAlert("Entry", "Tapped", "OK");
Label label = new Label { Text = "Tapped" };
if (Content is Layout layout)
{
layout.Children.Add(label);
}
}
}
}
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
namespace Maui.Controls.Sample.Issues
{

[Issue(IssueTracker.Github, 25038, "MAUI Entry in Windows always shows ClearButton if initially hidden and shown even if ClearButtonVisibility set to 'Never'", PlatformAffected.UWP)]
[Issue(IssueTracker.Github, 25038, "MAUI Entry in Windows always shows ClearButton if initially hidden and shown even if ClearButtonVisibility set to Never", PlatformAffected.UWP)]
public partial class Issue25038 : ContentPage
{
public Issue25038()
Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ public void ControlCanBeFocusedByUnfocusedEvent()
{
App.WaitForElement(TheEntry);
App.EnterText(TheEntry, Success);
App.WaitForElement(Success);
App.WaitForElementTillPageNavigationSettled(Success);
}
}
}
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#if !MACCATALYST && !WINDOWS
#if TEST_FAILS_ON_WINDOWS //for more information:https://github.com/dotnet/maui/issues/24968
using NUnit.Framework;
using UITest.Appium;
using UITest.Core;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
#if !MACCATALYST
using NUnit.Framework;
using NUnit.Framework;
using UITest.Appium;
using UITest.Core;

Expand All @@ -20,8 +19,7 @@ public void TapGestureRecognizerNotWorkingOnEntry()
{
App.WaitForElement("Entry");
App.Tap("Entry");
VerifyScreenshot();
App.WaitForElement("Tapped");
}
}
}
#endif
}
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
using NUnit.Framework;
#if TEST_FAILS_ON_WINDOWS
using NUnit.Framework;
using UITest.Appium;
using UITest.Core;

Expand All @@ -13,7 +14,6 @@ public Issue17283(TestDevice testDevice) : base(testDevice) { }
[Test]
[Category(UITestCategories.CarouselView)]
[FailsOnWindowsWhenRunningOnXamarinUITest("Currently fails on Windows; see https://github.com/dotnet/maui/issues/24482")]
[FailsOnMacWhenRunningOnXamarinUITest("VerifyScreenshot is not implemented on Mac")]
public void CarouselViewShouldScrollToRightPosition()
{
App.WaitForElement("goToLastItemButton");
Expand All @@ -24,4 +24,5 @@ public void CarouselViewShouldScrollToRightPosition()
VerifyScreenshot();
}
}
}
}
#endif
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ public Issue18775(TestDevice device) : base(device)

[Test]
[Category(UITestCategories.TabbedPage)]
[FailsOnMacWhenRunningOnXamarinUITest]
public void TabbedPageUnselectedBarTextColorConsistency()
{
App.WaitForElement("MauiLabel");
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#if !ANDROID && !MACCATALYST
#if TEST_FAILS_ON_ANDROID //more information: https://github.com/dotnet/maui/issues/26050
using NUnit.Framework;
using UITest.Appium;
using UITest.Core;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@ public Issue22104(TestDevice testDevice) : base(testDevice)

[Test]
[Category(UITestCategories.CollectionView)]
[FailsOnMacWhenRunningOnXamarinUITest]
[FailsOnIOSWhenRunningOnXamarinUITest]
public void VerifyCollectionViewVisualState()
{
App.WaitForElement("CollectionView");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@ public void Issue23291Test()
{
App.WaitForElement("button");
App.Click("button");
App.WaitForElement("openFragmentTwoButton");
App.WaitForElementTillPageNavigationSettled("openFragmentTwoButton");
App.Click("openFragmentTwoButton");
App.WaitForElement("fragmentTwoLabel");
App.WaitForElementTillPageNavigationSettled("fragmentTwoLabel");
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ public Issue24405(TestDevice testDevice) : base(testDevice)

[Test]
[Category(UITestCategories.Entry)]
[FailsOnMacWhenRunningOnXamarinUITest]
public void VerifyEntryHorizontalEndTextAlignmentPosition()
{
App.WaitForElement("button");
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#if !MACCATALYST && !WINDOWS
#if TEST_FAILS_ON_WINDOWS // more information: https://github.com/dotnet/maui/issues/24968
using NUnit.Framework;
using UITest.Appium;
using UITest.Core;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,10 @@ public Issue25038(TestDevice testDevice) : base(testDevice)
{
}

public override string Issue => "MAUI Entry in Windows always shows ClearButton if initially hidden and shown even if ClearButtonVisibility set to 'Never'";
public override string Issue => "MAUI Entry in Windows always shows ClearButton if initially hidden and shown even if ClearButtonVisibility set to Never";

[Test]
[Category(UITestCategories.Entry)]
[FailsOnMacWhenRunningOnXamarinUITest]
public void VerifyEntryClearButtonVisibility()
{
App.WaitForElement("button");
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
#if !MACCATALYST
using NUnit.Framework;
using UITest.Appium;
using UITest.Core;
Expand All @@ -20,9 +19,14 @@ public void SearchBarMaxLength()
App.EnterText("SearchBar", "r");
App.Click("ChangeValue");
App.EnterText("SearchBar", "r");
App.EnterText("SearchBar", "c");
App.EnterText("SearchBar", "c");
#if MACCATALYST
// On MacCatalyst, pressing the ESC key during screenshot capture clears the text.
// This causes the image generated in CI to differ from local runs.
Assert.That(App.WaitForElement("Sear").GetText(), Is.EqualTo("Sear"));
#else
VerifyScreenshot();
#endif
}
}
}
#endif
}
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#if !MACCATALYST
#if TEST_FAILS_ON_CATALYST // more information:https://github.com/dotnet/maui/issues/26864
using System;
using System.Collections.Generic;
using System.Linq;
Expand Down
Binary file not shown.
Binary file not shown.

0 comments on commit 46096ff

Please sign in to comment.