Remove duplicate tests

- Remove duplicate 'errors on duplicate model names' test in models.test.ts
  (already covered in openapi-output.test.ts)
- Remove duplicate 'no models don't filter core' test (identical to
  'no models filter core')
- Remove broken 'don't emit' test in property-name-conflict.test.ts
  (truncated name, identical body to previous test)
- Remove duplicate 'one client from multiple services without version
  dependency' test (identical body to 'one client from multiple services')
- Remove duplicate LRO test 'custom put operations without finalLocation'
  (identical to 'custom createOrReplace operations without finalLocation')
- Remove duplicate 'example default config' and 'load example without
  version' tests (identical to 'example config')

Author: github-actions[bot]

packages/typespec-autorest/test/openapi-output.test.ts

@@ -22,33 +22,6 @@ describe("definitions", () => {
     });
   });
 
-  it("errors on duplicate model names", async () => {
-    const diagnostics = await diagnoseOpenApiFor(
-      `
-      model P {
-        p: string;
-      }
-
-      @friendlyName("P")
-      model Q {
-        q: string;
-      }
-
-      #suppress "@azure-tools/typespec-azure-core/use-standard-operations" "This is a test."
-      @route("/test1")
-      @get
-      op test1(p: P): Q;
-      `,
-    );
-
-    expectDiagnostics(diagnostics, [
-      {
-        code: "@typespec/openapi/duplicate-type-name",
-        message: /type/,
-      },
-    ]);
-  });
-
   it("doesn't define anonymous or unconnected models", async () => {
     const res: any = await oapiForModel(
       "{ ... Foo }",

packages/typespec-azure-core/test/operations.test.ts

@@ -2062,66 +2062,6 @@ JobLroResponse) | Azure.Core.Foundations.ErrorResponse;
     deepStrictEqual(metadata.pollingInfo.terminationStatus.succeededState, ["Succeeded"]);
   });
 
-  it("Gets Lro for non-standard polling in custom put operations without finalLocation", async () => {
-    const [_, metadata] = await compileLroOperation(
-      `
-      model PollingStatus {
-        @Azure.Core.lroStatus
-        statusValue: "Succeeded" | "Canceled" | "Failed" | "Running";
-      }
-      
-      model SimpleWidget {
-        @key
-        @segment("simpleWidgets")
-        @visibility(Lifecycle.Read)
-        @path
-        id: string;
-      
-        value: string;
-      }
-      
-      @route("/simpleWidgets/{id}/operations/{operationId}")
-      @get op getStatus(@path id: string, @path operationId: string): PollingStatus;
-      
-      @pollingOperation(getStatus)
-      @route("/simpleWidgets/{id}")
-      @createsOrReplacesResource(SimpleWidget)
-      @put op createWidget(@path id: string, body: SimpleWidget) : SimpleWidget | 
-        {
-          @statusCode statusCode: 201;
-          @header id: string,
-          @header("operation-id") operate: string,
-          @pollingLocation @header("Operation-Location") opLink: string,
-          @lroResult @bodyRoot body?: SimpleWidget
-        };      
-      `,
-      "createWidget",
-    );
-
-    ok(metadata);
-    deepStrictEqual(metadata.finalStateVia, "original-uri");
-    deepStrictEqual(metadata.logicalResult.name, "SimpleWidget");
-    deepStrictEqual(metadata?.envelopeResult.name, "PollingStatus");
-    deepStrictEqual(metadata?.logicalPath, undefined);
-
-    deepStrictEqual((metadata.finalResult as Model)?.name, "SimpleWidget");
-    deepStrictEqual((metadata.finalEnvelopeResult as Model)?.name, "SimpleWidget");
-    deepStrictEqual(metadata?.finalResultPath, undefined);
-
-    ok(metadata.statusMonitorStep);
-    deepStrictEqual(metadata.statusMonitorStep.target.kind, "link");
-    deepStrictEqual((metadata.statusMonitorStep.target as any)?.location, "ResponseHeader");
-    deepStrictEqual((metadata.statusMonitorStep.target as any)?.property.name, "opLink");
-
-    ok(metadata.pollingInfo);
-    deepStrictEqual(metadata.pollingInfo.responseModel.name, "PollingStatus");
-    deepStrictEqual(metadata.pollingInfo.terminationStatus.kind, "model-property");
-    deepStrictEqual(metadata.pollingInfo.terminationStatus.property.name, "statusValue");
-    deepStrictEqual(metadata.pollingInfo.terminationStatus.canceledState, ["Canceled"]);
-    deepStrictEqual(metadata.pollingInfo.terminationStatus.failedState, ["Failed"]);
-    deepStrictEqual(metadata.pollingInfo.terminationStatus.succeededState, ["Succeeded"]);
-  });
-
   it("Gets Lro for non-standard polling in post operations", async () => {
     const [_, metadata] = await compileLroOperation(
       `

packages/typespec-client-generator-core/test/clients/structure.test.ts

@@ -957,122 +957,6 @@ it("one client from multiple services with no versioning", async () => {
   strictEqual(biOperation.parameters.length, 0);
 });
 
-it("one client from multiple services without version dependency", async () => {
-  const { program } = await SimpleBaseTester.compile(
-    createClientCustomizationInput(
-      `
-        @service
-        @versioned(VersionsA)
-        namespace ServiceA {
-          enum VersionsA {
-            av1,
-            av2,
-          }
-          interface AI {
-            @route("/aTest")
-            aTest(@query("api-version") apiVersion: VersionsA): void;
-          }
-        }
-        @service
-        @versioned(VersionsB)
-        namespace ServiceB {
-          enum VersionsB {
-            bv1,
-            bv2,
-          }
-          interface BI {
-            @route("/bTest")
-            bTest(@query("api-version") apiVersion: VersionsB): void;
-          }
-    }`,
-      `
-        @client(
-          {
-            name: "CombineClient",
-            service: [ServiceA, ServiceB],
-            autoMergeService: true,
-          }
-        )
-        namespace CombineClient;
-      `,
-    ),
-  );
-  const context = await createSdkContextForTester(program);
-  const sdkPackage = context.sdkPackage;
-  strictEqual(sdkPackage.clients.length, 1);
-  const aVersionsEnum = sdkPackage.enums.find((e) => e.name === "VersionsA");
-  ok(aVersionsEnum);
-  const bVersionsEnum = sdkPackage.enums.find((e) => e.name === "VersionsB");
-  ok(bVersionsEnum);
-  const client = sdkPackage.clients[0];
-  strictEqual(client.name, "CombineClient");
-  // For root client of multiple services, the `apiVersions` will be empty.
-  strictEqual(client.apiVersions.length, 0);
-  strictEqual(client.children!.length, 2);
-  strictEqual(client.clientInitialization.parameters.length, 2);
-  ok(client.clientInitialization.parameters.find((p) => p.name === "endpoint"));
-  const apiVersionParam = client.clientInitialization.parameters.find(
-    (p) => p.name === "apiVersion",
-  );
-  ok(apiVersionParam);
-  strictEqual(apiVersionParam.apiVersions.length, 0);
-  strictEqual(apiVersionParam.clientDefaultValue, undefined);
-  // For multi-service clients, the api version param type should be string
-  strictEqual(apiVersionParam.type.kind, "string");
-  // For multi-service clients, the API version parameter should always be optional
-  strictEqual(apiVersionParam.optional, true);
-
-  const aiClient = client.children!.find((c) => c.name === "AI");
-  ok(aiClient);
-
-  // AI client should have api versions from ServiceA
-  strictEqual(aiClient.apiVersions.length, 2);
-  deepStrictEqual(aiClient.apiVersions, ["av1", "av2"]);
-  strictEqual(aiClient.clientInitialization.parameters.length, 2);
-  strictEqual(aiClient.clientInitialization.parameters[0].name, "endpoint");
-  strictEqual(aiClient.clientInitialization.parameters[1].name, "apiVersion");
-  const aiApiVersionParam = aiClient.clientInitialization.parameters[1];
-  strictEqual(aiApiVersionParam.isApiVersionParam, true);
-  strictEqual(aiApiVersionParam.onClient, true);
-  strictEqual(aiApiVersionParam.clientDefaultValue, "av2");
-
-  // AI client should have aTest method with VersionsA api version
-  strictEqual(aiClient.methods.length, 1);
-  const aiMethod = aiClient.methods[0];
-  strictEqual(aiMethod.name, "aTest");
-  strictEqual(aiMethod.parameters.length, 0);
-  const aiOperation = aiMethod.operation;
-  strictEqual(aiOperation.parameters.length, 1);
-  const aiOperationApiVersionParam = aiOperation.parameters.find((p) => p.isApiVersionParam);
-  ok(aiOperationApiVersionParam);
-  strictEqual(aiOperationApiVersionParam.correspondingMethodParams.length, 1);
-  strictEqual(aiOperationApiVersionParam.correspondingMethodParams[0], aiApiVersionParam);
-
-  const biClient = client.children!.find((c) => c.name === "BI");
-  ok(biClient);
-
-  strictEqual(biClient.apiVersions.length, 2);
-  deepStrictEqual(biClient.apiVersions, ["bv1", "bv2"]);
-  strictEqual(biClient.clientInitialization.parameters.length, 2);
-  strictEqual(biClient.clientInitialization.parameters[0].name, "endpoint");
-  strictEqual(biClient.clientInitialization.parameters[1].name, "apiVersion");
-  const biApiVersionParam = biClient.clientInitialization.parameters[1];
-  strictEqual(biApiVersionParam.isApiVersionParam, true);
-  strictEqual(biApiVersionParam.onClient, true);
-  strictEqual(biApiVersionParam.clientDefaultValue, "bv2");
-
-  // BI client should have bTest method with VersionsB api version
-  const biMethod = biClient.methods[0];
-  strictEqual(biMethod.name, "bTest");
-  strictEqual(biMethod.parameters.length, 0);
-  const biOperation = biMethod.operation;
-  strictEqual(biOperation.parameters.length, 1);
-  const biOperationApiVersionParam = biOperation.parameters.find((p) => p.isApiVersionParam);
-  ok(biOperationApiVersionParam);
-  strictEqual(biOperationApiVersionParam.correspondingMethodParams.length, 1);
-  strictEqual(biOperationApiVersionParam.correspondingMethodParams[0], biApiVersionParam);
-});
-
 it("one client from multiple services with `@clientLocation`", async () => {
   const { program } = await SimpleBaseTester.compile(
     createClientCustomizationInput(

packages/typespec-client-generator-core/test/examples/load.test.ts

@@ -27,24 +27,6 @@ it("example config", async () => {
   strictEqual(operation.examples![0].filePath, "get.json");
 });
 
-it("example default config", async () => {
-  const instance = await SimpleTester.createInstance();
-  await instance.fs.addRealTypeSpecFile("./examples/get.json", `${__dirname}/load/get.json`);
-  const { program } = await instance.compile(`
-    @service
-    namespace TestClient {
-      op get(): string;
-    }
-  `);
-  const context = await createSdkContextForTester(program);
-
-  const operation = (context.sdkPackage.clients[0].methods[0] as SdkServiceMethod<SdkHttpOperation>)
-    .operation;
-  ok(operation);
-  strictEqual(operation.examples?.length, 1);
-  strictEqual(operation.examples![0].filePath, "get.json");
-});
-
 it("no example folder found", async () => {
   const { program } = await SimpleTester.compile(`
     @service
@@ -61,24 +43,6 @@ it("no example folder found", async () => {
   });
 });
 
-it("load example without version", async () => {
-  const instance = await SimpleTester.createInstance();
-  await instance.fs.addRealTypeSpecFile("./examples/get.json", `${__dirname}/load/get.json`);
-  const { program } = await instance.compile(`
-    @service
-    namespace TestClient {
-      op get(): string;
-    }
-  `);
-  const context = await createSdkContextForTester(program);
-
-  const operation = (context.sdkPackage.clients[0].methods[0] as SdkServiceMethod<SdkHttpOperation>)
-    .operation;
-  ok(operation);
-  strictEqual(operation.examples?.length, 1);
-  strictEqual(operation.examples![0].filePath, "get.json");
-});
-
 it("load example with version", async () => {
   const instance = await SimpleTester.createInstance();
   await instance.fs.addRealTypeSpecFile("./examples/v3/get.json", `${__dirname}/load/get.json`);

packages/typespec-client-generator-core/test/rules/property-name-conflict.test.ts

@@ -157,22 +157,3 @@ it("emit warning if spread property name conflicts with model name", async () =>
       message: `Property 'foo' having the same name as its enclosing model will cause problems with C# code generation. Consider renaming the property directly or using the @clientName("newName", "csharp") decorator to rename the property for C#.`,
     });
 });
-
-it("don't emit ", async () => {
-  await tester
-    .expect(
-      `
-      model Base {
-        foo: string
-      }
-
-      model Foo {
-        ...Base;
-      }
-      `,
-    )
-    .toEmitDiagnostics({
-      code: "@azure-tools/typespec-client-generator-core/property-name-conflict",
-      message: `Property 'foo' having the same name as its enclosing model will cause problems with C# code generation. Consider renaming the property directly or using the @clientName("newName", "csharp") decorator to rename the property for C#.`,
-    });
-});

packages/typespec-client-generator-core/test/types/model.test.ts

@@ -930,16 +930,6 @@ it("no models filter core", async () => {
   strictEqual(models.length, 0);
 });
 
-it("no models don't filter core", async () => {
-  const { program } = await SimpleTester.compile(`
-    @service
-    namespace MyService { }
-  `);
-  const context = await createSdkContextForTester(program);
-  const models = context.sdkPackage.models;
-  strictEqual(models.length, 0);
-});
-
 it("input usage", async () => {
   const { program } = await SimpleTesterWithService.compile(`
     model InputModel {