test: made unit tests getFeedSubmissionResult and modify parseResponse

src/http.ts

@@ -247,19 +247,11 @@ const defaultFetch = ({ url, method, headers, data }: Request): Promise<RequestR
       return Promise.reject(error.response.data)
     })
 
-const parseGetReport = (response: RequestResponse): [string, RequestMeta] => [
-  response.data,
-  {
-    requestId: response.headers['x-mws-request-id'],
-    timestamp: new Date(response.headers['x-mws-timestamp']),
-    quotaMax: Number(response.headers['x-mws-quota-max']),
-    quotaRemaining: Number(response.headers['x-mws-quota-remaining']),
-    quotaResetOn: new Date(response.headers['x-mws-quota-resetson']),
-  },
-]
-
-const parseResponse = <T>(response: RequestResponse): [T, RequestMeta] => {
-  const responseData = parser.parse(response.data)
+const parseResponse = <T>(
+  response: RequestResponse,
+  parseString = false,
+): [T | string, RequestMeta] => {
+  const responseData = parseString ? response.data : parser.parse(response.data)
   return [
     responseData,
     {
@@ -334,10 +326,15 @@ export class HttpClient {
         throw new InvalidUPCIdentifier(`${info.action} request failed`)
       }
 
-      // GetReport returns a string that should be treated as a file instead of XML data
-      // http://docs.developer.amazonservices.com/en_CA/reports/Reports_GetReport.html
-      if (info.action === 'GetReport') {
-        return parseGetReport(response)
+      /**
+       * GetReport and GetFeedSubmissionResult returns a string that should be treated as a file instead of XML data
+       * Get Report
+       * http://docs.developer.amazonservices.com/en_CA/reports/Reports_GetReport.html
+       * GetFeedSubmissionResult
+       * http://docs.developer.amazonservices.com/en_CA/feeds/Feeds_GetFeedSubmissionResult.html
+       */
+      if (info.action === 'GetReport' || info.action === 'GetFeedSubmissionResult') {
+        return parseResponse(response, true)
       }
 
       return parseResponse(response)

test/unit/feeds.test.ts

@@ -3,6 +3,24 @@ import { NextToken } from '../../src/parsing'
 import { createMockHttpClient, mockMwsFail, parsingError } from '../utils'
 
 describe('feeds', () => {
+  describe('getFeedSubmissionResult', () => {
+    it('returns an XML file when succesful', async () => {
+      expect.assertions(1)
+
+      const mockGetFeedSubmissionResult = createMockHttpClient('feeds_get_feed_submission_result')
+
+      expect(await mockGetFeedSubmissionResult.feeds.getFeedSubmissionResult()).toMatchSnapshot()
+    })
+
+    it('throws a parsing error when the response isnt valid', async () => {
+      expect.assertions(1)
+
+      await expect(() => mockMwsFail.feeds.getFeedSubmissionResult()).rejects.toStrictEqual(
+        new ParsingError(parsingError),
+      )
+    })
+  })
+
   describe('cancelFeedSubmissions', () => {
     it('returns count and submission info of canceled feed submissions', async () => {
       expect.assertions(1)