revert unexpected test case changes

Author: kazrael2119

packages/typespec-ts/test-next/unit/metadata/package-json.test.ts

@@ -123,7 +123,7 @@ describe("Package file generation", () => {
       expect(packageFile["//sampleConfiguration"]).toEqual(expectedSampleConfig);
     });
 
-    it("[esm] should include correct base entrypoints by default", () => {
+    it("[esm] should include correct entrypoints (without react-native by default)", () => {
       const model = createMockModel({
         ...baseConfig,
         withSamples: true,
@@ -138,9 +138,13 @@ describe("Package file generation", () => {
       expect(packageFile).to.have.property("module", "./dist/esm/index.js");
       expect(packageFile).to.have.property("types", "./dist/commonjs/index.d.ts");
       expect(packageFile).to.have.property("browser", "./dist/browser/index.js");
+      // Default: no react-native entrypoint
+      expect(packageFile).not.toHaveProperty("react-native");
       expect(packageFile).to.have.property("exports");
       expect(packageFile.exports["./package.json"]).to.equal("./package.json");
       expect(packageFile.exports["."]).to.have.property("browser");
+      // Default: no react-native in exports
+      expect(packageFile.exports["."]).not.toHaveProperty("react-native");
       expect(packageFile.exports["."]).to.have.property("import");
       expect(packageFile.exports["."]).to.have.property("require");
       expect(packageFile.exports["."]["import"]).toEqual({
@@ -149,21 +153,20 @@ describe("Package file generation", () => {
       });
     });
 
-    it("[esm] should include consistent export conditions", () => {
+    it("[esm] should include react-native entrypoints when generateReactNativeTarget is true", () => {
       const model = createMockModel({
         ...baseConfig,
         withSamples: true,
+        generateReactNativeTarget: true,
       });
       const packageFileContent = buildPackageFile(model);
       const packageFile = JSON.parse(packageFileContent?.content ?? "{}");
 
-      expect(packageFile.exports["."]["browser"]).toEqual({
-        types: "./dist/browser/index.d.ts",
-        default: "./dist/browser/index.js",
-      });
-      expect(packageFile.exports["."]["require"]).toEqual({
-        types: "./dist/commonjs/index.d.ts",
-        default: "./dist/commonjs/index.js",
+      expect(packageFile).to.have.property("react-native", "./dist/react-native/index.js");
+      expect(packageFile.exports["."]).to.have.property("react-native");
+      expect(packageFile.exports["."]["react-native"]).toEqual({
+        types: "./dist/react-native/index.d.ts",
+        default: "./dist/react-native/index.js",
       });
     });
 
@@ -314,7 +317,7 @@ describe("Package file generation", () => {
       expect(packageFile.scripts).to.have.property("pack", "pnpm pack 2>&1");
     });
 
-    it("should include browser entrypoint by default for modular ARM packages", () => {
+    it("should include browser but not react-native entrypoints by default", () => {
       const model = createMockModel({
         ...baseConfig,
         azureArm: true,
@@ -324,21 +327,22 @@ describe("Package file generation", () => {
       const packageFile = JSON.parse(packageFileContent?.content ?? "{}");
 
       expect(packageFile).to.have.property("browser", "./dist/browser/index.js");
-      expect(packageFile.exports["."]).to.have.property("browser");
+      // Default: no react-native entrypoint
+      expect(packageFile).not.toHaveProperty("react-native");
     });
 
-    it("should include standard export conditions for modular ARM packages", () => {
+    it("should include react-native entrypoint when generateReactNativeTarget is true", () => {
       const model = createMockModel({
         ...baseConfig,
         azureArm: true,
         isModularLibrary: true,
+        generateReactNativeTarget: true,
       });
       const packageFileContent = buildPackageFile(model);
       const packageFile = JSON.parse(packageFileContent?.content ?? "{}");
 
       expect(packageFile).to.have.property("browser", "./dist/browser/index.js");
-      expect(packageFile.exports["."]).to.have.property("import");
-      expect(packageFile.exports["."]).to.have.property("require");
+      expect(packageFile).to.have.property("react-native", "./dist/react-native/index.js");
     });
   });
 

packages/typespec-ts/test-next/unit/metadata/warp-config.test.ts

@@ -7,32 +7,35 @@ import { buildWarpConfig } from "../../../src/metadata/build-warp-config.js";
 import { createMockModel } from "./mock-helper.js";
 
 describe("warp.config.yml generation", () => {
-  it("should generate a self-contained config without polyfillSuffix by default", () => {
+  it("should generate a self-contained config without polyfillSuffix and without react-native by default", () => {
     const model = createMockModel({});
 
     const result = buildWarpConfig(model);
     expect(result).toBeDefined();
     expect(result!.path).toBe("warp.config.yml");
     expect(result!.content).not.toContain("extends:");
     expect(result!.content).not.toContain("polyfillSuffix");
+    // Default: three targets without react-native
     expect(result!.content).toContain("name: browser");
+    expect(result!.content).not.toContain("name: react-native");
     expect(result!.content).toContain("name: esm");
     expect(result!.content).toContain("name: commonjs");
     expect(result!.content).toContain("tsconfig:");
     expect(result!.content).toContain('"./package.json"');
     expect(result!.content).toContain('"."');
   });
 
-  it("should include the standard build targets", () => {
-    const model = createMockModel({});
+  it("should include react-native target when generateReactNativeTarget is true", () => {
+    const model = createMockModel({
+      generateReactNativeTarget: true,
+    });
 
     const result = buildWarpConfig(model);
     expect(result).toBeDefined();
-    expect(result!.path).toBe("warp.config.yml");
     expect(result!.content).toContain("name: browser");
+    expect(result!.content).toContain("name: react-native");
     expect(result!.content).toContain("name: esm");
     expect(result!.content).toContain("name: commonjs");
-    expect(result!.content).toContain("moduleType: commonjs");
   });
 
   it("should include custom exports alongside base exports", () => {