Skip to content

Commit e67efd7

Browse files
authored
[Attestation] Migrate to core v2 (Azure#15504)
* [Attestation] Migrate to core v2 * revert swagger/README.md * fix test failure
1 parent 38dd317 commit e67efd7

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

48 files changed

+1806
-1950
lines changed

sdk/attestation/attestation/karma.conf.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ module.exports = function(config) {
6868
"ATTESTATION_ISOLATED_SIGNING_KEY",
6969
"AZURE_CLIENT_ID",
7070
"AZURE_CLIENT_SECRET",
71-
"AZURE_TENANT_ID",
71+
"AZURE_TENANT_ID"
7272
],
7373

7474
// test results reporter to use

sdk/attestation/attestation/package.json

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,9 @@
55
"description": "Describes the interface for the per-tenant enclave service.",
66
"version": "1.0.0-beta.3",
77
"dependencies": {
8-
"@azure/core-http": "^1.2.0",
8+
"@azure/core-auth": "^1.3.0",
9+
"@azure/core-client": "^1.0.0",
10+
"@azure/core-rest-pipeline": "^1.0.3",
911
"@azure/core-tracing": "1.0.0-preview.11",
1012
"@azure/logger": "^1.0.0",
1113
"tslib": "^2.0.0",
@@ -89,7 +91,7 @@
8991
"clean": "rimraf dist dist-esm dist-test temp types *.tgz *.log",
9092
"execute:samples": "echo skipped",
9193
"extract-api": "api-extractor run --local",
92-
"format": "echo skip format as this package has only auto generated code",
94+
"format": "prettier --write --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"samples-dev/**/*.ts\" \"*.{js,json}\"",
9395
"integration-test:browser": "karma start --single-run",
9496
"integration-test:node": "nyc mocha -r esm --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 5000000 --full-trace \"dist-esm/test/{,!(browser)/**/}*.spec.js\"",
9597
"integration-test": "npm run integration-test:node && npm run integration-test:browser",

sdk/attestation/attestation/review/attestation.api.md

Lines changed: 17 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,11 @@
44
55
```ts
66

7-
import * as coreHttp from '@azure/core-http';
8-
import { OperationOptions } from '@azure/core-http';
9-
import { PipelineOptions } from '@azure/core-http';
10-
import { TokenCredential } from '@azure/core-http';
7+
import { CommonClientOptions } from '@azure/core-client';
8+
import * as coreAuth from '@azure/core-auth';
9+
import * as coreClient from '@azure/core-client';
10+
import { OperationOptions } from '@azure/core-client';
11+
import { TokenCredential } from '@azure/core-auth';
1112

1213
// @public
1314
export class AttestationAdministrationClient {
@@ -24,16 +25,11 @@ export interface AttestationAdministrationClientOperationOptions extends Operati
2425
}
2526

2627
// @public
27-
export interface AttestationAdministrationClientOptions extends PipelineOptions {
28+
export interface AttestationAdministrationClientOptions extends CommonClientOptions {
2829
}
2930

3031
// @public
31-
export type AttestationAttestTpmResponse = TpmAttestationResponse & {
32-
_response: coreHttp.HttpResponse & {
33-
bodyAsText: string;
34-
parsedBody: TpmAttestationResponse;
35-
};
36-
};
32+
export type AttestationAttestTpmResponse = TpmAttestationResponse;
3733

3834
// @public
3935
export interface AttestationCertificateManagementBody {
@@ -61,7 +57,7 @@ export interface AttestationClientOperationOptions extends OperationOptions {
6157
}
6258

6359
// @public
64-
export interface AttestationClientOptions extends PipelineOptions {
60+
export interface AttestationClientOptions extends CommonClientOptions {
6561
}
6662

6763
// @public
@@ -204,7 +200,7 @@ export type DataType = string;
204200
// @public (undocumented)
205201
export class GeneratedClient extends GeneratedClientContext {
206202
// Warning: (ae-forgotten-export) The symbol "GeneratedClientOptionalParams" needs to be exported by the entry point index.d.ts
207-
constructor(credentials: coreHttp.TokenCredential | coreHttp.ServiceClientCredentials, instanceUrl: string, options?: GeneratedClientOptionalParams);
203+
constructor(credentials: coreAuth.TokenCredential, instanceUrl: string, options?: GeneratedClientOptionalParams);
208204
// Warning: (ae-forgotten-export) The symbol "Attestation" needs to be exported by the entry point index.d.ts
209205
//
210206
// (undocumented)
@@ -228,8 +224,8 @@ export class GeneratedClient extends GeneratedClientContext {
228224
}
229225

230226
// @public (undocumented)
231-
export class GeneratedClientContext extends coreHttp.ServiceClient {
232-
constructor(credentials: coreHttp.TokenCredential | coreHttp.ServiceClientCredentials, instanceUrl: string, options?: GeneratedClientOptionalParams);
227+
export class GeneratedClientContext extends coreClient.ServiceClient {
228+
constructor(credentials: coreAuth.TokenCredential, instanceUrl: string, options?: GeneratedClientOptionalParams);
233229
// (undocumented)
234230
apiVersion: string;
235231
// (undocumented)
@@ -294,26 +290,16 @@ export enum KnownPolicyModification {
294290
// @public
295291
export class PolicyCertificates {
296292
constructor(client: AttestationClient);
297-
add(policyCertificateToAdd: string, options?: coreHttp.OperationOptions): Promise<PolicyCertificatesAddResponse>;
298-
get(options?: coreHttp.OperationOptions): Promise<PolicyCertificatesGetResponse>;
299-
remove(policyCertificateToRemove: string, options?: coreHttp.OperationOptions): Promise<PolicyCertificatesRemoveResponse>;
293+
add(policyCertificateToAdd: string, options?: coreClient.OperationOptions): Promise<PolicyCertificatesAddResponse>;
294+
get(options?: coreClient.OperationOptions): Promise<PolicyCertificatesGetResponse>;
295+
remove(policyCertificateToRemove: string, options?: coreClient.OperationOptions): Promise<PolicyCertificatesRemoveResponse>;
300296
}
301297

302298
// @public
303-
export type PolicyCertificatesAddResponse = PolicyCertificatesModifyResponse & {
304-
_response: coreHttp.HttpResponse & {
305-
bodyAsText: string;
306-
parsedBody: PolicyCertificatesModifyResponse;
307-
};
308-
};
299+
export type PolicyCertificatesAddResponse = PolicyCertificatesModifyResponse;
309300

310301
// @public
311-
export type PolicyCertificatesGetResponse = PolicyCertificatesResponse & {
312-
_response: coreHttp.HttpResponse & {
313-
bodyAsText: string;
314-
parsedBody: PolicyCertificatesResponse;
315-
};
316-
};
302+
export type PolicyCertificatesGetResponse = PolicyCertificatesResponse;
317303

318304
// @public
319305
export interface PolicyCertificatesModificationResult {
@@ -327,12 +313,7 @@ export interface PolicyCertificatesModifyResponse {
327313
}
328314

329315
// @public
330-
export type PolicyCertificatesRemoveResponse = PolicyCertificatesModifyResponse & {
331-
_response: coreHttp.HttpResponse & {
332-
bodyAsText: string;
333-
parsedBody: PolicyCertificatesModifyResponse;
334-
};
335-
};
316+
export type PolicyCertificatesRemoveResponse = PolicyCertificatesModifyResponse;
336317

337318
// @public
338319
export interface PolicyCertificatesResponse {

0 commit comments

Comments
 (0)