fix: refs #8534 enhance stateQueryGuard to handle identical routes and improve test coverage

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();
+        });
     });
 });

src/router/hooks.js

@@ -84,7 +84,6 @@ function waitUntilFalse(ref) {
         const stop = watch(
             ref,
             (val) => {
-                console.log('val: ', val);
                 if (!val) {
                     stop();
                     resolve();

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"]');