[FinallyDelegate] rely on PlatformNotSupportedException in order to determine remoting support (#10)

lewurm · web-flow · commit 70bb70b0ffd0 · 2018-02-26T16:47:11.000+01:00
diff --git a/NUnitLite-1.0.0/src/framework/FinallyDelegate.cs b/NUnitLite-1.0.0/src/framework/FinallyDelegate.cs
@@ -24,10 +24,6 @@
 // WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 // ***********************************************************************
 
-#if __IOS__ || __WATCHOS__ || __TVOS__
-#define NO_REMOTING
-#endif
-
 using System;
 using System.Diagnostics;
 using System.Threading;
@@ -76,14 +72,12 @@ public class FinallyDelegate
 		// Thread_1 creates Thread_2, it needs to have the same "thread local" values.
 		// that is achieved with `CallContext`.
 		//
-		// Unfortunately, remoting isn't available on every platform, thus we can't
-		// support this scenario. Luckily, this scenario is very rare.
+		// Unfortunately, remoting isn't available on every platform (it will
+		// throw PlatformNotSupportedexception), thus we can't support this
+		// scenario. Luckily, this scenario is very rare.
 
-#if NO_REMOTING
-		static Container container;
-#else
+		Container container = null;
 		private static readonly string CONTEXT_KEY = "TestResultName";
-#endif
 
 		public FinallyDelegate () {
 			this.testStack = new Stack<Tuple<TestExecutionContext, long, TestResult>>();
@@ -96,22 +90,18 @@ public void Set (TestExecutionContext context, long startTicks, TestResult resul
 			/* keep name in LogicalCallContext, because this will be inherited by
 			 * Threads spawned by the test case */
 			var guid = Guid.NewGuid();
-#if NO_REMOTING
-			container = new Container (guid);
-#else
-			CallContext.SetData(CONTEXT_KEY, new Container(guid));
-#endif
+			try {
+				CallContext.SetData(CONTEXT_KEY, new Container(guid));
+			} catch {
+				container = new Container (guid);
+			}
 
 			this.lookupTable.Add(guid, result);
 			this.testStack.Push(frame);
 		}
 
 		public void HandleUnhandledExc (Exception ex) {
-#if NO_REMOTING
-			Container c = container;
-#else
-			Container c = (Container) CallContext.GetData(CONTEXT_KEY);
-#endif
+			Container c = container ?? (Container) CallContext.GetData(CONTEXT_KEY);
 			TestResult result = this.lookupTable [c.guid];
 			result.RecordException(ex);
 			result.ThreadCrashFail = true;
