Skip to content

Commit 04de4fe

Browse files
authored
[tests] Cosmetic changes to the marshal methods hang test (#9357)
Context: #9352 Remove a misleading copy pasta comment and change the location of message we use to test whether the app actually ran.
1 parent 884c56e commit 04de4fe

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/MSBuildDeviceIntegration/Tests/MarshalMethodsGCHangTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ protected override void OnStart ()
4747
} catch (Java.IO.IOException) {
4848
GC.Collect ();
4949
}
50+
Console.WriteLine (""XXX:OnStart done"");
5051
}
5152
5253
class StreamMediaDataSource (System.IO.Stream data) : MediaDataSource
@@ -81,7 +82,6 @@ public override void Close ()
8182
}
8283
";
8384

84-
// All Tests here require the emulator to be started with -writable-system
8585
[Test]
8686
public void MarshalMethodsAppRuns ()
8787
{
@@ -103,7 +103,7 @@ public void MarshalMethodsAppRuns ()
103103
Assert.True (apkBuilder.Install (proj), "Project should have installed.");
104104
RunProjectAndAssert (proj, apkBuilder);
105105

106-
const string expectedLogcatOutput = "XXX:END //StreamMediaDataSource.ReadAt";
106+
const string expectedLogcatOutput = "XXX:OnStart done";
107107
Assert.IsTrue (
108108
MonitorAdbLogcat (
109109
InstallAndRunTests.CreateLineChecker (expectedLogcatOutput),

0 commit comments

Comments
 (0)