Skip to content

Commit

Permalink
fix(types): fix signatures of listing methods (#703)
Browse files Browse the repository at this point in the history
  • Loading branch information
AVaksman authored and JustinBeckwith committed May 7, 2019
1 parent ada995e commit 42937a8
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 4 deletions.
5 changes: 3 additions & 2 deletions src/bucket.ts
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ interface BucketOptions {
userProject?: string;
}

export type GetFilesResponse = [File[], {}, Metadata];
export interface GetFilesCallback {
(
err: Error | null,
Expand Down Expand Up @@ -1899,7 +1900,7 @@ class Bucket extends ServiceObject {
return new File(this, name, options);
}

getFiles(query?: GetFilesOptions): Promise<[File[]]>;
getFiles(query?: GetFilesOptions): Promise<GetFilesResponse>;
getFiles(query: GetFilesOptions, callback: GetFilesCallback): void;
getFiles(callback: GetFilesCallback): void;
/**
Expand Down Expand Up @@ -2007,7 +2008,7 @@ class Bucket extends ServiceObject {
getFiles(
queryOrCallback?: GetFilesOptions | GetFilesCallback,
callback?: GetFilesCallback
): void | Promise<[File[]]> {
): void | Promise<GetFilesResponse> {
let query = typeof queryOrCallback === 'object' ? queryOrCallback : {};
if (!callback) {
callback = queryOrCallback as GetFilesCallback;
Expand Down
1 change: 1 addition & 0 deletions src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,7 @@ export {
GetBucketResponse,
GetFilesCallback,
GetFilesOptions,
GetFilesResponse,
GetLabelsCallback,
GetLabelsOptions,
GetLabelsResponse,
Expand Down
9 changes: 7 additions & 2 deletions src/storage.ts
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,14 @@ export interface BucketCallback {
(err: Error | null, bucket?: Bucket | null, apiResponse?: Metadata): void;
}

export type GetBucketsResponse = [Bucket[]];
export type GetBucketsResponse = [Bucket[], {}, Metadata];
export interface GetBucketsCallback {
(err: Error | null, buckets: Bucket[]): void;
(
err: Error | null,
buckets: Bucket[],
nextQuery?: {},
apiResponse?: Metadata
): void;
}
export interface GetBucketsRequest {
prefix?: string;
Expand Down

0 comments on commit 42937a8

Please sign in to comment.