test: polish (#1280)

Author: lazzzis

packages/pinia/__tests__/actions.spec.ts

@@ -142,11 +142,6 @@ describe('Actions', () => {
     expect(() => store.throws()).toThrowError('fail')
   })
 
-  it('throws errors', () => {
-    const store = useStore()
-    expect(() => store.throws()).toThrowError('fail')
-  })
-
   it('throws async errors', async () => {
     const store = useStore()
     expect.assertions(1)
@@ -175,7 +170,7 @@ describe('Actions', () => {
       {
         $id: store.$id,
         simple,
-        // otherwise it would faial
+        // otherwise it would fail
         toggle() {},
       }.simple()
     ).toBe('simple')

packages/pinia/__tests__/getters.spec.ts

@@ -78,7 +78,7 @@ describe('Getters', () => {
     setActivePinia(pinia1)
     const aStore = useA()
 
-    // simulate a different applications
+    // simulate a different application
     setActivePinia(pinia2)
     const bStore = useB()
     bStore.b = 'c'

packages/pinia/__tests__/mapHelpers.spec.ts

@@ -52,7 +52,6 @@ describe('Map Helpers', () => {
       })
 
       const wrapper = mount(Component, { global: { plugins: [pinia] } })
-      // const store = useStore()
       // @ts-expect-error: by default this shouldn't exist
       expect(wrapper.vm.main).toBeDefined()
       expect(wrapper.vm.mainStore).not.toBeDefined()

packages/pinia/__tests__/state.spec.ts

@@ -95,17 +95,6 @@ describe('State', () => {
     expect(pinia.state.value.main.nested.n).toBe(3)
   })
 
-  // it('watch', () => {
-  //   setActivePinia(createPinia())
-  //   defineStore({
-  //     id: 'main',
-  //     state: () => ({
-  //       name: 'Eduardo',
-  //       counter: 0,
-  //     }),
-  //   })()
-  // })
-
   it('state can be watched', async () => {
     const store = useStore()
     const spy = jest.fn()