Nie można przetestować metod pocztowych przy użyciu Jest z powodu błędu. Nie można odczytać mockImplementation of undefined

9

Mam usługę interfejsu API, w której mam różne metody wykonywania połączeń z interfejsami API. Z powodzeniem przetestowałem wszystkie żądania GET, ale mam problemy z testowaniem żądań POST.

To jest metoda:

export default class ApiService {
  static makeApiCall = <T>(
    url: string,
    oneCb: <T>(d: Data) => T,
    secondCb: (d: T) => void,
    errorCb?: (a: ErrorModel) => void,
    method = 'get',
    data = {},
  ): Promise<void> => {
    const config: AxiosRequestConfig = {};
    if (method === 'post') {
      config.headers = header;
      return ApiClient.post(url, data, config)
        .then(res => callback(normalizeCallback(res.data))).catch(error => someHandler(error));            
    } else {
      return ApiClient.get(url)
        .then(res => callback(normalizeCallback(res.data))).catch(error => someHandler(error));
    }
  };

  // ONLY ONE POST METHOD TO MAKE IT MORE CLEAR
  static someArchiveMethod = (
    callback: (a: SuccessModel) => void,
    errorCallback: (error: ErrorModel) => void,
    cardId: string
  ): Promise<void> => {
    return ApiService.makeApiCall<SuccessfulResponse>(
      'appreciationCard/archive',
      Normalizer.successfulResponse,
      callback,
      errorCallback,
      'post',
      { cardId }
    );
  };

  // HERE BELOW THE GET METHODS
  static getPeople = (cb: (a: PeopleModel[]) => void, page?: number, limit?: number): Promise<void> => {
    const queryDetails = { page, limit };
    return ApiService.makeApiCall<PeopleModel[]>(
      `people?${toQueryString(queryDetails)}`,
      Normalizer.normalizePeople,
      callback
    );
  };
};

W ten sposób testuję wszystko związane z GET:

describe('apiService', () => {
  beforeAll(() => {
    expect(ApiClient.defaults.headers.common.Authorization).toBe('Bearer test token');
    // @ts-ignore
    ApiClient.get.mockImplementation((url: string) => {
      return Promise.resolve({ data: mockData });
    });
  });

  it('should call api client method', () => {
    ApiService.makeApiCall(
      'testUrl',
      data => data,
      res => res,
      err => err,
      'get'
    );

    expect(ApiClient.get).toBeCalledTimes(1);
    expect(ApiClient.get).toBeCalledWith('testUrl');
  });

  it('should call callbacks consequently', done => {
    ApiService.makeApiCall('testUrl', firstCallback, secondCallback).then(() => {
      expect(firstCallback).toBeCalledTimes(1);
      expect(firstCallback).toBeCalledWith(mockData);
      expect(secondCallback).toBeCalledTimes(1);
      expect(secondCallback).toBeCalledWith(firstCallback(mockData));
      done();
    });
  });
});

describe('api service error flow', () => {
  beforeAll(() => {
    // @ts-ignore
    ApiClient.get.mockImplementation((url: string) => {
      console.log('error result');
      return Promise.reject(mockError);
    });
  });

  it('should handle error', done => {
    console.error = jest.fn();

    const firstCallback = jest.fn((data: any) => data);
    const secondCallback = jest.fn((data: any) => data);

    ApiService.makeApiCall('testUrl', firstCallback, secondCallback).then(() => {
      expect(firstCallback).toBeCalledTimes(0);
      expect(secondCallback).toBeCalledTimes(0);
      expect(console.error).toBeCalledTimes(1);
      expect(console.error).toBeCalledWith('ApiClient testUrl', mockError);
      done();
    });
  });
});

describe('apiService methods', () => {
  beforeAll(() => {
    ApiClient.get.mockImplementation((url: string) => {
      expect(ApiClient.defaults.headers.common.Authorization).toBe('Bearer test token');

      return Promise.resolve({ data: mockData });
    });
  });

  it('getPeople method call with one param', () => {
    ApiService.getPeople(jest.fn(), 1, 1).then(() => {
      expect(ApiClient.get).toBeCalledWith('people?page=1&limit=1');
    });
  });
})

Pomyślałem, że tylko zmiana wszystkich instancji ApiClient.getna ApiClient.postto będzie działać w celu przetestowania żądań POST. Ale kiedy próbuję to zrobić, mówi to can not read mockImplementation of undefined. Próbowałem zmienić metody w testach, aby użyć postparametru w celu zastąpienia parametru, method = 'get'ale nie mam powodzenia, pojawia się ten błąd

TypeError: apiClient_1.default.post nie jest funkcją

jakieś pomysły?

Reagowanie
źródło
Jednym z powodów byłby ApiClientbrak metody post.
Tomas
Cześć, @Tomas spójrz na tę linię -> return ApiClient.post(url, data, config) .then(res => callback(normalizeCallback(res.data))).catch(error => someHandler(error));i działa poprawnie, gdy próbuję wysłać żądanie. Mam na myśli, że mam 17 żądań postów, które działają tak, jak muszą. Dlaczego więc w testach nie działają?
Reakcja
@Reacting Udostępnij przykładowy test jednostki „post”
Oron Ben-David
@ OronBen-David Wspomniałem w pytaniu, że próbowałem dokładnie tak samo jak w getteście, ale zamiast tego zmieniłem wszystkie wystąpienia geti ustawiłem post.
Reagowanie
Rozumiem, ale łatwiej będzie wymienić kod, który nie działa
Oron Ben-David

Odpowiedzi:

5

Zbadałem twój problem. Przede wszystkim chcę powiedzieć, że twój kod ma kilka problemów, takich jak wywołanie zwrotne, którego nie zdefiniowałeś, niejasne zdefiniowanie ApiClientitp.

Dlatego stworzyłem przykład Repl w celu odtworzenia problemu, w którym nieco uprościłem kod, ale wszystkie główne elementy są dostępne.

Proszę spojrzeć https://repl.it/@SergeyMell/Some-Jesting

Działa skutecznie zarówno geti postmetod bez problemów. Oto główne punkty, na które powinieneś zwrócić uwagę:

  1. Używając axiosjako ApiClient. (Nie było jasne z twojego pytania, więc założyłem, że tak jest)
    const ApiClient = require('axios');
  2. Włączanie żartów axios(Załóżmy, że robisz to samo)
    jest.mock('axios');
  3. Nakładanie prób na oba geti postżądania w podobny sposób (na swój sposób)

    ApiClient.get.mockImplementation((url) => {
      return Promise.resolve({ data: mockData });
    });
    
    ApiClient.post.mockImplementation((url) => {
      return Promise.resolve({ data: mockData });
    });

Więc proszę sprawdź mój przykład, sprawdź różnice w kodzie i daj mi znać odnośnie dodatkowej detalizacji, której możesz potrzebować.

Sergey Mell
źródło
0

Spróbuj zmienić mockImplementationnamockImplementationOnce

Oron Ben-David
źródło