Skip to content

Commit e620d85

Browse files
committed
Enable tests that rely on one instance
Tests can be enabled again as the web driver has been fixed to not leave instances open by accident.
1 parent 38d005a commit e620d85

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

src/FlaUI.WebDriver.UITests/SessionTests.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,6 @@ public void NewSession_AppTopLevelWindowZero_ReturnsError()
154154
Assert.That(newSession, Throws.TypeOf<WebDriverArgumentException>().With.Message.EqualTo("Capability appium:appTopLevelWindow '0x0' should not be zero"));
155155
}
156156

157-
[Explicit("Sometimes multiple processes are left open")]
158157
[TestCase("FlaUI WPF Test App")]
159158
[TestCase("FlaUI WPF .*")]
160159
public void NewSession_AppTopLevelWindowTitleMatch_IsSupported(string match)
@@ -172,7 +171,7 @@ public void NewSession_AppTopLevelWindowTitleMatch_IsSupported(string match)
172171
Assert.That(testAppProcess.Process.HasExited, Is.False);
173172
}
174173

175-
[Test, Explicit("Sometimes multiple processes are left open")]
174+
[Test]
176175
public void NewSession_AppTopLevelWindowTitleMatchMultipleMatching_ReturnsError()
177176
{
178177
using var testAppProcess = new TestAppProcess();
@@ -319,7 +318,7 @@ public void NewCommandTimeout_NotExpired_DoesNotEndSession()
319318
Assert.That(() => driver.Title, Throws.Nothing);
320319
}
321320

322-
[Test, Explicit(("Sometimes multiple processes are left open"))]
321+
[Test]
323322
public void NewCommandTimeout_SessionWithAppTopLevelWindowTitleMatch_ClosesSessionButDoesNotCloseApp()
324323
{
325324
using var testAppProcess = new TestAppProcess();

0 commit comments

Comments
 (0)