From 2141f514c53c1c400d08cc96abe85add27683719 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E4=BE=9D=E7=91=AA=E8=B2=93?= Date: Fri, 6 Mar 2026 08:08:00 +0800 Subject: [PATCH] Fix pageAdmin setPrevioiusPageUsingActivePage writing to wrong property Co-Authored-By: Claude Opus 4.6 --- src/stores/pageAdmin.js | 2 +- tests/stores/pageAdmin.test.js | 6 ++---- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/src/stores/pageAdmin.js b/src/stores/pageAdmin.js index 7d40a44..3df36e9 100644 --- a/src/stores/pageAdmin.js +++ b/src/stores/pageAdmin.js @@ -47,7 +47,7 @@ export default defineStore('pageAdminStore', { * Set the previous(usually current) pages to the ones we just decide. */ setPrevioiusPageUsingActivePage() { - this.previoiusPage = this.activePage; + this.previousPage = this.activePage; }, /** * Set the boolean value of status of pending state of the pate diff --git a/tests/stores/pageAdmin.test.js b/tests/stores/pageAdmin.test.js index a18f830..37e7eec 100644 --- a/tests/stores/pageAdmin.test.js +++ b/tests/stores/pageAdmin.test.js @@ -29,12 +29,10 @@ describe('pageAdminStore', () => { expect(store.previousPage).toBe('PERFORMANCE'); }); - it('setPrevioiusPageUsingActivePage copies activePage', () => { + it('setPrevioiusPageUsingActivePage copies activePage to previousPage', () => { store.setActivePage('CONFORMANCE'); store.setPrevioiusPageUsingActivePage(); - // Note: bug in source - writes to this.previoiusPage (typo) - // instead of this.previousPage, so previousPage stays 'MAP' - expect(store.previousPage).toBe('MAP'); + expect(store.previousPage).toBe('CONFORMANCE'); }); it('setIsPagePendingBoolean sets boolean', () => {