diff --git a/test/bundler/bundler_plugin.test.ts b/test/bundler/bundler_plugin.test.ts index 1bfeb88e9cb6..7d5b2dee15cc 100644 --- a/test/bundler/bundler_plugin.test.ts +++ b/test/bundler/bundler_plugin.test.ts @@ -26,7 +26,6 @@ describe("bundler", () => { }; itBundled("plugin/Resolve", { - todo: true, files: resolveFixture, // The bundler testing api has a shorthand where the plugins array can be // the `setup` function of one plugin. @@ -85,7 +84,6 @@ describe("bundler", () => { }); itBundled("plugin/LoadThrowPrimative", { files: loadFixture, - todo: true, plugins(builder) { builder.onLoad({ filter: /\.magic$/ }, args => { throw "123"; @@ -108,7 +106,6 @@ describe("bundler", () => { }); itBundled("plugin/LoadThrowPrimativeAsync", { files: loadFixture, - todo: true, plugins(builder) { builder.onLoad({ filter: /\.magic$/ }, async args => { throw 123; @@ -179,7 +176,6 @@ describe("bundler", () => { }); itBundled("plugin/ResolveThrowPrimative", { files: resolveFixture, - todo: true, plugins(builder) { builder.onResolve({ filter: /\.magic$/ }, args => { throw "123"; @@ -202,7 +198,6 @@ describe("bundler", () => { }); itBundled("plugin/ResolveThrowPrimativeAsync", { files: resolveFixture, - todo: true, plugins(builder) { builder.onResolve({ filter: /\.magic$/ }, async args => { throw 123; diff --git a/test/bundler/esbuild/dce.test.ts b/test/bundler/esbuild/dce.test.ts index fdec783c9e36..260d709d42d9 100644 --- a/test/bundler/esbuild/dce.test.ts +++ b/test/bundler/esbuild/dce.test.ts @@ -3173,7 +3173,6 @@ describe("bundler", () => { }); // im confused what this is testing. cross platform slash? there is none?? not even in the go source itBundled("dce/PackageJsonSideEffectsFalseCrossPlatformSlash", { - todo: true, files: { "/Users/user/project/src/entry.js": /* js */ ` import "demo-pkg/foo" diff --git a/test/bundler/esbuild/default.test.ts b/test/bundler/esbuild/default.test.ts index a17df2e2ff02..9fbc3d237c67 100644 --- a/test/bundler/esbuild/default.test.ts +++ b/test/bundler/esbuild/default.test.ts @@ -1,6 +1,6 @@ import assert from "assert"; import { describe, expect } from "bun:test"; -import { isMacOS, isMusl, osSlashes } from "harness"; +import { osSlashes } from "harness"; import path from "path"; import { dedent, ESBUILD_PATH, itBundled } from "../expectBundled"; @@ -2091,7 +2091,6 @@ describe("bundler", () => { } `, }, - todo: true, minifyIdentifiers: true, bundling: false, format: "cjs", @@ -4549,7 +4548,6 @@ describe("bundler", () => { }, }); itBundled("default/DefineInfiniteLoopESBuildIssue2407", { - todo: true, files: { "/entry.js": /* js */ ` a.b() @@ -5313,7 +5311,6 @@ describe("bundler", () => { }, }); const RequireShimSubstitutionBrowser = itBundled("default/RequireShimSubstitutionBrowser", { - todo: isMacOS || isMusl, files: { "/entry.js": /* js */ ` Promise.all([ @@ -5381,7 +5378,6 @@ describe("bundler", () => { }, }); itBundled("default/RequireShimSubstitutionNode", { - todo: isMacOS || isMusl, files: RequireShimSubstitutionBrowser.options.files, runtimeFiles: RequireShimSubstitutionBrowser.options.runtimeFiles, target: "node", diff --git a/test/bundler/esbuild/extra.test.ts b/test/bundler/esbuild/extra.test.ts index da25b1347629..33d96f7fad73 100644 --- a/test/bundler/esbuild/extra.test.ts +++ b/test/bundler/esbuild/extra.test.ts @@ -163,8 +163,6 @@ describe("bundler", () => { run: true, }); itBundled("extra/TypeofRequireESM", { - // we do not have require defined in target browser - todo: true, files: { "in.js": `check(typeof require)`, "runtime.js": ` @@ -1006,7 +1004,6 @@ describe("bundler", () => { run: true, }); itBundled(`extra/${minify.label || "NoMinify"}CatchScope2`, { - todo: true, files: { "in.js": ` let y diff --git a/test/bundler/esbuild/packagejson.test.ts b/test/bundler/esbuild/packagejson.test.ts index 6983b3162a29..468d39d7ed05 100644 --- a/test/bundler/esbuild/packagejson.test.ts +++ b/test/bundler/esbuild/packagejson.test.ts @@ -1810,7 +1810,6 @@ describe("bundler", () => { }, }); itBundled("packagejson/ImportSelfUsingRequire", { - todo: true, files: { "/Users/user/project/src/index.js": /* js */ ` module.exports = 'index' @@ -1840,7 +1839,6 @@ describe("bundler", () => { }, }); itBundled("packagejson/ImportSelfUsingImport", { - todo: true, files: { "/Users/user/project/src/index.js": /* js */ ` import xyz from "xyz" @@ -1869,7 +1867,6 @@ describe("bundler", () => { }, }); itBundled("packagejson/ImportSelfUsingRequireScoped", { - todo: true, files: { "/Users/user/project/src/index.js": /* js */ ` module.exports = 'index' @@ -1899,7 +1896,6 @@ describe("bundler", () => { }, }); itBundled("packagejson/ImportSelfUsingImportScoped", { - todo: true, files: { "/Users/user/project/src/index.js": /* js */ ` import xyz from "@some-scope/xyz"