@coorpacademy/app-review 0.1.7 → 0.1.8

package/src/actions/data/skills.ts

@@ -1,24 +0,0 @@
-import {Skills} from '../../types/skills';
-
-export const RECEIVED_SKILLS = '@@data/skills/RECEIVED_SKILLS';
-export const ERROR_FETCHING_SKILLS = '@@data/skills/ERROR_FETCHING_SKILLS';
-
-export type ReceivedSkills = {
-  type: '@@data/skills/RECEIVED_SKILLS';
-  payload: Skills;
-};
-
-export type ErrorFetchingSkills = {
-  type: '@@data/skills/ERROR_FETCHING_SKILLS';
-  payload: Error;
-};
-
-export const receivedSkills = (skills: Skills): ReceivedSkills => ({
-  type: RECEIVED_SKILLS,
-  payload: skills
-});
-
-export const errorFetchingSkills = (error: Error): ErrorFetchingSkills => ({
-  type: ERROR_FETCHING_SKILLS,
-  payload: error
-});

package/src/middlewares/on-receive-progression.test.ts

@@ -1,60 +0,0 @@
-import test from 'ava';
-import {AnyAction, Dispatch, MiddlewareAPI} from 'redux';
-import {receiveProgression} from '../actions/data/progression';
-import {Action} from '../actions';
-import {AppOptions} from '../types/common';
-import {mockedProgression} from '../services/post-progression.mock';
-import {sleep} from '../services/tools/sleep';
-import {STORE_FIRST_SLIDE} from '../actions/data/slides';
-import onReceiveProgression from './on-receive-progression';
-
-const appOptions: AppOptions = {
-  token: process.env.API_TEST_TOKEN || ''
-};
-
-const otherAction = (
-  payload: string
-): {
-  type: string;
-  payload: string;
-} => ({
-  type: '@@other/OTHER',
-  payload
-});
-
-test('should not dispatch anything if action is not RECEIVE_PROGRESSION', t => {
-  t.plan(1);
-  const dispatch = (): void => {
-    t.fail();
-  };
-  const next = (): boolean => {
-    return t.pass();
-  };
-
-  onReceiveProgression({dispatch} as unknown as MiddlewareAPI)(next as Dispatch<AnyAction>)(
-    otherAction('other')
-  );
-});
-
-test('should dispatch actions: STORE_FIRST_SLIDE', async t => {
-  t.plan(2);
-  const dispatch = (action: Action): void => {
-    switch (action.type) {
-      case STORE_FIRST_SLIDE: {
-        const slide = action.payload;
-        t.deepEqual(slide, {});
-        return;
-      }
-      default:
-        t.fail();
-        break;
-    }
-  };
-  const next = (): boolean => {
-    return t.pass();
-  };
-  onReceiveProgression({dispatch} as MiddlewareAPI)(next as Dispatch<AnyAction>)(
-    receiveProgression({token: appOptions.token, progression: mockedProgression})
-  );
-  await sleep(1000); // wait for the .then to fire
-});

package/src/middlewares/on-receive-progression.ts

@@ -1,42 +0,0 @@
-import type {Middleware, Store} from 'redux';
-import {Dispatch} from 'redux';
-import get from 'lodash/fp/get';
-
-// eslint-disable-next-line import/no-named-as-default
-import fetchSlide from '../services/fetch-slide';
-
-import {
-  ErrorFetchingSlide,
-  errorFetchingSlide,
-  StoreFirstSlide,
-  storeFirstSlide
-} from '../actions/data/slides';
-import {ReceiveProgression, RECEIVE_PROGRESSION} from '../actions/data/progression';
-
-const onReceiveProgression: Middleware<
-  {},
-  Store,
-  Dispatch<ReceiveProgression | StoreFirstSlide | ErrorFetchingSlide>
-> =
-  ({dispatch}) =>
-  next =>
-  (action: ReceiveProgression) => {
-    if (action.type === RECEIVE_PROGRESSION) {
-      const {progression, token} = action.payload;
-      const slideRef: string = get('state.nextContent.ref', progression);
-      // TOASK: do we need to check for exit nodes before fetching??
-
-      // TODO: To implement fetchSlide in next ticket
-      fetchSlide(slideRef, token)
-        .then(slide => {
-          return dispatch(storeFirstSlide(slide));
-        })
-        .catch(err => {
-          return dispatch(errorFetchingSlide(err));
-        });
-    }
-
-    return next(action);
-  };
-
-export default onReceiveProgression;

package/src/middlewares/on-start-app.test.ts

@@ -1,68 +0,0 @@
-import test from 'ava';
-import {AnyAction, Dispatch, MiddlewareAPI} from 'redux';
-import {STORE_TOKEN} from '../actions/data/token';
-import {RECEIVE_PROGRESSION} from '../actions/data/progression';
-import {Action} from '../actions';
-import {AppOptions} from '../types/common';
-import {NAVIGATE_TO, startApp} from '../actions/ui/navigation';
-import {mockedProgression} from '../services/post-progression.mock';
-import {VIEWS} from '../common';
-import {sleep} from '../services/tools/sleep';
-import onStartApp from './on-start-app';
-
-const appOptions: AppOptions = {
-  token: process.env.API_TEST_TOKEN || '',
-  skillRef: 'skill_NJC0jFKoH'
-};
-
-const otherAction = (
-  payload: string
-): {
-  type: string;
-  payload: string;
-} => ({
-  type: '@@other/OTHER',
-  payload
-});
-
-test('should not dispatch anything if action is not START_APP', t => {
-  t.plan(1);
-  const dispatch = (): void => {
-    t.fail();
-  };
-  const next = (): boolean => {
-    return t.pass();
-  };
-
-  onStartApp({dispatch} as unknown as MiddlewareAPI)(next as Dispatch<AnyAction>)(
-    otherAction('other')
-  );
-});
-
-test('should dispatch actions: STORE_TOKEN && RECEIVE_PROGRESSION if an skillRef is given, on a successful response', async t => {
-  t.plan(5);
-  const dispatch = (action: Action): void => {
-    switch (action.type) {
-      case STORE_TOKEN:
-        t.is(action.payload, appOptions.token);
-        return;
-      case RECEIVE_PROGRESSION: {
-        const {token, progression} = action.payload;
-        t.is(token, appOptions.token);
-        t.deepEqual(progression, JSON.parse(JSON.stringify(mockedProgression)));
-        return;
-      }
-      case NAVIGATE_TO:
-        t.is(action.payload, VIEWS.slides);
-        return;
-      default:
-        t.fail();
-        break;
-    }
-  };
-  const next = (): boolean => {
-    return t.pass();
-  };
-  onStartApp({dispatch} as MiddlewareAPI)(next as Dispatch<AnyAction>)(startApp(appOptions));
-  await sleep(1000); // wait for the .then to fire
-});

package/src/middlewares/on-start-app.ts

@@ -1,50 +0,0 @@
-import type {Middleware, Store} from 'redux';
-import {Dispatch} from 'redux';
-
-import {VIEWS} from '../common';
-import {Action} from '../actions';
-import {storeToken} from '../actions/data/token';
-import {receivedSkills, errorFetchingSkills} from '../actions/data/skills';
-import {navigateTo, START_APP, ViewPath} from '../actions/ui/navigation';
-
-// eslint-disable-next-line import/no-named-as-default
-import fetchSkills from '../services/fetch-skills';
-// eslint-disable-next-line import/no-named-as-default
-import postProgression from '../services/post-progression';
-import {errorCreatingProgression, receiveProgression} from '../actions/data/progression';
-
-const onStartApp: Middleware<{}, Store, Dispatch<Action>> =
-  ({dispatch}) =>
-  next =>
-  (action: Action) => {
-    if (action.type === START_APP) {
-      const {skillRef, token} = action.payload;
-
-      dispatch(storeToken(token));
-
-      if (skillRef) {
-        postProgression(skillRef, token)
-          .then(progression => {
-            return dispatch(receiveProgression({progression, token}));
-          })
-          .catch(err => {
-            return dispatch(errorCreatingProgression(err));
-          });
-      } else {
-        fetchSkills(token)
-          .then(skills => {
-            return dispatch(receivedSkills(skills));
-          })
-          .catch(err => {
-            return dispatch(errorFetchingSkills(err));
-          });
-      }
-
-      const initialView: ViewPath = skillRef ? VIEWS.slides : VIEWS.skills;
-      dispatch(navigateTo(initialView));
-    }
-
-    return next(action);
-  };
-
-export default onStartApp;

package/src/services/fetch-skills.mock.ts

@@ -1,21 +0,0 @@
-import crossFetch from 'cross-fetch';
-import {Skills} from '../types/skills';
-import {okJSONResponse} from './tools/fetch.mocks';
-
-const result: Skills = [
-  {
-    skillRef: '_skill-ref',
-    slidesToReview: 2,
-    name: 'skill-test',
-    custom: false
-  },
-  {
-    skillRef: '_skill-ref-2',
-    slidesToReview: 2,
-    name: 'skill-test-2',
-    custom: true
-  }
-];
-
-const mockFetchSkills: typeof crossFetch = okJSONResponse(result);
-export default mockFetchSkills;

package/src/services/fetch-skills.test.ts

@@ -1,32 +0,0 @@
-import test from 'ava';
-import {Skills} from '../types/skills';
-import {fetchSkills} from './fetch-skills';
-import {okJSONResponse, token} from './tools/fetch.mocks';
-
-test('should fetch skills with success', async t => {
-  const result: Skills = [
-    {
-      slidesToReview: 1,
-      name: 'skill-test',
-      custom: false,
-      skillRef: '_skill-ref'
-    }
-  ];
-
-  const mockedFetchSkills = fetchSkills(okJSONResponse(result));
-  const skills = await mockedFetchSkills(token);
-
-  t.deepEqual(result, skills);
-});
-
-test('should reject bad token', async t => {
-  const badToken = 'token is not a jwt';
-  const mockedFetchSkills = fetchSkills(okJSONResponse([]));
-
-  const error = await t.throwsAsync(() => mockedFetchSkills(badToken));
-
-  t.is(
-    error?.message,
-    "Invalid token specified: Cannot read properties of undefined (reading 'replace')"
-  );
-});

package/src/services/post-progression.mock.ts

@@ -1,35 +0,0 @@
-import crossFetch from 'cross-fetch';
-import {ProgressionFromAPI} from '../actions/data/progression';
-import {okJSONResponse} from './tools/fetch.mocks';
-
-export const mockedProgression: ProgressionFromAPI = {
-  _id: '62b1d1087aa12f00253f40ee',
-  state: {
-    // ------------------------------------------------------
-    // unused for now / to decide whether to use it when the
-    // validation-progression is plugged in
-    pendingSlides: [],
-    step: {
-      current: Number.NaN
-    },
-    // ------------------------------------------------------
-    isCorrect: true,
-    nextContent: {
-      ref: 'sli_N1XACJobn',
-      type: 'slide'
-    }
-    // slides: [
-    //   'sli_NJ3m5KtWh',
-    //   'sli_V1gKpYYZ2',
-    //   'sli_VJzOgotbh',
-    //   'sli_V1ug-dtZn',
-    //   'sli_VyZU-ejW3',
-    //   'sli_Vy8ChgjZh',
-    //   'sli_V1ijS1oWh',
-    //   'sli_N1XACJobn'
-    // ],
-  }
-};
-
-const mockCreateProgression: typeof crossFetch = okJSONResponse(mockedProgression);
-export default mockCreateProgression;

package/src/services/post-progression.test.ts

@@ -1,41 +0,0 @@
-import test from 'ava';
-import {ProgressionFromAPI} from '../actions/data/progression';
-import {postProgression} from './post-progression';
-import {okJSONResponse, token} from './tools/fetch.mocks';
-
-const skillRef = '_skill-ref';
-
-test('should create progression with success', async t => {
-  const result: ProgressionFromAPI = {
-    _id: '123456123456',
-    state: {
-      // is "isCorrect" always true for a 1st progression?
-      isCorrect: true,
-      step: {
-        current: 1
-      },
-      nextContent: {
-        type: 'slide',
-        ref: 'sli_NJ3m5KtWh'
-      },
-      pendingSlides: []
-    }
-  };
-
-  const mockedPostProgression = postProgression(okJSONResponse(result));
-  const progression = await mockedPostProgression(skillRef, token);
-
-  t.deepEqual(result, progression);
-});
-
-test('should reject bad token', async t => {
-  const badToken = 'token is not a jwt';
-  const mockedPostProgression = postProgression(okJSONResponse(null));
-
-  const error = await t.throwsAsync(() => mockedPostProgression(skillRef, badToken));
-
-  t.is(
-    error?.message,
-    "Invalid token specified: Cannot read properties of undefined (reading 'replace')"
-  );
-});

package/src/services/tools/fetch.mocks.ts

@@ -1,41 +0,0 @@
-import crossFetch from 'cross-fetch';
-
-export const toResponse = (
-  statusCode: number,
-  statusText: string,
-  body: string
-): Promise<Response> =>
-  Promise.resolve({
-    ok: statusCode < 400,
-    status: statusCode,
-    statusText,
-    json: (): Promise<unknown> => Promise.resolve(body).then(JSON.parse),
-    text: (): Promise<string> => Promise.resolve(body)
-  } as Response);
-
-export const okJSONResponse =
-  (body: unknown): typeof crossFetch =>
-  () =>
-    toResponse(200, 'OK', JSON.stringify(body));
-
-export const internalServerErrorJSONResponse =
-  (body: unknown): typeof crossFetch =>
-  () =>
-    toResponse(500, 'InternalServerError', JSON.stringify(body));
-
-export const notFoundJSONResponse =
-  (body: unknown): typeof crossFetch =>
-  () =>
-    toResponse(404, 'Not Found', JSON.stringify(body));
-
-export const okTextResponse =
-  (body: string): typeof crossFetch =>
-  () =>
-    toResponse(200, 'OK', body);
-
-export const internalServerErrorTextResponse =
-  (body: string): typeof crossFetch =>
-  () =>
-    toResponse(500, 'InternalServerError', body);
-
-export const token: string = process.env.API_TEST_TOKEN || 'no-API_TEST_TOKEN';

package/src/types/skills.ts

@@ -1,8 +0,0 @@
-export type Skill = {
-  skillRef: string;
-  slidesToReview: number;
-  custom: boolean;
-  name: string;
-};
-
-export type Skills = Skill[];