Now detecting if tests are submitted

Author: csharpfritz

src/NotebookUnit/RunTestsMagicCommand.cs

@@ -4,6 +4,7 @@
 using System.Threading.Tasks;
 using Microsoft.DotNet.Interactive;
 using Microsoft.DotNet.Interactive.Events;
+using Microsoft.DotNet.Interactive.Formatting;
 
 namespace NotebookUnit
 {
@@ -14,28 +15,38 @@ public async Task OnLoadAsync(Kernel kernel)
 
       var testCommand = new Command("#!test", "Run the tests in this notebook")
       {
-        new Option<string>("-s", "Dunno why I'm adding this, I just want it to run")
+        new Option<Action[]>(new string[] {"-t", "--tests" }, () => new Action[] { }, "Specify tests to run")
       };
 
-      testCommand.Handler = CommandHandler.Create(async (KernelInvocationContext context) =>
-      {
-        context.Publish(
-        new DisplayedValueProduced
-          ("This is a test",
-          context.Command,
-          new[] { new FormattedValue("text/plain", "This is my formatted value") }
-          )
-        );
-      });
+      testCommand.Handler =
+      CommandHandler.Create(async (Action[] tests, KernelInvocationContext context) =>
+        RunTests(tests, context));
 
       kernel.AddDirective(testCommand);
 
-      // This doesn't appear to be working
-      //if (KernelInvocationContext.Current is { } context)
-      //{
-      //    await context.DisplayAsync($"`{nameof(RunTestsMagicCommand)}` is loaded. It adds unit test capabilities");
-      //}
+      if (KernelInvocationContext.Current is { } context)
+      {
+        await context.DisplayAsync($"`{nameof(RunTestsMagicCommand)}` is loaded. It enables testing in notebooks", "text/markdown");
+      }
 
     }
-  }
+
+		private object RunTests(Action[] tests, KernelInvocationContext context)
+		{
+
+      if (tests == null || tests.Length == 0) {
+        context.Fail(message: "No tests");
+        return null;
+			}
+
+      context.Publish(new DisplayedValueProduced("Tests to be run", context.Command, new[] {
+        new FormattedValue(PlainTextFormatter.MimeType, "Tests will be run")
+      }));
+
+      return new object();
+
+		}
+
+
+	}
 }

src/TestnotebookUnit.ipynb

@@ -10,7 +10,7 @@
       "text/html": [
        "\r\n",
        "<div>\r\n",
-       "    <div id='dotnet-interactive-this-cell-28751.6c5bc3419e7d4f0fba9610b517a8d24c' style='display: none'>\r\n",
+       "    <div id='dotnet-interactive-this-cell-21552.91bdddaa9c0f49eeb06e70d46df15153' style='display: none'>\r\n",
        "        The below script needs to be able to find the current output cell; this is an easy method to get it.\r\n",
        "    </div>\r\n",
        "    <script type='text/javascript'>\r\n",
@@ -70,23 +70,24 @@
        "}\r\n",
        "\r\n",
        "function loadDotnetInteractiveApi() {\r\n",
-       "    probeAddresses([\"http://192.168.1.151:1025/\", \"http://127.0.0.1:1025/\"])\r\n",
+       "    probeAddresses([\"http://192.168.1.71:1000/\", \"http://172.19.160.1:1000/\", \"http://127.0.0.1:1000/\"])\r\n",
        "        .then((root) => {\r\n",
        "            // use probing to find host url and api resources\r\n",
        "            // load interactive helpers and language services\r\n",
        "            let dotnetInteractiveRequire = require.config({\r\n",
-       "                context: '28751.6c5bc3419e7d4f0fba9610b517a8d24c',\r\n",
+       "                context: '21552.91bdddaa9c0f49eeb06e70d46df15153',\r\n",
        "                paths: {\r\n",
        "                    'dotnet-interactive': `${root}resources`\r\n",
        "                }\r\n",
        "            }) || require;\r\n",
        "\r\n",
        "            let dotnetInteractiveExtensionsRequire = require.config({\r\n",
-       "                context: '28751.6c5bc3419e7d4f0fba9610b517a8d24c',\r\n",
+       "                context: '21552.91bdddaa9c0f49eeb06e70d46df15153',\r\n",
        "                paths: {\r\n",
        "                    'dotnet-interactive-extensions': `${root}extensions`\r\n",
        "                }\r\n",
        "            }) || require;\r\n",
+       "\r\n",
        "            if (!window.dotnetInteractiveRequire) {\r\n",
        "                window.dotnetInteractiveRequire = dotnetInteractiveRequire;\r\n",
        "            }\r\n",
@@ -131,7 +132,16 @@
     {
      "data": {
       "text/html": [
-       "<div><strong>Restore sources</strong><ul><li><span>/home/csharpfritz/dev/csharp_with_csharpfritz/src/NotebookUnit/bin/Debug</span></li></ul></div>"
+       "<div><strong>Restore sources</strong><ul><li><span>c:/dev/csharp_with_csharpfritz/src/NotebookUnit/bin/Debug</span></li></ul></div>"
+      ]
+     },
+     "metadata": {},
+     "output_type": "display_data"
+    },
+    {
+     "data": {
+      "text/html": [
+       "<div><strong>Restore sources</strong><ul><li><span>c:/dev/csharp_with_csharpfritz/src/NotebookUnit/bin/Debug</span></li><li><span>c:/dev/csharp_with_csharpfritz/src</span></li></ul></div>"
       ]
      },
      "metadata": {},
@@ -145,20 +155,54 @@
      },
      "metadata": {},
      "output_type": "display_data"
+    },
+    {
+     "data": {
+      "text/plain": [
+       "Loaded kernel extension \"RunTestsMagicCommand\" from assembly C:\\Users\\jefritz\\.nuget\\packages\\notebookunit\\0.1.0\\interactive-extensions\\dotnet\\NotebookUnit.dll"
+      ]
+     },
+     "metadata": {},
+     "output_type": "display_data"
+    },
+    {
+     "data": {
+      "text/markdown": [
+       "`RunTestsMagicCommand` is loaded. It enables testing in notebooks"
+      ]
+     },
+     "metadata": {},
+     "output_type": "display_data"
     }
    ],
    "source": [
-    "#i nuget:/home/csharpfritz/dev/csharp_with_csharpfritz/src/NotebookUnit/bin/Debug\n",
+    "#i nuget:c:/dev/csharp_with_csharpfritz/src/NotebookUnit/bin/Debug\n",
+    "#i nuget:c:/dev/csharp_with_csharpfritz/src\n",
+    "\n",
     "#r nuget:NotebookUnit,0.1.0 "
    ]
   },
   {
    "cell_type": "code",
    "execution_count": 2,
    "metadata": {},
-   "outputs": [],
+   "outputs": [
+    {
+     "ename": "Unhandled exception",
+     "evalue": "No tests",
+     "output_type": "error",
+     "traceback": [
+      "No tests"
+     ]
+    }
+   ],
    "source": [
-    "#!test -h"
+    "public void DoThis() {\n",
+    "    display(\"Ran this\");\n",
+    "}\n",
+    "\n",
+    "\n",
+    "#!test -t new Action[] {() => true}"
    ]
   }
  ],