fix(testing): correct order of plugin installation

Author: posva

packages/testing/src/testing.spec.ts

@@ -224,4 +224,61 @@ describe('Testing', () => {
     store.n++
     expect(store.double).toBe(6)
   })
+
+  it('actions are stubbed even when replaced by other plugins', () => {
+    const spy = jest.fn()
+    mount(Counter, {
+      global: {
+        plugins: [
+          createTestingPinia({
+            plugins: [
+              ({ store }) => {
+                const { increment } = store.increment
+                store.increment = spy
+                spy.mockImplementation(increment)
+              },
+            ],
+          }),
+        ],
+      },
+    })
+    const counter = useCounter()
+
+    counter.increment()
+    counter.increment(5)
+    expect(counter.n).toBe(0)
+    expect(counter.increment).toHaveBeenCalledTimes(2)
+    expect(counter.increment).toHaveBeenLastCalledWith(5)
+    // the actual spy is never called because we stub the action
+    expect(spy).toHaveBeenCalledTimes(0)
+  })
+
+  it('pass through replaced actions in plugins', () => {
+    const spy = jest.fn()
+    mount(Counter, {
+      global: {
+        plugins: [
+          createTestingPinia({
+            stubActions: false,
+            plugins: [
+              ({ store }) => {
+                const { increment } = store.increment
+                store.increment = spy
+                spy.mockImplementation(increment)
+              },
+            ],
+          }),
+        ],
+      },
+    })
+    const counter = useCounter()
+
+    counter.increment()
+    counter.increment(5)
+    expect(counter.n).toBe(0)
+    expect(counter.increment).toHaveBeenCalledTimes(2)
+    expect(counter.increment).toHaveBeenLastCalledWith(5)
+    expect(spy).toHaveBeenCalledTimes(2)
+    expect(spy).toHaveBeenLastCalledWith(5)
+  })
 })

packages/testing/src/testing.ts

@@ -109,7 +109,8 @@ export function createTestingPinia({
   // allow computed to be manually overridden
   pinia._p.push(WritableComputed)
 
-  plugins.forEach((plugin) => pinia.use(plugin))
+  // bypass waiting for the app to be installed to ensure the action stubbing happens last
+  plugins.forEach((plugin) => pinia._p.push(plugin))
 
   const createSpy =
     _createSpy ||