Merge pull request #497 from lightpanda-io/upgrade-jsruntime

ci: add a browser fetch test

Author: krichprollsch

.github/workflows/wpt.yml

@@ -41,8 +41,8 @@ jobs:
   wpt:
     name: web platform tests
 
-    # Don't run the CI with draft PR.
-    if: github.event.pull_request.draft == false
+    # Only for PR without draft.
+    if: github.event_name == 'pull_request' && github.event.pull_request.draft == false
 
     runs-on: ubuntu-latest
 

.github/workflows/zig-test.yml

@@ -66,6 +66,25 @@ jobs:
             zig-out/bin/lightpanda
           retention-days: 1
 
+  browser-fetch:
+    name: browser fetch
+    needs: zig-build-dev
+
+    # Don't run the CI with draft PR.
+    if: github.event.pull_request.draft == false
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: download artifact
+        uses: actions/download-artifact@v4
+        with:
+          name: lightpanda-build-dev
+
+      - run: chmod a+x ./lightpanda
+
+      - run: ./lightpanda fetch https://httpbin.io/xhr/get
+
   zig-test:
     name: zig test
 

src/main.zig

@@ -52,7 +52,7 @@ pub fn main() !void {
     const alloc = if (builtin.mode == .Debug) gpa.allocator() else std.heap.c_allocator;
 
     defer if (builtin.mode == .Debug) {
-        _ = gpa.detectLeaks();
+        if (gpa.detectLeaks()) std.posix.exit(1);
     };
 
     var args_arena = std.heap.ArenaAllocator.init(alloc);