Fix review comments

Signed-off-by: Artem Savchenko <[email protected]>
hcengineering · Dec 3, 2024 · b9a0a8e · b9a0a8e
1 parent 445af9a
commit b9a0a8e
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/plugins/test-management-resources/src/components/test-result/NextButton.svelte b/plugins/test-management-resources/src/components/test-result/NextButton.svelte
@@ -25,12 +25,11 @@
   export let object: WithLookup<TestResult> | undefined
   let isLoading = true
   let hasNext = false
-  let navigateToNext = false
 
   const client = getClient()
   const hierarchy = client.getHierarchy()
 
-  testIteratorStore.subscribe(() => {
+  const unsubscribe = testIteratorStore.subscribe(() => {
     hasNext = testResultIteratorProvider.getIterator()?.hasNext() ?? false
   })
 
@@ -41,13 +40,11 @@
     isLoading = false
   })
   onDestroy(() => {
-    if (navigateToNext) {
-      testResultIteratorProvider.reset()
-    }
+    testResultIteratorProvider.reset()
+    unsubscribe()
   })
 
   async function goToNextItem (): Promise<void> {
-    navigateToNext = true
     const iterator = testResultIteratorProvider.getIterator()
     if (iterator !== undefined) {
       const nextItem = iterator.next()