Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

Commit

Permalink
fix: fixes for integration tests and remove void from return
Browse files Browse the repository at this point in the history
  • Loading branch information
justinemmanuelmercado committed Aug 10, 2020
1 parent 3c703da commit 044c161
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/sections/feeds.ts
Expand Up @@ -227,7 +227,7 @@ export class Feeds {

async getFeedSubmissionResult(
parameters: GetFeedSubmissionResultParameters,
): Promise<[FeedSubmission | Record<string, unknown>, RequestMeta] | void> {
): Promise<[FeedSubmission | Record<string, unknown>, RequestMeta]> {
const stringResponse = parameters.format === 'xml' || !parameters.format

const [response, meta] = await this.httpClient.request(
Expand Down
4 changes: 2 additions & 2 deletions test/integration/feeds.test.ts
Expand Up @@ -13,7 +13,7 @@ describe(`${Feeds.name}`, () => {

const parameters = {
FeedSubmissionId: '51793018437',
format: 'xml',
format: 'xml' as 'xml',
}

const [response] = await feeds.getFeedSubmissionResult(parameters)
Expand All @@ -28,7 +28,7 @@ describe(`${Feeds.name}`, () => {

const parameters = {
FeedSubmissionId: '51793018437',
format: 'json',
format: 'json' as 'json',
}

const [response] = await feeds.getFeedSubmissionResult(parameters)
Expand Down

0 comments on commit 044c161

Please sign in to comment.