CODE: Refactor and spawn into a separate library TC-related actions/reducers/services

Submission 253940 by liuliquan to the challenge
https://www.topcoder.com/challenges/30064737

Author: birdofpreyru

Diff for: .babelrc

@@ -0,0 +1,3 @@
+{
+  "presets": ["topcoder-react-utils/config/babel/node-ssr"]
+}

Diff for: .circleci/config.yml

@@ -0,0 +1,53 @@
+version: 2
+jobs:
+  # Release.
+  # TODO: Probably, it is possible to reuse the test job below as a part of
+  # the release job; but as I don't know how to do it at the moment, let's
+  # just dublicate the logic for now.
+  release:
+    docker:
+      - image: circleci/node:8.9.4
+    steps:
+      - checkout
+      - restore_cache:
+          # Careful here, though, at the moment, there should be no problem
+          # to reuse the same cache of "node_modules" between test and release
+          # jobs.
+          key: test-node-modules-{{ checksum "package-lock.json" }}
+      - run: npm install
+      - save_cache:
+          key: test-node-modules-{{ checksum "package-lock.json" }}
+          paths:
+            - node_modules
+      - run: npm test
+      - run: echo "//registry.npmjs.org/:_authToken=$NPM_TOKEN" >> ~/.npmrc
+      - run: npm publish
+
+  # Just tests commited code.
+  test:
+    docker:
+      - image: circleci/node:8.9.4
+    steps:
+      - checkout
+      - restore_cache:
+          key: test-node-modules-{{ checksum "package-lock.json" }}
+      - run: npm install
+      - save_cache:
+          key: test-node-modules-{{ checksum "package-lock.json" }}
+          paths:
+            - node_modules
+      - run: npm test
+
+workflows:
+  version: 2
+  build:
+    jobs:
+      - release:
+          filters:
+            branches:
+              only: master
+      - test:
+          filters:
+            branches:
+              ignore: master
+            

Diff for: .eslintignore

@@ -0,0 +1,3 @@
+__coverage__
+dist
+node_modules

Diff for: .eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "./node_modules/topcoder-react-utils/config/eslint/default.json"
+}

Diff for: .gitignore

@@ -0,0 +1,5 @@
+__coverage__
+.build-info
+.sass-cache
+dist
+node_modules

Diff for: .nvmrc

@@ -0,0 +1 @@
+v8.11.1

Diff for: .stylelintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "topcoder-react-utils/config/stylelint/default"
+}

Diff for: CHANGELOG.md

@@ -0,0 +1,4 @@
+# Topcoder React Lib
+
+### v0.0.x
+Pre-release drafts of the initial library version. A big journey starts here.

Diff for: README.md

@@ -0,0 +1,25 @@
+# Topcoder React Lib
+
+The [Topcoder](https://www.topcoder.com) lib for internal ReactJS projects.
+
+### Development
+```shell
+# Install dependencies
+npm install
+
+# Run test & build
+npm test
+npm run build
+
+# Go to other project which depends on the topcoder-react-lib, config its package.json so 
+# that the 'topcoder-react-lib' points to the local foler path of topcoder-react-lib:
+#
+# "dependencies": {
+#   "topcoder-react-lib": "<local-path-to-topcoder-react-lib>",
+#   ......
+# }
+#
+```
+
+### License
+UNLICENSED &mdash; for internal Topcoder projects only.

Diff for: __tests__/.babelrc

@@ -0,0 +1,3 @@
+{
+  "presets": ["topcoder-react-utils/config/babel/node-ssr"]
+}

Diff for: __tests__/.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "../node_modules/topcoder-react-utils/config/eslint/jest.json"
+}

Diff for: __tests__/actions/auth.js

@@ -0,0 +1,76 @@
+jest.setMock('isomorphic-fetch', {});
+
+const GROUPS_REQ_URL =
+  'https://api.topcoder-dev.com/v3/groups?memberId=12345&membershipType=user';
+const PROFILE_REQ_URL = 'https://api.topcoder-dev.com/v3/members/username12345';
+
+global.fetch = jest.fn(url => Promise.resolve({
+  json: () => {
+    let content;
+    switch (url) {
+      case GROUPS_REQ_URL: content = ['Group1', 'Group2']; break;
+      case PROFILE_REQ_URL: content = { userId: 12345 }; break;
+      default: throw new Error('Unexpected URL!');
+    }
+    return {
+      result: { content, status: 200 },
+    };
+  },
+}));
+
+const { actions } = require('../../src');
+
+describe('fetch with success response', () => {
+  beforeEach(() => jest.clearAllMocks());
+
+  test('auth.loadProfile works as expected when authenticated', () => {
+    const action = actions.auth.loadProfile('token');
+    expect(action.type).toBe('AUTH/LOAD_PROFILE');
+    return action.payload.then((res) => {
+      expect(global.fetch).toHaveBeenCalledWith(PROFILE_REQ_URL, {
+        headers: {
+          Authorization: 'Bearer token',
+          'Content-Type': 'application/json',
+        },
+      });
+      expect(global.fetch).toHaveBeenCalledWith(GROUPS_REQ_URL, {
+        headers: {
+          Authorization: 'Bearer token',
+          'Content-Type': 'application/json',
+        },
+      });
+      expect(res).toEqual({
+        groups: ['Group1', 'Group2'],
+        userId: 12345,
+      });
+    });
+  });
+
+  test('auth.loadProfile with empty token', () => {
+    const action = actions.auth.loadProfile('');
+    expect(action.type).toBe('AUTH/LOAD_PROFILE');
+    return action.payload.then((res) => {
+      expect(res).toBe(null);
+    });
+  });
+});
+
+describe('fetch with failed response', () => {
+  beforeAll(() => {
+    global.fetch = jest.fn(() => Promise.resolve({
+      json: () => ({
+        result: { status: 404 },
+      }),
+    }));
+  });
+
+  test('fetch return 404', () => {
+    const action = actions.auth.loadProfile('token');
+    expect(action.type).toBe('AUTH/LOAD_PROFILE');
+    return action.payload.then((res) => {
+      expect(res).toEqual({
+        groups: [],
+      });
+    });
+  });
+});

Diff for: __tests__/actions/challenge.js

@@ -0,0 +1,76 @@
+import { actions } from '../../src';
+
+jest.mock('../../src/services/challenges');
+
+const mockFetch = resolvesTo => jest.fn(() =>
+  Promise.resolve({ json: () => resolvesTo }));
+
+let originalFetch;
+
+beforeAll(() => {
+  originalFetch = global.fetch;
+});
+
+afterAll(() => {
+  global.fetch = originalFetch;
+});
+
+describe('challenge.fetchChallengeInit', () => {
+  const a = actions.challenge.getDetailsInit(12345);
+
+  test('has expected type', () => {
+    expect(a.type).toBe('CHALLENGE/GET_DETAILS_INIT');
+  });
+
+  test('payload is the challenge ID, converted to string, if necessary', () =>
+    expect(a.payload).toBe('12345'));
+});
+
+describe('challenge.fetchSubmissionsInit', () => {
+  const a = actions.challenge.getSubmissionsInit(12345);
+
+  test('has expected type', () => {
+    expect(a.type).toBe('CHALLENGE/GET_SUBMISSIONS_INIT');
+  });
+
+  test('payload is challengeId', () =>
+    expect(a.payload).toBe('12345'));
+});
+
+describe('challenge.getDetailsDone', () => {
+  global.fetch = mockFetch({ result: { content: ['DUMMY DATA'] } });
+
+  const a = actions.challenge.getDetailsDone(12345);
+
+  test('has expected type', () => {
+    expect(a.type).toBe('CHALLENGE/GET_DETAILS_DONE');
+  });
+
+  const mockChallenge =
+    require('../../src/services/__mocks__/data/challenge-normalized.json');
+
+  test('payload is a promise which resolves to the expected object', () =>
+    a.payload.then((res) => {
+      res.communities = Array.from(res.communities);
+      expect(res).toEqual(mockChallenge);
+    }));
+});
+
+describe('challenge.fetchSubmissionsDone', () => {
+  global.fetch = mockFetch({
+    challengeId: '12345',
+    submissions: 'DUMMY DATA',
+  });
+
+  const a = actions.challenge.getSubmissionsDone(12345, {});
+
+  test('has expected type', () => {
+    expect(a.type).toBe('CHALLENGE/GET_SUBMISSIONS_DONE');
+  });
+
+  test('payload is a promise which resolves to the expected object', () =>
+    a.payload.then(res => expect(res).toEqual({
+      challengeId: '12345',
+      submissions: 'DUMMY DATA',
+    })));
+});

Diff for: __tests__/actions/smp.js

@@ -0,0 +1,41 @@
+import { actions } from '../../src';
+
+let originalFetch;
+
+beforeAll(() => {
+  originalFetch = global.fetch;
+});
+
+afterAll(() => {
+  global.fetch = originalFetch;
+});
+
+describe('smp.deleteSubmissionDone', () => {
+  global.fetch = jest.fn(() => Promise.resolve());
+
+  const a = actions.smp.deleteSubmissionDone('Token V3', 'submissionId');
+
+  test('has expected type', () => {
+    expect(a.type).toBe('SMP/DELETE_SUBMISSION_DONE');
+  });
+
+  test('Calls the correct endpoint', () => {
+    expect(global.fetch).toHaveBeenCalledWith('https://api.topcoder-dev.com/v3/submissions/submissionId', {
+      headers: {
+        Authorization: 'Bearer Token V3',
+        'Content-Type': 'application/json',
+      },
+      method: 'DELETE',
+    });
+  });
+
+  test('payload be submissionId', () =>
+    a.payload.then(res => expect(res).toEqual('submissionId')));
+});
+
+describe('smp.downloadSubmission', () => {
+  test('does not throw', () => {
+    expect(() =>
+      actions.smp.downloadSubmission({}, 'design', '12345')).not.toThrow();
+  });
+});