From 21532add46331a54d03493871abafc6c9662f309 Mon Sep 17 00:00:00 2001
From: "chihao.ou" <ou727963665@gmail.com>
Date: Tue, 15 Sep 2026 20:04:35 +0800
Subject: [PATCH] fix(payment): reset the order page when changing status

---
 frontend/src/views/user/UserOrdersView.vue    |  2 +-
 .../__tests__/UserOrdersView.filters.spec.ts  | 54 +++++++++++++++++++
 2 files changed, 55 insertions(+), 1 deletion(-)
 create mode 100644 frontend/src/views/user/__tests__/UserOrdersView.filters.spec.ts

diff --git a/frontend/src/views/user/UserOrdersView.vue b/frontend/src/views/user/UserOrdersView.vue
index c3ed80eb59..003357a098 100644
--- a/frontend/src/views/user/UserOrdersView.vue
+++ b/frontend/src/views/user/UserOrdersView.vue
@@ -4,7 +4,7 @@
       <!-- Filters -->
       <div class="card p-4">
         <div class="flex flex-wrap items-center gap-3">
-          <Select v-model="currentFilter" :options="statusFilters" class="w-36" @change="fetchOrders" />
+          <Select v-model="currentFilter" :options="statusFilters" class="w-36" @change="handlePageChange(1)" />
           <div class="flex flex-1 items-center justify-end gap-2">
             <button @click="fetchOrders" :disabled="loading" class="btn btn-secondary" :title="t('common.refresh')">
               <Icon name="refresh" size="md" :class="loading ? 'animate-spin' : ''" />
diff --git a/frontend/src/views/user/__tests__/UserOrdersView.filters.spec.ts b/frontend/src/views/user/__tests__/UserOrdersView.filters.spec.ts
new file mode 100644
index 0000000000..2bf22d8956
--- /dev/null
+++ b/frontend/src/views/user/__tests__/UserOrdersView.filters.spec.ts
@@ -0,0 +1,54 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { enableAutoUnmount, flushPromises, mount } from '@vue/test-utils'
+import UserOrdersView from '../UserOrdersView.vue'
+import Select from '@/components/common/Select.vue'
+import Pagination from '@/components/common/Pagination.vue'
+
+const api = vi.hoisted(() => ({ getMyOrders: vi.fn(), getRefundEligibleProviders: vi.fn() }))
+vi.mock('@/api/payment', () => ({ paymentAPI: api }))
+vi.mock('@/stores', () => ({ useAppStore: () => ({ showError: vi.fn() }) }))
+vi.mock('vue-router', () => ({ useRouter: () => ({ push: vi.fn() }) }))
+vi.mock('vue-i18n', async (importOriginal) => ({
+  ...await importOriginal<typeof import('vue-i18n')>(),
+  useI18n: () => ({ t: (key: string) => key })
+}))
+enableAutoUnmount(afterEach)
+
+beforeEach(() => {
+  vi.clearAllMocks()
+  api.getMyOrders.mockResolvedValue({ data: { items: [], total: 100 } })
+  api.getRefundEligibleProviders.mockResolvedValue({ data: { provider_instance_ids: [] } })
+})
+
+async function openOrders() {
+  const wrapper = mount(UserOrdersView, {
+    global: { stubs: {
+      AppLayout: { template: '<div><slot /></div>' },
+      OrderTable: true, BaseDialog: true, Icon: true, Pagination: true, teleport: true
+    } }
+  })
+  await flushPromises()
+  wrapper.getComponent(Pagination).vm.$emit('update:page', 4)
+  await flushPromises()
+  return wrapper
+}
+
+describe('order status filtering', () => {
+  it('loads the first page with the selected status', async () => {
+    const wrapper = await openOrders()
+    const select = wrapper.getComponent(Select)
+    await select.get('button').trigger('click')
+    await select.findAll('[role="option"]').find(option => option.text() === 'payment.status.pending')!.trigger('click')
+    await flushPromises()
+    expect(api.getMyOrders).toHaveBeenLastCalledWith({ page: 1, page_size: 20, status: 'PENDING' })
+    expect(wrapper.getComponent(Pagination).props('page')).toBe(1)
+    expect(api.getMyOrders).toHaveBeenCalledTimes(3)
+  })
+
+  it('keeps the current page on manual refresh', async () => {
+    const wrapper = await openOrders()
+    await wrapper.get('[title="common.refresh"]').trigger('click')
+    await flushPromises()
+    expect(api.getMyOrders).toHaveBeenLastCalledWith({ page: 4, page_size: 20, status: undefined })
+  })
+})