Upgrade to the latest dev dependency before TypeSpec GA#3135
Upgrade to the latest dev dependency before TypeSpec GA#3135MaryGao merged 32 commits intoAzure:mainfrom
Conversation
…nto monitor-nightly325
MaryGao
changed the title
| }); | ||
|
|
||
| it("bytes/bytes[] in general model - should be treated as base64 string", async () => { | ||
| it.skip("bytes/bytes[] in general model - should be treated as base64 string", async () => { |
| }); | ||
|
|
||
| it("bytes/bytes[]/union in general model- should be treated as base64 string", async () => { | ||
| it.skip("bytes/bytes[]/union in general model- should be treated as base64 string", async () => { |
| updateRun: (threadId: string, runId: string, options?: AgentsUpdateRunOptionalParams) => Promise<ThreadRun>; | ||
| updateThread: (threadId: string, options?: AgentsUpdateThreadOptionalParams) => Promise<AgentThread>; | ||
| uploadFile: (file: Uint8Array, purpose: FilePurpose, options?: AgentsUploadFileOptionalParams) => Promise<OpenAIFile>; | ||
| uploadFile: (body: { |
There was a problem hiding this comment.
Anonymous model in modular not supported - https://github.com/azure/autorest.typescript/blob/main/packages/typespec-ts/test/modularIntegration/payloadMultipart.spec.ts#L32-L36
|
|
||
| export declare interface FormDataOperations { | ||
| anonymousModel: (profileImage: Uint8Array, options?: FormDataAnonymousModelOptionalParams) => Promise<void>; | ||
| anonymousModel: (body: { |
There was a problem hiding this comment.
| // Not supported yet | ||
| // One column is removed in old version but added in latest version | ||
| // The only way is to cast to any | ||
| it.skip("versioning removed test modelV33_V2preview", async () => { |
There was a problem hiding this comment.
Skipped this case because we don't support this formally. We couldn't bypass this because spector has more strict check: microsoft/typespec#6565.
| // Not supported yet | ||
| // One column is removed in old version but added in latest version | ||
| // The only way is to cast to any | ||
| it.skip("versioning removed test modelV33_V2preview", async () => { |
There was a problem hiding this comment.
Skipped this case due to we don't support this formally. We couldn't bypass this because spector has more strict check: microsoft/typespec#6565.
| // Not supported yet | ||
| // One column is removed in old version but added in latest version | ||
| // The only way is to cast to any | ||
| it.skip("versioning removed test modelV33_V2preview", async () => { |
There was a problem hiding this comment.
Skipped this case due to we don't support this formally. We couldn't bypass this because spector has more strict check: microsoft/typespec#6565.
There was a problem hiding this comment.
@v-jiaodi Could you have another pr to re-write this case? We need to build different vesion for different api-version not the same version to share them.
| { | ||
| subscriptionId: context.subscriptionId, | ||
| "api-version": context.apiVersion, | ||
| "api%2Dversion": context.apiVersion, |
There was a problem hiding this comment.
Accepted change: #3145 and I verify api-version should work for mgmt SDK.
|
|
||
| import { NetworkAnalyticsApiContext as Client } from "../index.js"; | ||
| import { | ||
| errorResponseDeserializer, |
There was a problem hiding this comment.
@qiaozha Could we fix the import order? It seems the order is unstable and we need to regen serveral times to match with sequence in ci, the most bad case is we can't produce same order with ci verison.
MaryGao
changed the title
|
Merge this and if anything incorrect let us fix this in following pr. /@qiaozha |
3 participants
fixes #3145