fix: revert RLC unit test changes, change || back to &&, add modularUnit scenario test

Agent-Logs-Url: https://github.com/Azure/autorest.typescript/sessions/e6e4c70e-0694-45e4-9a47-b449e0e7f6b4

Co-authored-by: kazrael2119 <98569699+kazrael2119@users.noreply.github.com>

Author: Copilot

packages/typespec-ts/src/utils/operationUtil.ts

@@ -207,7 +207,7 @@ export function isBinaryPayload(
     knownMediaType(ct)
   );
   for (const type of knownMediaTypes) {
-    if (type === KnownMediaType.Binary || isBytes) {
+    if (type === KnownMediaType.Binary && isBytes) {
       return true;
     }
   }

packages/typespec-ts/test/modularUnit/scenarios/operations/bodyParam/bytesWithWildcardContentType.md

@@ -0,0 +1,54 @@
+# bytes body with */* content type should be treated as binary
+
+When an operation has `*/* ` content type and a `bytes` body, `isBinaryPayload` should return
+`true` so the parameter is typed as `Uint8Array` (binary) rather than `string` (base64).
+
+## TypeSpec
+
+```tsp
+@post op uploadFile(
+  @header contentType: "*/*",
+  @body body: bytes
+): void;
+```
+
+## Operations
+
+```ts operations
+import { TestingContext as Client } from "./index.js";
+import { UploadFileOptionalParams } from "./options.js";
+import {
+  StreamableMethod,
+  PathUncheckedResponse,
+  createRestError,
+  operationOptionsToRequestParameters,
+} from "@azure-rest/core-client";
+
+export function _uploadFileSend(
+  context: Client,
+  body: Uint8Array,
+  options: UploadFileOptionalParams = { requestOptions: {} },
+): StreamableMethod {
+  return context
+    .path("/")
+    .post({ ...operationOptionsToRequestParameters(options), contentType: "*/*", body: body });
+}
+
+export async function _uploadFileDeserialize(result: PathUncheckedResponse): Promise<void> {
+  const expectedStatuses = ["204"];
+  if (!expectedStatuses.includes(result.status)) {
+    throw createRestError(result);
+  }
+
+  return;
+}
+
+export async function uploadFile(
+  context: Client,
+  body: Uint8Array,
+  options: UploadFileOptionalParams = { requestOptions: {} },
+): Promise<void> {
+  const result = await _uploadFileSend(context, body, options);
+  return _uploadFileDeserialize(result);
+}
+```

packages/typespec-ts/test/unit/bytesGenerator.spec.ts

@@ -539,34 +539,6 @@ describe("bytes", () => {
       `
       );
     });
-    it("*/* content type with bytes body - should be treated as binary", async () => {
-      const parameters = await emitParameterFromTypeSpec(
-        `
-        @post op uploadFile(
-          @header contentType: "*/*",
-          @body body: bytes
-        ): void;
-        `
-      );
-      assert.ok(parameters);
-      await assertEqualContent(
-        parameters?.content!,
-        `
-        import type { RequestParameters } from "@azure-rest/core-client";
-
-        export interface UploadFileBodyParam {
-          /** Value may contain any sequence of octets */
-          body: string | Uint8Array | ReadableStream<Uint8Array> | NodeJS.ReadableStream;
-        }
-
-        export interface UploadFileMediaTypesParam {
-          contentType: "*/*";
-        }
-
-        export type UploadFileParameters = UploadFileMediaTypesParam & UploadFileBodyParam & RequestParameters;
-        `
-      );
-    });
     // TODO: we need more discussions about current behavior
     // This case is not finalized yet and some validations would be added in tcgc
     it.skip("mixed non-binary and binary content types - should report errors?", async () => {

packages/typespec-ts/test/unit/responsesGenerator.spec.ts

@@ -214,7 +214,7 @@ describe("Responses.ts", () => {
       `
       );
     });
-    it("@header contentType text/plain with bytes body - should be treated as binary (trust body type)", async () => {
+    it("@header contentType text/plain should keep format to byte(finally string)", async () => {
       const responses = await emitResponsesFromTypeSpec(`
       @get op read(): {@header contentType: "text/plain", @body body: bytes};
       `);
@@ -232,8 +232,7 @@ describe("Responses.ts", () => {
        /** The request has succeeded. */
        export interface Read200Response extends HttpResponse {
          status: "200";
-         /** Value may contain any sequence of octets */
-         body: Uint8Array;
+         body: string;
          headers: RawHttpHeaders & Read200Headers;
        }
       `