nalalangan/grocery-app
1
0
Fork 0
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

fix: silence store selector debug logs

Browse Source
This commit is contained in:
Nico 2026-06-04 06:11:14 -07:00
parent f9aa38e57a
commit 90878a6d7f
2 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
frontend/src/context/StoreContext.jsx
View File

@ -84,7 +84,6 @@ export const StoreProvider = ({ children }) => {
useEffect(() => {
if (!activeHousehold || stores.length === 0) return;
console.log('[StoreContext] Setting active store from:', stores);
const householdStoreStorageKey = `activeHouseholdStoreId_${activeHousehold.id}`;
const legacyLocationStorageKey = `activeStoreId_${activeHousehold.id}`;
const savedHouseholdStoreId = localStorage.getItem(householdStoreStorageKey);
@ -94,7 +93,6 @@ export const StoreProvider = ({ children }) => {
chooseLocationByStoredIds(stores, activeHousehold.id, savedHouseholdStoreId) ||
chooseStoreByHouseholdStoreId(stores, savedHouseholdStoreId);
if (store) {
console.log('[StoreContext] Found saved household store:', store);
setActiveStoreState(store);
return;
}
@ -105,7 +103,6 @@ export const StoreProvider = ({ children }) => {
const savedLocation = stores.find(s => String(s.id) === String(savedLocationId));
const store = savedLocation || chooseStoreByHouseholdStoreId(stores, getHouseholdStoreKey(savedLocation));
if (store) {
console.log('[StoreContext] Migrated saved location to store:', store);
const householdStoreId = getHouseholdStoreKey(store);
setActiveStoreState(store);
localStorage.setItem(householdStoreStorageKey, householdStoreId);
@ -122,7 +119,6 @@ export const StoreProvider = ({ children }) => {
stores,
getHouseholdStoreKey(stores.find(s => s.is_default) || stores[0])
);
console.log('[StoreContext] Using store:', defaultStore);
setActiveStoreState(defaultStore);
const householdStoreId = getHouseholdStoreKey(defaultStore);
localStorage.setItem(householdStoreStorageKey, householdStoreId);
@ -146,9 +142,7 @@ export const StoreProvider = ({ children }) => {
setLoading(true);
setError(null);
try {
console.log('[StoreContext] Loading stores for household:', activeHousehold.id);
const response = await getHouseholdStores(activeHousehold.id);
console.log('[StoreContext] Loaded stores:', response.data);
setStores(response.data);
} catch (err) {
console.error('[StoreContext] Failed to load stores:', err);

8
frontend/tests/store-selector.spec.ts
View File

@ -95,6 +95,13 @@ async function mockGroceryShell(page: Page, stores: Array<Record<string, unknown
test("grocery store selector shows one selected store and picks from a modal", async ({ page }) => {
await page.setViewportSize({ width: 473, height: 1000 });
const storeContextMessages: string[] = [];
page.on("console", (message) => {
const text = message.text();
if (text.includes("[StoreContext]")) {
storeContextMessages.push(text);
}
});
await mockGroceryShell(page, [
{
id: 10,
@ -191,6 +198,7 @@ test("grocery store selector shows one selected store and picks from a modal", a
await expect(storeSelector).toContainText("Costco");
await expect(locationSelector).toContainText("Fontana");
await expect(page.getByRole("button", { name: "Manage Map" })).toBeVisible();
expect(storeContextMessages).toEqual([]);
});
test("single grocery store selector click does not open picker modal", async ({ page }) => {