From 3ec9aa574ba679dcc231a6373b6572062a513f37 Mon Sep 17 00:00:00 2001
From: Olusegun Akintayo <akintayo.segun@gmail.com>
Date: Fri, 4 Aug 2023 14:45:21 +0100
Subject: [PATCH] Add title for failed refactor code

---
 .../BlockaidBanner.constants.ts               |  5 +++-
 .../UI/BlockaidBanner/BlockaidBanner.tsx      | 28 ++++++++-----------
 locales/languages/en.json                     |  2 +-
 3 files changed, 16 insertions(+), 19 deletions(-)

diff --git a/app/components/UI/BlockaidBanner/BlockaidBanner.constants.ts b/app/components/UI/BlockaidBanner/BlockaidBanner.constants.ts
index d0f5c45df95..60823e49483 100644
--- a/app/components/UI/BlockaidBanner/BlockaidBanner.constants.ts
+++ b/app/components/UI/BlockaidBanner/BlockaidBanner.constants.ts
@@ -22,4 +22,7 @@ export const REASON_DESCRIPTION_I18N_KEY_MAP = Object.freeze({
   [Reason.unfairTrade]: 'blockaid_banner.unfair_trade_description',
 });
 
-export const SUSPICIOUS_TITLED_REQUESTS = [Reason.rawSignatureFarming];
+export const REASON_TITLE_I18N_KEY_MAP: Record<string, string> = Object.freeze({
+  [Reason.rawSignatureFarming]: 'blockaid_banner.suspicious_request_title',
+  [Reason.failed]: 'blockaid_banner.failed_title',
+});
diff --git a/app/components/UI/BlockaidBanner/BlockaidBanner.tsx b/app/components/UI/BlockaidBanner/BlockaidBanner.tsx
index 75dba4aede7..90b7d8a3cde 100644
--- a/app/components/UI/BlockaidBanner/BlockaidBanner.tsx
+++ b/app/components/UI/BlockaidBanner/BlockaidBanner.tsx
@@ -22,30 +22,23 @@ import AttributionLink from './AttributionLink';
 import {
   ATTRIBUTION_LINE_TEST_ID,
   REASON_DESCRIPTION_I18N_KEY_MAP,
-  SUSPICIOUS_TITLED_REQUESTS,
+  REASON_TITLE_I18N_KEY_MAP,
 } from './BlockaidBanner.constants';
 import styleSheet from './BlockaidBanner.styles';
 import { BlockaidBannerProps, FlagType, Reason } from './BlockaidBanner.types';
 
-const getTitle = (reason: Reason): string => {
-  if (SUSPICIOUS_TITLED_REQUESTS.indexOf(reason) >= 0) {
-    return strings('blockaid_banner.suspicious_request_title');
-  }
-  return strings('blockaid_banner.deceptive_request_title');
-};
+const getTitle = (reason: Reason): string =>
+  strings(
+    REASON_TITLE_I18N_KEY_MAP[reason] ||
+      'blockaid_banner.deceptive_request_title',
+  );
 
-const getTitleDescription = (
-  reason: Reason,
-): { title: string; description: string } => {
-  const title = getTitle(reason);
-  const description = strings(
+const getDescription = (reason: Reason) =>
+  strings(
     REASON_DESCRIPTION_I18N_KEY_MAP[reason] ||
       REASON_DESCRIPTION_I18N_KEY_MAP[Reason.other],
   );
 
-  return { title, description };
-};
-
 const BlockaidBanner = (bannerProps: BlockaidBannerProps) => {
   const { style, flagType, reason, features, onToggleShowDetails } =
     bannerProps;
@@ -56,12 +49,13 @@ const BlockaidBanner = (bannerProps: BlockaidBannerProps) => {
     return null;
   }
 
-  const { title, description } = getTitleDescription(reason);
-
   if (flagType === FlagType.Benign) {
     return null;
   }
 
+  const title = getTitle(reason);
+  const description = getDescription(reason);
+
   if (flagType === FlagType.Failed) {
     return (
       <BannerAlert
diff --git a/locales/languages/en.json b/locales/languages/en.json
index 1264ef11b3f..68b389eaf05 100644
--- a/locales/languages/en.json
+++ b/locales/languages/en.json
@@ -6,7 +6,7 @@
     "attribution_link_name": "Blockaid",
     "blur_farming_description": "If you approve this request, someone can steal your assets listed on Blur.",
     "deceptive_request_title": "This is a deceptive request",
-    "failed": "Request may not be safe",
+    "failed_title": "Request may not be safe",
     "failed_description": "Because of an error, this request was not verified by the security provider. Proceed with caution.",
     "malicious_domain_description": "You're interacting with a malicious domain. If you approve this request, you might lose your assets.",
     "other_description": "If you approve this request, you might lose your assets.",