From cd1c149210098594fd68ba2fdd290f00d3a5653c Mon Sep 17 00:00:00 2001 From: Adam Simon Date: Thu, 21 Nov 2024 22:39:41 +0100 Subject: [PATCH] Temporary disable tests on ubuntu-latest and macOS-latest for Bun --- .github/workflows/js-sdk-ci.yml | 8 ++++++-- README.md | 2 +- test/bun/index.ts | 2 +- 3 files changed, 8 insertions(+), 4 deletions(-) diff --git a/.github/workflows/js-sdk-ci.yml b/.github/workflows/js-sdk-ci.yml index 2f7e7a4..b30d0b9 100644 --- a/.github/workflows/js-sdk-ci.yml +++ b/.github/workflows/js-sdk-ci.yml @@ -104,8 +104,12 @@ jobs: runs-on: ${{ matrix.os }} strategy: matrix: - bun-version: ["1.1.0", "latest"] - os: [ubuntu-latest, windows-latest, macOS-latest] + bun-version: ["1.1.0", "1.1.34"] + # Bun fails to run tests on ubuntu-latest and macOS-latest. + # It may not be possible at all for now: https://github.com/oven-sh/bun/issues/10001 + # Bun version "latest" (1.1.36) seems to have a regression, + # so we use 1.1.34 until it fixed. + os: [windows-latest] name: Bun ${{ matrix.bun-version }} (${{ matrix.os }}) test steps: - uses: actions/checkout@v4 diff --git a/README.md b/README.md index 367eb76..c57b26d 100644 --- a/README.md +++ b/README.md @@ -222,7 +222,7 @@ The SDK is [tested](https://github.com/configcat/js-unified-sdk/blob/master/.git - Chromium (64.0.3282.0, 72.0.3626.0, 80.0.3987.0) - Firefox (latest, latest-beta, 84.0) - @configcat/sdk/bun: - - Bun (v1.1.0, latest stable) on Windows / Ubuntu / macOS + - Bun (v1.1.0, v1.1.34) on Windows - @configcat/sdk/deno: - Deno (v1.31, v1.46, latest stable) on Windows / Ubuntu / macOS - @configcat/sdk/node: diff --git a/test/bun/index.ts b/test/bun/index.ts index 61faacb..04c7e1f 100644 --- a/test/bun/index.ts +++ b/test/bun/index.ts @@ -54,6 +54,6 @@ for (const file of glob.globIterateSync(normalizePathSeparator(testDir) + "/**/* segments[segments.length - 1] = path.basename(fileName, path.extname(fileName)); /* eslint-disable @typescript-eslint/no-require-imports */ - require("../" + segments.join("/")); + await import("../" + segments.join("/")); } }