chore: add test reports (#37)

* chore: add test reports

* test: enhance coverage

* test: enhance coverage

Author: aa900031

.github/workflows/test.yml

@@ -34,3 +34,10 @@ jobs:
         uses: codecov/codecov-action@v3
         env:
           CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+
+      - name: Upload test results to Codecov
+        if: ${{ !cancelled() }}
+        uses: codecov/test-results-action@v1
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          files: ./test-reports/junit.xml

.gitignore

@@ -1,3 +1,4 @@
 dist/
 node_modules/
 coverage/
+test-reports/

package.json

@@ -53,7 +53,7 @@
 	},
 	"scripts": {
 		"build": "unbuild",
-		"test": "vitest run --coverage",
+		"test": "vitest run",
 		"test:vue": "vue-demi-switch 3 && vitest run --dir ./src/vue",
 		"test:vue-2": "vue-demi-switch 2 vue-2 && vitest run --dir ./src/vue",
 		"test:vue-2.7": "vue-demi-switch 2.7 vue-2.7 && vitest run --dir ./src/vue",

src/core/index-v4.test.ts

@@ -124,5 +124,6 @@ describe('core (v4)', () => {
 		})
 
 		expect(unsubscribe).toBeTypeOf('function')
+		unsubscribe()
 	})
 })

src/core/index.test.ts

@@ -122,5 +122,6 @@ describe('core', () => {
 		})
 
 		expect(unsubscribe).toBeTypeOf('function')
+		unsubscribe()
 	})
 })

src/core/index.ts

@@ -56,13 +56,15 @@ export function subscribeQueryCallbacks<
 
 		switch (event.action.type) {
 			case 'success': {
+				/* istanbul ignore else */
 				if (!event.action.manual) {
 					onSuccess?.(event.action.data)
 					onSettled?.(event.action.data, null)
 				}
 				break
 			}
 			case 'error': {
+				/* istanbul ignore else */
 				if (!isCancelledError(event.action.error)) {
 					onError?.(event.action.error)
 					onSettled?.(undefined, event.action.error)

src/vue/index-v4.test.ts

@@ -1,6 +1,7 @@
 import { QueryClient, useQuery, useQueryClient, VueQueryPlugin } from '@tanstack/vue-query-v4'
 import { waitFor } from '@testing-library/vue'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { inject } from 'vue-demi'
 import { cleanUp, mountSetup } from '../../test/vue-mount'
 import { useQueryCallbacks } from './index'
 
@@ -144,6 +145,38 @@ describe('vue (v4)', () => {
 		expect(onSuccess).toBeCalledTimes(1)
 		expect(onSuccess).toBeCalledWith('bar')
 	})
+
+	it('should call onSuccess with `queryClientKey`', async () => {
+		const onSuccess = vi.fn()
+		const QUERY_KEY = ['foo']
+		const queryClientKey = '_vue_query_'
+
+		const { result: query } = mountSetup(() => {
+			const queryClient = inject(queryClientKey) as QueryClient
+			const result = useQuery({
+				queryKey: QUERY_KEY,
+				queryFn: () => Promise.resolve('bar'),
+				queryClient,
+			})
+
+			useQueryCallbacks({
+				queryKey: QUERY_KEY,
+				queryClientKey,
+				onSuccess,
+			})
+
+			return result
+		}, (app) => {
+			app.provide(queryClientKey, new QueryClient())
+		})
+
+		expect(query.data.value).toBeUndefined()
+		await waitFor(() => expect(query.data.value).not.toBeUndefined())
+
+		expect(query.data.value).toBe('bar')
+		expect(onSuccess).toBeCalledTimes(1)
+		expect(onSuccess).toBeCalledWith('bar')
+	})
 })
 
 function useQueryClientSetup<T>(

src/vue/index.test.ts

@@ -1,6 +1,7 @@
 import { QueryClient, useQuery, useQueryClient, VueQueryPlugin } from '@tanstack/vue-query'
 import { waitFor } from '@testing-library/vue'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { inject } from 'vue-demi'
 import { cleanUp, mountSetup } from '../../test/vue-mount'
 import { useQueryCallbacks } from './index'
 
@@ -139,6 +140,37 @@ describe('vue', () => {
 		expect(onSuccess).toBeCalledTimes(1)
 		expect(onSuccess).toBeCalledWith('bar')
 	})
+
+	it('should call onSuccess with `queryClientKey`', async () => {
+		const onSuccess = vi.fn()
+		const QUERY_KEY = ['foo']
+		const queryClientKey = '_vue_query_'
+
+		const { result: query } = mountSetup(() => {
+			const queryClient = inject(queryClientKey) as QueryClient
+			const result = useQuery({
+				queryKey: QUERY_KEY,
+				queryFn: () => Promise.resolve('bar'),
+			}, queryClient)
+
+			useQueryCallbacks({
+				queryKey: QUERY_KEY,
+				queryClientKey,
+				onSuccess,
+			})
+
+			return result
+		}, (app) => {
+			app.provide(queryClientKey, new QueryClient())
+		})
+
+		expect(query.data.value).toBeUndefined()
+		await waitFor(() => expect(query.data.value).not.toBeUndefined())
+
+		expect(query.data.value).toBe('bar')
+		expect(onSuccess).toBeCalledTimes(1)
+		expect(onSuccess).toBeCalledWith('bar')
+	})
 })
 
 function useQueryClientSetup<T>(

vitest.config.ts

@@ -5,7 +5,14 @@ export default defineConfig({
 		dir: './src',
 		environment: 'happy-dom',
 		coverage: {
+			enabled: true,
 			provider: 'istanbul',
 		},
+		reporters: [
+			'junit',
+		],
+		outputFile: {
+			junit: './test-reports/junit.xml',
+		},
 	},
 })