From: Philippe Perrin <philippe.perrin@sonarsource.com>
Date: Thu, 9 Jan 2020 08:10:41 +0000 (+0100)
Subject: Code smells fixes
X-Git-Tag: 8.2.0.32929~137
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=e174fc4437e276857bd7b820f905515bc9ce29f1;p=sonarqube.git

Code smells fixes
---

diff --git a/server/sonar-web/src/main/js/apps/securityHotspots/components/HotspotViewerReviewHistoryTab.tsx b/server/sonar-web/src/main/js/apps/securityHotspots/components/HotspotViewerReviewHistoryTab.tsx
index 7796b5e41d0..07989ed42f7 100644
--- a/server/sonar-web/src/main/js/apps/securityHotspots/components/HotspotViewerReviewHistoryTab.tsx
+++ b/server/sonar-web/src/main/js/apps/securityHotspots/components/HotspotViewerReviewHistoryTab.tsx
@@ -38,9 +38,9 @@ export default function HotspotViewerReviewHistoryTab(props: HotspotViewerReview
 
   return (
     <div className="padded">
-      {history.map((elt, i) => (
-        <React.Fragment key={i}>
-          {i > 0 && <hr />}
+      {history.map((elt, historyIndex) => (
+        <React.Fragment key={historyIndex}>
+          {historyIndex > 0 && <hr />}
           <div className="padded">
             <div className="display-flex-center">
               {elt.user.name && (
@@ -74,8 +74,8 @@ export default function HotspotViewerReviewHistoryTab(props: HotspotViewerReview
 
             {elt.type === ReviewHistoryType.Diff && elt.diffs && (
               <div className="spacer-top">
-                {elt.diffs.map((diff, i) => (
-                  <IssueChangelogDiff diff={diff} key={i} />
+                {elt.diffs.map((diff, diffIndex) => (
+                  <IssueChangelogDiff diff={diff} key={diffIndex} />
                 ))}
               </div>
             )}
diff --git a/server/sonar-web/src/main/js/apps/securityHotspots/components/__tests__/FilterBar-test.tsx b/server/sonar-web/src/main/js/apps/securityHotspots/components/__tests__/FilterBar-test.tsx
index 8a7a32c1877..ad65adf0343 100644
--- a/server/sonar-web/src/main/js/apps/securityHotspots/components/__tests__/FilterBar-test.tsx
+++ b/server/sonar-web/src/main/js/apps/securityHotspots/components/__tests__/FilterBar-test.tsx
@@ -54,10 +54,11 @@ it('should trigger onChange for status', () => {
     .props();
 
   if (!onChange) {
-    return fail("Select's onChange should be defined");
+    fail("Select's onChange should be defined");
+  } else {
+    onChange({ value: HotspotStatusFilter.SAFE });
+    expect(onChangeFilters).toBeCalledWith({ status: HotspotStatusFilter.SAFE });
   }
-  onChange({ value: HotspotStatusFilter.SAFE });
-  expect(onChangeFilters).toBeCalledWith({ status: HotspotStatusFilter.SAFE });
 });
 
 it('should trigger onChange for self-assigned toggle', () => {
@@ -67,10 +68,11 @@ it('should trigger onChange for self-assigned toggle', () => {
   const { onCheck } = wrapper.find(RadioToggle).props();
 
   if (!onCheck) {
-    return fail("RadioToggle's onCheck should be defined");
+    fail("RadioToggle's onCheck should be defined");
+  } else {
+    onCheck(AssigneeFilterOption.ALL);
+    expect(onChangeFilters).toBeCalledWith({ assignedToMe: false });
   }
-  onCheck(AssigneeFilterOption.ALL);
-  expect(onChangeFilters).toBeCalledWith({ assignedToMe: false });
 });
 
 it('should trigger onChange for leak period', () => {
@@ -83,10 +85,11 @@ it('should trigger onChange for leak period', () => {
     .props();
 
   if (!onChange) {
-    return fail("Select's onChange should be defined");
+    fail("Select's onChange should be defined");
+  } else {
+    onChange({ value: true });
+    expect(onChangeFilters).toBeCalledWith({ sinceLeakPeriod: true });
   }
-  onChange({ value: true });
-  expect(onChangeFilters).toBeCalledWith({ sinceLeakPeriod: true });
 });
 
 function shallowRender(props: Partial<FilterBarProps> = {}) {