From 9d53418e21178e23e011bd5bde5b5cd951f5fc00 Mon Sep 17 00:00:00 2001
From: alexm <alexm@verdnatura.es>
Date: Thu, 27 Mar 2025 07:53:17 +0100
Subject: [PATCH] fix: refs #8534 enhance stateQueryGuard to handle identical
 routes and improve test coverage

---
 src/router/__tests__/hooks.spec.js                       | 9 ++++++++-
 src/router/hooks.js                                      | 1 -
 .../integration/entry/entryCard/entryDescriptor.spec.js  | 4 ----
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/router/__tests__/hooks.spec.js b/src/router/__tests__/hooks.spec.js
index 7fa4161631f..1bc3e2e0b27 100644
--- a/src/router/__tests__/hooks.spec.js
+++ b/src/router/__tests__/hooks.spec.js
@@ -17,10 +17,11 @@ vi.mock('src/stores/useStateQueryStore', () => {
 
 describe('hooks', () => {
     describe('stateQueryGuard', () => {
+        const foo = { name: 'foo' };
         it('should wait until the state query is not loading and then call next()', async () => {
             const next = vi.fn();
 
-            const guardPromise = stateQueryGuard(next);
+            const guardPromise = stateQueryGuard(foo, { name: 'bar' }, next);
             expect(next).not.toHaveBeenCalled();
 
             testStateQuery.isLoading.value = false;
@@ -28,5 +29,11 @@ describe('hooks', () => {
             await guardPromise;
             expect(next).toHaveBeenCalled();
         });
+
+        it('should ignore if both routes are the same', async () => {
+            const next = vi.fn();
+            stateQueryGuard(foo, foo, next);
+            expect(next).toHaveBeenCalled();
+        });
     });
 });
diff --git a/src/router/hooks.js b/src/router/hooks.js
index add773e7fcf..bd9e5334fd3 100644
--- a/src/router/hooks.js
+++ b/src/router/hooks.js
@@ -84,7 +84,6 @@ function waitUntilFalse(ref) {
         const stop = watch(
             ref,
             (val) => {
-                console.log('val: ', val);
                 if (!val) {
                     stop();
                     resolve();
diff --git a/test/cypress/integration/entry/entryCard/entryDescriptor.spec.js b/test/cypress/integration/entry/entryCard/entryDescriptor.spec.js
index 55447100831..8185866db99 100644
--- a/test/cypress/integration/entry/entryCard/entryDescriptor.spec.js
+++ b/test/cypress/integration/entry/entryCard/entryDescriptor.spec.js
@@ -28,12 +28,8 @@ describe('EntryDescriptor', () => {
                     cy.get('.q-notification__message')
                         .eq(2)
                         .should('have.text', 'Entry prices recalculated');
-
-                    cy.get('[data-cy="descriptor-more-opts"]').click();
                     cy.deleteEntry();
 
-                    cy.log(previousUrl);
-
                     cy.visit(previousUrl);
 
                     cy.waitForElement('[data-cy="entry-buys"]');