fix(imgur): resolves errors with the returning success of the deleteT…

…ag method for the imgur adapter

src/client.ts

@@ -144,6 +144,12 @@ export class BikeTagClient extends EventEmitter {
           // Only exclude PUT, PATCH and DELETE methods from cache
           methods: ['put', 'patch', 'delete'],
         },
+        // Attempt reading stale cache data when response status is either 4xx or 5xx
+        readOnError: (error) => {
+          return error.response.status >= 400 && error.response.status < 600
+        },
+        // Deactivate `clearOnStale` option so that we can actually read stale cache data
+        clearOnStale: false,
       },
       headers,
       responseType,

src/common/payloads.ts

@@ -1,5 +1,5 @@
 import { Tag } from './schema'
-import { CommonData } from './types'
+import { CommonPayloadData } from './types'
 
 export type SortOptions = 'new' | 'relevance' | 'hot' | 'top' | 'comments'
 
@@ -13,13 +13,13 @@ export type getTagsPayload = {
   sort?: SortOptions
   limit?: number
   tagnumbers: number[]
-} & CommonData
+} & CommonPayloadData
 
 export type deleteTagsPayload = {
   slugs?: string[]
   tagnumbers: number[]
   tags?: Partial<Tag>[]
-} & CommonData
+} & CommonPayloadData
 
 export type getTagPayload = {
   tagnumber: number
@@ -31,7 +31,7 @@ export type getTagPayload = {
   account?: string
   slug: string
   fields?: string[]
-} & CommonData
+} & CommonPayloadData
 
 export type updateTagPayload = {
   hash?: string
@@ -52,7 +52,7 @@ export type uploadTagImagePayload = {
   type: 'found' | 'mystery'
   slug?: string
   stream: ReadableStream
-} & CommonData
+} & CommonPayloadData
 
 export type ImgurUploadPayload = {
   image: string
@@ -68,7 +68,7 @@ export type getGamePayload = {
   name: string
   hash?: string
   fields?: string[]
-} & CommonData
+} & CommonPayloadData
 
 export type getPlayersPayload = {
   fields?: string[]
@@ -78,49 +78,49 @@ export type getPlayersPayload = {
   limit?: number
   account?: string
   subreddit?: string
-} & CommonData
+} & CommonPayloadData
 
 export type getPlayerPayload = {
   fields?: string[]
   slug?: string
   hash?: string
   account?: string
   subreddit?: string
-} & CommonData
+} & CommonPayloadData
 
 export type getSettingPayload = {
   fields?: string[]
   slug?: string
-} & CommonData
+} & CommonPayloadData
 
 export type getSettingsPayload = {
   fields?: string[]
   sort?: SortOptions
   limit?: number
   slugs?: string[]
-} & CommonData
+} & CommonPayloadData
 
 export type getAmbassadorsPayload = {
   fields?: string[]
   limit?: number
   sort?: SortOptions
   slugs?: string[]
-} & CommonData
+} & CommonPayloadData
 export type getAmbassadorPayload = {
   fields?: string[]
   slug?: string
-} & CommonData
+} & CommonPayloadData
 
 export type importTagPayload = Pick<
   Tag,
   'discussionUrl' | 'mysteryImageUrl' | 'foundImageUrl' | 'mentionUrl'
 > &
-  CommonData
+  CommonPayloadData
 
 export type getQueuePayload = {
   hash?: string
   queuehash?: string
-} & CommonData
+} & CommonPayloadData
 
 export type queueTagPayload = {
   queuehash?: string

src/common/types.ts

@@ -79,6 +79,8 @@ export interface CommonData {
   concise?: boolean
   cached?: boolean
 }
+
+export type CommonPayloadData = CommonData
 export interface AccessToken {
   accessToken: string
 }

src/imgur/deleteTag.ts

@@ -12,6 +12,7 @@ export async function deleteTag(
   const hashes = []
   const hasImageUrls = payload.foundImageUrl || payload.mysteryImageUrl
   let tag = payload
+
   if (!hasImageUrls && (payload.tagnumber || payload.slug)) {
     tag = await this.getTags(payload.tagnumber ?? payload.slug)
   }
@@ -37,7 +38,9 @@ export async function deleteTag(
 
   return {
     data: responses,
-    success: true,
+    success: responses.reduce((o, r) => {
+      return o && (typeof r === 'boolean') && !!r
+    }, true),
     source: AvailableApis[AvailableApis.imgur],
     status: HttpStatusCode.Ok,
   }