fix(logging): silence react-query logger by default

Author: ismay

services/data/src/__tests__/integration.test.tsx

@@ -1,23 +1,8 @@
 import { render, waitFor } from '@testing-library/react'
 import React, { ReactNode } from 'react'
-import { setLogger } from 'react-query'
 import { CustomDataProvider, DataQuery } from '../react'
 import { QueryRenderInput } from '../types'
 
-beforeAll(() => {
-    // Prevent the react-query logger from logging to the console
-    setLogger({
-        log: jest.fn(),
-        warn: jest.fn(),
-        error: jest.fn(),
-    })
-})
-
-afterAll(() => {
-    // Restore the original react-query logger
-    setLogger(console)
-})
-
 describe('Testing custom data provider and useQuery hook', () => {
     it('Should render without failing', async () => {
         const data = {

services/data/src/react/hooks/useDataQuery.test.tsx

@@ -1,23 +1,8 @@
 import { renderHook, act } from '@testing-library/react-hooks'
 import React from 'react'
-import { setLogger } from 'react-query'
 import { CustomDataProvider } from '../components/CustomDataProvider'
 import { useDataQuery } from './useDataQuery'
 
-beforeAll(() => {
-    // Prevent the react-query logger from logging to the console
-    setLogger({
-        log: jest.fn(),
-        warn: jest.fn(),
-        error: jest.fn(),
-    })
-})
-
-afterAll(() => {
-    // Restore the original react-query logger
-    setLogger(console)
-})
-
 describe('useDataQuery', () => {
     describe('parameters: onComplete', () => {
         it('Should call onComplete with the data after a successful fetch', async () => {

services/data/src/react/hooks/useDataQuery.ts

@@ -1,11 +1,25 @@
 import { useState, useRef } from 'react'
-import { useQuery } from 'react-query'
+import { useQuery, setLogger } from 'react-query'
 import { Query, QueryOptions } from '../../engine'
 import { FetchError } from '../../engine/types/FetchError'
 import { QueryRenderInput, QueryRefetchFunction } from '../../types'
 import { useDataEngine } from './useDataEngine'
 import { useStaticInput } from './useStaticInput'
 
+const noop = () => {
+    /**
+     * Used to silence the default react-query logger. Eventually we
+     * could expose the setLogger functionality and remove the call
+     * to setLogger here.
+     */
+}
+
+setLogger({
+    log: noop,
+    warn: noop,
+    error: noop,
+})
+
 export const useDataQuery = (
     query: Query,
     {