Fix test result navigation arrows behaviour (#44)

allure-framework · Dec 22, 2024 · 3e0745c · 3e0745c
1 parent 2d7d243
commit 3e0745c
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 7 deletions.
diff --git a/.husky/pre-commit b/.husky/pre-commit
@@ -1 +1 @@
-yarn test
+yarn lint-staged
diff --git a/package.json b/package.json
@@ -15,6 +15,7 @@
     "clean": "yarn workspaces foreach -Avvpt run clean",
     "eslint": "yarn workspaces foreach -Avvpt run eslint",
     "eslint:format": "yarn workspaces foreach -Avvpt run eslint:format",
+    "lint-staged": "lint-staged",
     "prepare": "husky",
     "prettier:format": "prettier --write './packages/*/{src,test,features}/**/*.{json,js,ts,tsx}' package.json ./packages/*/package.json",
     "test": "yarn allure run -- yarn workspaces foreach -Avvpt run test"

diff --git a/packages/web-awesome/src/components/app/TestResult/TestResultNavigation/index.tsx b/packages/web-awesome/src/components/app/TestResult/TestResultNavigation/index.tsx
@@ -54,9 +54,10 @@ export const TestResultNavigation: FunctionalComponent<TestResultNavigationProps
                   <IconButton
                     icon={LineArrowsChevronDown.id}
                     style={"ghost"}
-                    isDisabled={currentIndex <= 1}
+                    isDisabled={currentIndex === data.length}
                     data-testid="test-result-nav-prev"
-                    onClick={() => navigateTo(data[currentIndex - 2])}
+                    className={styles["test-result-nav-prev"]}
+                    onClick={() => navigateTo(data[currentIndex])}
                   />
                 </TooltipWrapper>
                 <Code
@@ -70,10 +71,9 @@ export const TestResultNavigation: FunctionalComponent<TestResultNavigationProps
                   <IconButton
                     icon={LineArrowsChevronDown.id}
                     style={"ghost"}
-                    isDisabled={currentIndex === data.length}
+                    isDisabled={currentIndex <= 1}
                     data-testid="test-result-nav-next"
-                    onClick={() => navigateTo(data[currentIndex])}
-                    className={styles["test-result-nav-next"]}
+                    onClick={() => navigateTo(data[currentIndex - 2])}
                   />
                 </TooltipWrapper>
               </div>

diff --git a/packages/web-awesome/src/components/app/TestResult/TestResultNavigation/styles.scss b/packages/web-awesome/src/components/app/TestResult/TestResultNavigation/styles.scss
@@ -40,7 +40,7 @@
   font-weight: 400;
 }
 
-.test-result-nav-next {
+.test-result-nav-prev {
   transform: rotate(-180deg);
   &:active {
     transform: scale(var(--allure-btn-pressed-scale)) rotate(-180deg);