Added eslint key-spacing rule (#204)

FamilySearch · Mar 15, 2024 · 00bfbdb · 00bfbdb
1 parent 898b234
commit 00bfbdb
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 6 deletions.
diff --git a/.eslintrc b/.eslintrc
@@ -63,6 +63,7 @@
     "space-infix-ops":"warn",
     "space-before-blocks": "warn",
     "keyword-spacing": "warn",
+    "key-spacing": 1,
     "strict": 1,
     "comma-dangle": 1,
     "triple-equals": 0,

diff --git a/agent/test/pewpewtest.spec.ts b/agent/test/pewpewtest.spec.ts
@@ -190,7 +190,7 @@ describe("PewPewTest", () => {
       endTime: now + 2,
       resultsFilename: ["test2"],
       status: TestStatus.Failed,
-      version:"version3",
+      version: "version3",
       userId: "user3"
     };
 

diff --git a/controller/components/StartTestForm/index.tsx b/controller/components/StartTestForm/index.tsx
@@ -174,7 +174,7 @@ export const StartTestForm = ({
     allDays: defaultDaysOfWeek.every((dayOfWeek: DayValue) => dayOfWeek.value),
     queueName: previousTestData?.queueName || queueInitialProps.queueName,
     pewpewVersion: previousTestData?.version || (recurringTest ? maxPewPewVersion : versionInitalProps.pewpewVersion),
-    environmentVariables:  previousTestData?.environmentVariables
+    environmentVariables: previousTestData?.environmentVariables
       ? Object.entries(previousTestData.environmentVariables).map(([variableName, variableValue]: [string, string | null], index: number) => ({
         // Map these to the placeholder values
         name: "previousVar" + index, // We can't use the date or we'll get constant redraws, use the index

diff --git a/controller/components/TestInfo/index.tsx b/controller/components/TestInfo/index.tsx
@@ -54,7 +54,7 @@ export const TestInfo = ({ testData, ...testInfoProps }: TestInfoProps) => {
   const defaultState: TestInfoState = {
     message: testInfoProps.message || undefined,
     messageId: testInfoProps.messageId || undefined,
-    killTest : testInfoProps.killTest || undefined,
+    killTest: testInfoProps.killTest || undefined,
     error: testInfoProps.error || undefined
   };
   const [state, setState] = useState(defaultState);

diff --git a/guide/results-viewer-react/.eslintrc b/guide/results-viewer-react/.eslintrc
@@ -50,6 +50,7 @@
     "space-infix-ops":"warn",
     "space-before-blocks": "warn",
     "keyword-spacing": "warn",
+    "key-spacing": 1,
     "strict": 1,
     "comma-dangle": 1,
     "triple-equals": 0,

diff --git a/guide/results-viewer-react/src/components/Modal/story.tsx b/guide/results-viewer-react/src/components/Modal/story.tsx
@@ -63,7 +63,7 @@ const ModalUploadHarDemo = () => {
          <div>
            {files.map((file, index) => (
              <div style={{paddingTop: "13px"}} key={index}>
-               <button name={`remove${index}`} style={{marginRight: "5px", marginTop:"40px"}} onClick={() => onRemoveFile(index)}>X</button>
+               <button name={`remove${index}`} style={{marginRight: "5px", marginTop: "40px"}} onClick={() => onRemoveFile(index)}>X</button>
                {file.name}
              </div>
            ))}

diff --git a/guide/results-viewer-react/src/components/YamlUrls/story.tsx b/guide/results-viewer-react/src/components/YamlUrls/story.tsx
@@ -45,9 +45,9 @@ const propsLoaded: UrlProps = {
     url: "https://www.pewpew.org",
     hitRate: "1hpm",
     headers: [
-      {id: "0", name : "Default Header", value: "Here is where default headers are. These next 3 are the only default. Authorization header only shows up when Authenticated button is selected"},
+      {id: "0", name: "Default Header", value: "Here is where default headers are. These next 3 are the only default. Authorization header only shows up when Authenticated button is selected"},
       ...getDefaultHeaders(true),
-      {id: "1", name : "Har Header", value: "Here is where har headers would be"}
+      {id: "1", name: "Har Header", value: "Here is where har headers would be"}
     ],
     method: "GET", authorization: null
   },