test: async throw

tmm · tmm · commit ae0cb2b2f939 · 2021-04-28T10:29:50.000-04:00
diff --git a/src/hooks/data/index.ts b/src/hooks/data/index.ts
@@ -0,0 +1,5 @@
+export * from './use-delete'
+export * from './use-filter'
+export * from './use-insert'
+export * from './use-select'
+export * from './use-update'
diff --git a/src/hooks/data/use-delete.ts b/src/hooks/data/use-delete.ts
@@ -1,8 +1,8 @@
 import { useCallback, useEffect, useRef, useState } from 'react'
 
-import { Count, Filter, PostgrestError, Returning } from '../types'
-import { useClient } from './use-client'
-import { initialState } from './state'
+import { Count, Filter, PostgrestError, Returning } from '../../types'
+import { initialState } from '../state'
+import { useClient } from '../use-client'
 
 export type UseDeleteState<Data = any> = {
     count?: number | null
@@ -14,7 +14,7 @@ export type UseDeleteState<Data = any> = {
 export type UseDeleteResponse<Data = any> = [
     UseDeleteState<Data>,
     (
-        filter: Filter<Data>,
+        filter?: Filter<Data>,
         options?: UseDeleteOptions,
     ) => Promise<Pick<UseDeleteState<Data>, 'count' | 'data' | 'error'>>,
 ]
@@ -42,7 +42,7 @@ export function useDelete<Data = any>(
         async (filter?: Filter<Data>, options?: UseDeleteOptions) => {
             const refine = filter ?? config.filter
             if (refine === undefined)
-                throw Error('update() should always be combined with `filter`')
+                throw Error('delete() should always be combined with `filter`')
 
             setState({ ...initialState, fetching: true })
             const source = client
diff --git a/src/hooks/data/use-filter.ts b/src/hooks/data/use-filter.ts
@@ -1,6 +1,6 @@
 import { useCallback } from 'react'
 
-import { Filter } from '../types'
+import { Filter } from '../../types'
 
 export function useFilter<Data = any>(filter: Filter<Data>, deps: any[] = []) {
     /* eslint-disable react-hooks/exhaustive-deps */
diff --git a/src/hooks/data/use-insert.ts b/src/hooks/data/use-insert.ts
@@ -1,8 +1,8 @@
 import { useCallback, useEffect, useRef, useState } from 'react'
 
-import { Count, PostgrestError, Returning } from '../types'
-import { useClient } from './use-client'
-import { initialState } from './state'
+import { Count, PostgrestError, Returning } from '../../types'
+import { initialState } from '../state'
+import { useClient } from '../use-client'
 
 export type UseInsertState<Data = any> = {
     count?: number | null
diff --git a/src/hooks/data/use-select.ts b/src/hooks/data/use-select.ts
@@ -1,8 +1,8 @@
 import { useCallback, useEffect, useMemo, useRef, useState } from 'react'
 
-import { Count, Filter, PostgrestError } from '../types'
-import { useClient } from './use-client'
-import { initialState } from './state'
+import { Count, Filter, PostgrestError } from '../../types'
+import { initialState } from '../state'
+import { useClient } from '../use-client'
 
 export type UseSelectState<Data = any> = {
     count?: number | null
diff --git a/src/hooks/data/use-update.ts b/src/hooks/data/use-update.ts
@@ -1,8 +1,8 @@
 import { useCallback, useEffect, useRef, useState } from 'react'
 
-import { Count, Filter, PostgrestError, Returning } from '../types'
-import { useClient } from './use-client'
-import { initialState } from './state'
+import { Count, Filter, PostgrestError, Returning } from '../../types'
+import { initialState } from '../state'
+import { useClient } from '../use-client'
 
 export type UseUpdateState<Data = any> = {
     count?: number | null
diff --git a/src/hooks/index.ts b/src/hooks/index.ts
@@ -1,7 +1,3 @@
+export * from './data'
+export * from './realtime'
 export * from './use-client'
-export * from './use-delete'
-export * from './use-filter'
-export * from './use-insert'
-export * from './use-select'
-export * from './use-subscription'
-export * from './use-update'
diff --git a/src/hooks/realtime/index.ts b/src/hooks/realtime/index.ts
@@ -0,0 +1 @@
+export * from './use-subscription'
diff --git a/src/hooks/realtime/use-subscription.ts b/src/hooks/realtime/use-subscription.ts
@@ -4,7 +4,7 @@ import {
     SupabaseRealtimePayload,
 } from '@supabase/supabase-js/dist/main/lib/types'
 
-import { useClient } from './use-client'
+import { useClient } from '../use-client'
 
 export type UseSubscriptionConfig = {
     event?: SupabaseEventTypes
diff --git a/src/hooks/use-client.ts b/src/hooks/use-client.ts
@@ -6,6 +6,6 @@ import { Context } from '../context'
 export const useClient = (): SupabaseClient => {
     const client = useContext(Context)
     if (client === undefined)
-        throw new Error('No client has been specified using Provider.')
+        throw Error('No client has been specified using Provider.')
     return client
 }
diff --git a/test/__snapshots__/use-client.test.tsx.snap b/test/__snapshots__/use-client.test.tsx.snap
@@ -0,0 +1,3 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`useClient should throw when not inside Provider 1`] = `"No client has been specified using Provider."`;
diff --git a/test/__snapshots__/use-delete.test.tsx.snap b/test/__snapshots__/use-delete.test.tsx.snap
@@ -0,0 +1,5 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`useDelete should throw when filter not provided to execute 1`] = `"delete() should always be combined with \`filter\`"`;
+
+exports[`useDelete should throw when not inside Provider 1`] = `"No client has been specified using Provider."`;
diff --git a/test/__snapshots__/use-insert.test.tsx.snap b/test/__snapshots__/use-insert.test.tsx.snap
@@ -0,0 +1,3 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`useInsert should throw when not inside Provider 1`] = `"No client has been specified using Provider."`;
diff --git a/test/__snapshots__/use-select.test.tsx.snap b/test/__snapshots__/use-select.test.tsx.snap
@@ -0,0 +1,3 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`useSelect should throw when not inside Provider 1`] = `"No client has been specified using Provider."`;
diff --git a/test/__snapshots__/use-subscription.test.tsx.snap b/test/__snapshots__/use-subscription.test.tsx.snap
@@ -0,0 +1,3 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`useSubscription should throw when not inside Provider 1`] = `"No client has been specified using Provider."`;
diff --git a/test/__snapshots__/use-update.test.tsx.snap b/test/__snapshots__/use-update.test.tsx.snap
@@ -0,0 +1,5 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`useUpdate should throw when filter not provided to execute 1`] = `"update() should always be combined with \`filter\`"`;
+
+exports[`useUpdate should throw when not inside Provider 1`] = `"No client has been specified using Provider."`;
diff --git a/test/use-client.test.tsx b/test/use-client.test.tsx
@@ -6,9 +6,7 @@ import { Wrapper as wrapper } from './utils'
 describe('useClient', () => {
     it('should throw when not inside Provider', () => {
         const { result } = renderHook(() => useClient())
-        expect(result.error).toEqual(
-            Error('No client has been specified using Provider.'),
-        )
+        expect(() => result.current).toThrowErrorMatchingSnapshot()
     })
 
     it('should return client', () => {
diff --git a/test/use-delete.test.tsx b/test/use-delete.test.tsx
@@ -1,12 +1,16 @@
 import { renderHook } from '@testing-library/react-hooks'
 
 import { useDelete } from '../src'
+import { Wrapper as wrapper } from './utils'
 
 describe('useDelete', () => {
     it('should throw when not inside Provider', () => {
         const { result } = renderHook(() => useDelete('todos'))
-        expect(result.error).toEqual(
-            Error('No client has been specified using Provider.'),
-        )
+        expect(() => result.current).toThrowErrorMatchingSnapshot()
+    })
+
+    it('should throw when filter not provided to execute', async () => {
+        const { result } = renderHook(() => useDelete('todos'), { wrapper })
+        await expect(result.current[1]()).rejects.toThrowErrorMatchingSnapshot()
     })
 })
diff --git a/test/use-insert.test.tsx b/test/use-insert.test.tsx
@@ -5,8 +5,6 @@ import { useInsert } from '../src'
 describe('useInsert', () => {
     it('should throw when not inside Provider', () => {
         const { result } = renderHook(() => useInsert('todos'))
-        expect(result.error).toEqual(
-            Error('No client has been specified using Provider.'),
-        )
+        expect(() => result.current).toThrowErrorMatchingSnapshot()
     })
 })
diff --git a/test/use-select.test.tsx b/test/use-select.test.tsx
@@ -5,8 +5,6 @@ import { useSelect } from '../src'
 describe('useSelect', () => {
     it('should throw when not inside Provider', () => {
         const { result } = renderHook(() => useSelect('todos'))
-        expect(result.error).toEqual(
-            Error('No client has been specified using Provider.'),
-        )
+        expect(() => result.current).toThrowErrorMatchingSnapshot()
     })
 })
diff --git a/test/use-subscription.test.tsx b/test/use-subscription.test.tsx
@@ -5,8 +5,6 @@ import { useSubscription } from '../src'
 describe('useSubscription', () => {
     it('should throw when not inside Provider', () => {
         const { result } = renderHook(() => useSubscription(jest.fn()))
-        expect(result.error).toEqual(
-            Error('No client has been specified using Provider.'),
-        )
+        expect(() => result.current).toThrowErrorMatchingSnapshot()
     })
 })
diff --git a/test/use-update.test.tsx b/test/use-update.test.tsx
@@ -1,12 +1,18 @@
 import { renderHook } from '@testing-library/react-hooks'
 
 import { useUpdate } from '../src'
+import { Wrapper as wrapper } from './utils'
 
 describe('useUpdate', () => {
     it('should throw when not inside Provider', () => {
         const { result } = renderHook(() => useUpdate('todos'))
-        expect(result.error).toEqual(
-            Error('No client has been specified using Provider.'),
-        )
+        expect(() => result.current).toThrowErrorMatchingSnapshot()
+    })
+
+    it('should throw when filter not provided to execute', async () => {
+        const { result } = renderHook(() => useUpdate('todos'), { wrapper })
+        await expect(
+            result.current[1]({ status: 'complete' }),
+        ).rejects.toThrowErrorMatchingSnapshot()
     })
 })

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,6 @@ import { Context } from '../context'`
`6`	`6`	`export const useClient = (): SupabaseClient => {`
`7`	`7`	`const client = useContext(Context)`
`8`	`8`	`if (client === undefined)`
`9`		`- throw new Error('No client has been specified using Provider.')`
	`9`	`+ throw Error('No client has been specified using Provider.')`
`10`	`10`	`return client`
`11`	`11`	`}`