Unskip more bundler tests

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;

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"

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",

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

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"