updating Deno version (#636)

* updating Deno version

Hoping to fix #377

* Updating how deno tests assert

* Updating Deno version

* Updating to deno 2.5.4

* Updating the deno sample app implementation and tests

Author: aaronpowell

.github/actions/setup-runtimes-caching/action.yml

@@ -66,7 +66,7 @@ runs:
     - uses: denoland/setup-deno@v2
       name: Setup Deno
       with:
-        deno-version: v2.1.4
+        deno-version: v2.5.4
 
     - uses: oven-sh/setup-bun@v2
       name: Setup Bun

examples/deno/CommunityToolkit.Aspire.Hosting.Deno.AppHost/Program.cs

@@ -5,10 +5,12 @@
 builder.AddDenoTask("vite-demo", taskName: "dev")
     .WithDenoPackageInstallation()
     .WithHttpEndpoint(env: "PORT")
-    .WithEndpoint();
+    .WithEndpoint()
+    .WithHttpHealthCheck("/");
 
-builder.AddDenoApp("oak-demo", "main.ts", permissionFlags: ["-E", "--allow-net"])
+builder.AddDenoApp("oak-demo", "main.ts", permissionFlags: ["--allow-env", "--allow-net"])
     .WithHttpEndpoint(env: "PORT")
-    .WithEndpoint();
+    .WithEndpoint()
+    .WithHttpHealthCheck("/health");
 
 builder.Build().Run();

examples/deno/oak-demo/deno.json

@@ -3,6 +3,7 @@
     "dev": "deno run --allow-env --allow-net --watch main.ts"
   },
   "imports": {
+    "@oak/oak": "jsr:@oak/oak@^17.1.6",
     "@std/assert": "jsr:@std/assert@1"
   }
 }

examples/deno/oak-demo/deno.lock

@@ -1,35 +1,34 @@
-import {
-  Application,
-  Context,
-  Router,
-} from "https://deno.land/x/[email protected]/mod.ts";
+import { Application, Router, Context } from "@oak/oak";
 
-const port = parseInt(Deno.env.get('PORT') || '8000');
+const port = parseInt(Deno.env.get("PORT") || "8000");
 
 const router = new Router();
 
 interface Weather {
-  Date: Date;
-  TemperatureC: number;
+    Date: Date;
+    TemperatureC: number;
 }
 
-router
-  .get("/weather", async (ctx: Context) => {
-    let forecast: Weather[] = [];
-    for(var i = 0; i < 5; i++) {
-      const weather: Weather = {
-        Date: new Date(new Date().setDate(new Date().getDate() + i)),
-        TemperatureC: Math.round(Math.random() * 36),
-      };
-      forecast.push(weather);
+router.get("/weather", (ctx: Context) => {
+    const forecast: Weather[] = [];
+    for (let i = 0; i < 5; i++) {
+        const weather: Weather = {
+            Date: new Date(new Date().setDate(new Date().getDate() + i)),
+            TemperatureC: Math.round(Math.random() * 36),
+        };
+        forecast.push(weather);
     }
     ctx.response.body = forecast;
-  });
+});
+
+router.get("/health", (ctx: Context) => {
+    ctx.response.body = { status: "ok" };
+});
 
 const app = new Application();
 
 app.use(router.routes());
 app.use(router.allowedMethods());
 
 console.log(`Server listening on port ${port}`);
-await app.listen({ port: port });
+await app.listen({ port });

examples/deno/oak-demo/main.ts

@@ -2,17 +2,18 @@
 
 namespace CommunityToolkit.Aspire.Hosting.Deno.Tests;
 
-#pragma warning disable CTASPIRE001
 public class AppHostTests(AspireIntegrationTestFixture<Projects.CommunityToolkit_Aspire_Hosting_Deno_AppHost> fixture) : IClassFixture<AspireIntegrationTestFixture<Projects.CommunityToolkit_Aspire_Hosting_Deno_AppHost>>
 {
     [Fact]
     public async Task ResourceStartsAndRespondsOk()
     {
         var appName = "vite-demo";
-        var httpClient = fixture.CreateHttpClient(appName);
 
-        await fixture.App.WaitForTextAsync("VITE", appName).WaitAsync(TimeSpan.FromSeconds(30));
+        await fixture.ResourceNotificationService
+            .WaitForResourceHealthyAsync(appName)
+            .WaitAsync(TimeSpan.FromSeconds(30));
 
+        var httpClient = fixture.CreateHttpClient(appName);
         var response = await httpClient.GetAsync("/");
 
         Assert.Equal(HttpStatusCode.OK, response.StatusCode);
@@ -22,10 +23,12 @@ public async Task ResourceStartsAndRespondsOk()
     public async Task ApiResourceStartsAndRespondsOk()
     {
         var appName = "oak-demo";
-        var httpClient = fixture.CreateHttpClient(appName);
 
-        await fixture.App.WaitForTextAsync("Server listening on port ", appName).WaitAsync(TimeSpan.FromSeconds(30));
+        await fixture.ResourceNotificationService
+            .WaitForResourceHealthyAsync(appName)
+            .WaitAsync(TimeSpan.FromSeconds(30));
 
+        var httpClient = fixture.CreateHttpClient(appName);
         var response = await httpClient.GetAsync("/weather");
 
         Assert.Equal(HttpStatusCode.OK, response.StatusCode);