Replace app termination with test complete event as referenced by issues nunit#114 and nunit#116.

Author: bstadick

src/nunit.xamarin/Helpers/ResultSummary.cs

@@ -35,7 +35,7 @@ namespace NUnit.Runner.Helpers
     /// <summary>
     ///     Helper class used to summarize the result of a test run.
     /// </summary>
-    internal class ResultSummary
+    public class ResultSummary
     {
         #region Private Fields
 
@@ -160,7 +160,7 @@ public TimeSpan Duration
         ///     Constructs a <see cref="ResultSummary"/> helper to summarize the <see cref="TestRunResult"/>.
         /// </summary>
         /// <param name="results">The test run results to summarize</param>
-        public ResultSummary(TestRunResult results)
+        internal ResultSummary(TestRunResult results)
         {
             _results = results;
             Initialize();

src/nunit.xamarin/Properties/Annotations.cs

@@ -1,6 +1,5 @@
 using System;
 
-#pragma warning disable 1591
 // ReSharper disable UnusedMember.Global
 // ReSharper disable UnusedParameter.Local
 // ReSharper disable MemberCanBePrivate.Global

src/nunit.xamarin/Services/TestOptions.cs

@@ -23,6 +23,8 @@
 
 using System;
 using System.IO;
+using System.Threading.Tasks;
+using NUnit.Runner.Helpers;
 
 namespace NUnit.Runner.Services
 {
@@ -47,11 +49,6 @@ public class TestOptions
         /// </summary>
         public bool AutoRun { get; set; }
 
-        /// <summary>
-        ///     Gets or sets if the application will terminate automatically after running the tests.
-        /// </summary>
-        public bool TerminateAfterExecution { get; set; }
-
         /// <summary>
         ///     Gets information about the tcp listener host and port.
         /// </summary>
@@ -70,6 +67,21 @@ public class TestOptions
 
         #endregion
 
+        #region Events
+
+        /// <summary>
+        ///     Delegate executed when the test run has completed.
+        /// </summary>
+        /// <param name="testResults">The results of the test run.</param>
+        public delegate void TestRunCompleted(ResultSummary testResults);
+
+        /// <summary>
+        ///     Event fired when the test run has completed.
+        /// </summary>
+        public event TestRunCompleted OnTestRunCompleted;
+
+        #endregion
+
         #region Constructors
 
         /// <summary>
@@ -82,5 +94,25 @@ public TestOptions()
         }
 
         #endregion
+
+        #region Internal Methods
+
+        /// <summary>
+        ///     Invokes <see cref="OnTestRunCompleted"/> when the test run has completed.
+        /// </summary>
+        /// <param name="testResults">The results of the test run.</param>
+        internal async Task InvokeOnTestRunCompleted(ResultSummary testResults)
+        {
+            if (OnTestRunCompleted != null)
+            {
+                await Task.Run(() => OnTestRunCompleted?.Invoke(testResults));
+            }
+            else
+            {
+                await Task.CompletedTask;
+            }
+        }
+
+        #endregion
     }
 }

src/nunit.xamarin/ViewModel/SummaryViewModel.cs

@@ -156,11 +156,11 @@ public SummaryViewModel()
             // Set the commands to display the results
             ViewAllResultsCommand = new Command(
                 async o => await Navigation.PushAsync(
-                    new ResultsView(new ResultsViewModel(_results.GetTestResults(), true))),
+                    new ResultsView(new ResultsViewModel(Results.GetTestResults(), true))),
                 o => !HasResults);
             ViewFailedResultsCommand = new Command(
                 async o => await Navigation.PushAsync(
-                    new ResultsView(new ResultsViewModel(_results.GetTestResults(), false))),
+                    new ResultsView(new ResultsViewModel(Results.GetTestResults(), false))),
                 o => !HasResults);
 
             // ReSharper restore AsyncVoidLambda
@@ -219,36 +219,18 @@ private async Task ExecuteTestsAsync()
             _resultProcessor = TestResultProcessor.BuildChainOfResponsibility(Options);
             await _resultProcessor.Process(summary).ConfigureAwait(false);
 
-            // Report results
+            // Report results on main thread as setting these properties will invoke binding updates
             Device.BeginInvokeOnMainThread(
-                () =>
+                // ReSharper disable once AsyncVoidLambda
+                async () =>
                 {
                     Results = summary;
                     Running = false;
 
-                    if (Options.TerminateAfterExecution)
-                    {
-                        TerminateWithSuccess();
-                    }
+                    await Options.InvokeOnTestRunCompleted(summary);
                 });
         }
 
-        /// <summary>
-        ///     Terminates the test runner.
-        /// </summary>
-        /// <remarks>This method is platform specific and not always guaranteed to work.</remarks>
-        private static void TerminateWithSuccess()
-        {
-#if __IOS__
-            var selector = new ObjCRuntime.Selector("terminateWithSuccess");
-            UIKit.UIApplication.SharedApplication.PerformSelector(selector, UIKit.UIApplication.SharedApplication, 0);
-#elif __DROID__
-            System.Environment.Exit(0);
-#elif WINDOWS_UWP
-            Windows.UI.Xaml.Application.Current.Exit();
-#endif
-        }
-
         #endregion
     }
 }

tests/nunit.runner.tests.Droid/MainActivity.cs

@@ -56,9 +56,6 @@ protected override void OnCreate(Bundle savedInstanceState)
                 // otherwise you must run them manually.
                 AutoRun = true,
 
-                // If True, the application will terminate automatically after running the tests.
-                //TerminateAfterExecution = true,
-
                 // Information about the tcp listener host and port.
                 // For now, send result as XML to the listening server.
                 //TcpWriterParameters = new TcpWriterInfo("192.168.0.108", 13000),
@@ -71,6 +68,9 @@ protected override void OnCreate(Bundle savedInstanceState)
                     Environment.DirectoryDownloads ?? string.Empty, "Nunit", "Results.xml")
             };
 
+            // Sets the command to execute after running the tests such as custom handling of the results or exiting the application.
+            //nunit.Options.OnTestRunCompleted += (testResults) => System.Environment.Exit(0);
+
             LoadApplication(nunit);
         }
     }

tests/nunit.runner.tests.iOS/AppDelegate.cs

@@ -64,9 +64,6 @@ public override bool FinishedLaunching(UIApplication app, NSDictionary options)
                 // otherwise you must run them manually.
                 AutoRun = true,
 
-                // If True, the application will terminate automatically after running the tests.
-                //TerminateAfterExecution = true,
-
                 // Information about the tcp listener host and port.
                 // For now, send result as XML to the listening server.
                 //TcpWriterParameters = new TcpWriterInfo("192.168.0.108", 13000),
@@ -81,6 +78,14 @@ public override bool FinishedLaunching(UIApplication app, NSDictionary options)
                             NSSearchPathDomain.User)[0].Path, "Results.xml")
             };
 
+            // Sets the command to execute after running the tests such as custom handling of the results or exiting the application.
+            //nunit.Options.OnTestRunCompleted += (testResults) =>
+            //{
+            //    ObjCRuntime.Selector selector = new ObjCRuntime.Selector("terminateWithSuccess");
+            //    UIKit.UIApplication.SharedApplication.PerformSelector(selector, UIKit.UIApplication.SharedApplication,
+            //        0);
+            //};
+
             LoadApplication(nunit);
 
             return base.FinishedLaunching(app, options);

tests/nunit.runner.tests.uwp/MainPage.xaml.cs

@@ -52,9 +52,6 @@ public MainPage()
                 // otherwise you must run them manually.
                 AutoRun = true,
 
-                // If True, the application will terminate automatically after running the tests.
-                //TerminateAfterExecution = true,
-
                 // Information about the tcp listener host and port.
                 // For now, send result as XML to the listening server.
                 // NOTE: Your UWP App must have Private Networks capability enabled
@@ -67,6 +64,10 @@ public MainPage()
                 // ResultFilePath = System.IO.Path.Combine(Windows.Storage.ApplicationData.Current.TemporaryFolder.Path, "Nunit", "Results.xml")
             };
 
+            // Sets the command to execute after running the tests such as custom handling of the results or exiting the application.
+            //nunit.Options.OnTestRunCompleted += (testResults) => Windows.UI.Xaml.Application.Current.Exit();
+
+
             LoadApplication(nunit);
         }
     }

tests/nunit.runner.tests.uwp/Properties/Default.rd.xml

@@ -12,7 +12,7 @@
     <TypeInstantiation Name="App1.AppClass" Arguments="System.Int32" Activate="Required Public" />
 
     Using the Namespace directive to apply reflection policy to all the types in a particular namespace
-    <Namespace Name="DataClasses.ViewModels" Seralize="All" />
+    <Namespace Name="DataClasses.ViewModels" Serialize="All" />
 -->
 
 <Directives xmlns="http://schemas.microsoft.com/netfx/2013/01/metadata">