From b267b756a7682d8a584c9e225197d36bb248ee42 Mon Sep 17 00:00:00 2001 From: Antoine Beauvais-Lacasse Date: Thu, 21 Sep 2017 00:58:09 -0400 Subject: [PATCH] @google-cloud/datastore: Remove unnecessary generics. --- .../google-cloud__datastore-tests.ts | 30 ++++++++++--------- types/google-cloud__datastore/index.d.ts | 24 +++++++-------- 2 files changed, 28 insertions(+), 26 deletions(-) diff --git a/types/google-cloud__datastore/google-cloud__datastore-tests.ts b/types/google-cloud__datastore/google-cloud__datastore-tests.ts index 6715415430..285cb1fb00 100644 --- a/types/google-cloud__datastore/google-cloud__datastore-tests.ts +++ b/types/google-cloud__datastore/google-cloud__datastore-tests.ts @@ -68,39 +68,41 @@ const complexQuery = ds.createQuery('special_namespace', kind) .offset(10); // Running queries: -const queryCallback: QueryCallback = (err: Error, entities: TestEntity[]) => entities[0][Datastore.KEY]; +const queryCallback: QueryCallback = (err: Error, entities: TestEntity[]) => entities[0][Datastore.KEY]; -ds.runQuery(query, queryCallback); -ds.runQuery(query, options, queryCallback); +ds.runQuery(query, queryCallback); +ds.runQuery(query, options, queryCallback); ds.runQuery(query, options); const queryStream: NodeJS.ReadableStream = complexQuery.runStream(); const dsQueryStream: NodeJS.ReadableStream = ds.runQueryStream(complexQuery, options); complexQuery.run() - .then((data: QueryResult) => { + .then((data: QueryResult) => { const {moreResults, endCursor} = data[1]; const frontEndResponse: any = {}; - if (moreResults === Datastore.NO_MORE_RESULTS) { + switch (moreResults) { + case Datastore.NO_MORE_RESULTS: frontEndResponse.nextPageCursor = null; - } else if (moreResults === Datastore.MORE_RESULTS_AFTER_CURSOR) { - frontEndResponse.nextPageCursor = endCursor; - } else if (moreResults === Datastore.MORE_RESULTS_AFTER_LIMIT) { + break; + case Datastore.MORE_RESULTS_AFTER_CURSOR: + case Datastore.MORE_RESULTS_AFTER_LIMIT: frontEndResponse.nextPageCursor = endCursor; } }); -query.run((err: Error, entities: TestEntity[], info: QueryInfo) => { +query.run((err: Error, entities: TestEntity[], info: QueryInfo) => { if (err) { return; } const {moreResults, endCursor} = info; const frontEndResponse: any = {entities}; - if (moreResults === ds.NO_MORE_RESULTS) { + switch (moreResults) { + case ds.NO_MORE_RESULTS: frontEndResponse.nextPageCursor = null; - } else if (moreResults === ds.MORE_RESULTS_AFTER_CURSOR) { - frontEndResponse.nextPageCursor = endCursor; - } else if (moreResults === ds.MORE_RESULTS_AFTER_LIMIT) { + break; + case ds.MORE_RESULTS_AFTER_CURSOR: + case ds.MORE_RESULTS_AFTER_LIMIT: frontEndResponse.nextPageCursor = endCursor; } }); @@ -166,7 +168,7 @@ transaction.run((err, activeTx: DatastoreTransaction) => { }); }); -let promisedTxStart: Promise = ds.transaction().run(); +const promisedTxStart: Promise = ds.transaction().run(); promisedTxStart.then((result: TransactionResult) => { const activeTx: DatastoreTransaction = result[0]; diff --git a/types/google-cloud__datastore/index.d.ts b/types/google-cloud__datastore/index.d.ts index 540969b6da..d2fd4cf947 100644 --- a/types/google-cloud__datastore/index.d.ts +++ b/types/google-cloud__datastore/index.d.ts @@ -165,9 +165,9 @@ declare module '@google-cloud/datastore/query' { offset(n: number): this; - run(callback: QueryCallback): void; - run(options: QueryOptions, callback: QueryCallback): void; - run(options?: QueryOptions): Promise>; + run(callback: QueryCallback): void; + run(options: QueryOptions, callback: QueryCallback): void; + run(options?: QueryOptions): Promise; runStream(): NodeJS.ReadableStream; } @@ -188,8 +188,8 @@ declare module '@google-cloud/datastore/query' { readonly moreResults: MoreResultsAfterCursor | MoreResultsAfterLimit | NoMoreResults; } - type QueryCallback = (err: Error, entities: T[], info: QueryInfo) => void; - type QueryResult = [T[], QueryInfo]; + type QueryCallback = (err: Error, entities: object[], info: QueryInfo) => void; + type QueryResult = [object[], QueryInfo]; } declare module '@google-cloud/datastore/request' { @@ -210,17 +210,17 @@ declare module '@google-cloud/datastore/request' { delete(keyOrKeys: DatastoreKey | ReadonlyArray, callback: CommitCallback): void; delete(keyOrKeys: DatastoreKey | ReadonlyArray): Promise | void; - get(key: DatastoreKey, options: QueryOptions, callback: GetCallback): void; - get(keys: ReadonlyArray, options: QueryOptions, callback: GetCallback): void; - get(key: DatastoreKey, callback: GetCallback): void; - get(keys: ReadonlyArray, callback: GetCallback): void; + get(key: DatastoreKey, options: QueryOptions, callback: GetCallback): void; + get(keys: ReadonlyArray, options: QueryOptions, callback: GetCallback): void; + get(key: DatastoreKey, callback: GetCallback): void; + get(keys: ReadonlyArray, callback: GetCallback): void; get(key: DatastoreKey, options?: QueryOptions): Promise<[object | undefined]>; get(keys: ReadonlyArray, options?: QueryOptions): Promise<[object[]]>; - runQuery(query: Query, options: QueryOptions, callback: QueryCallback): void; - runQuery(query: Query, callback: QueryCallback): void; - runQuery(query: Query, options?: QueryOptions): QueryResult; + runQuery(query: Query, options: QueryOptions, callback: QueryCallback): void; + runQuery(query: Query, callback: QueryCallback): void; + runQuery(query: Query, options?: QueryOptions): QueryResult; runQueryStream(query: Query, options?: QueryOptions): NodeJS.ReadableStream;