FIX: Some imports / exports had wrong syntax, not supported by Babel 7

Author: birdofpreyru

CHANGELOG.md

@@ -3,7 +3,8 @@
 ### V0.9.0
 - NodeJS updated to the latest LTS version (10.14.2);
 - NPM dependencies updated to the latest versions. In particular:
-  - Babel updated from v6 to v7.
+  - Babel updated from v6 to v7;
+  - Misc related fixes.
 
 ### v0.8.2
 - Adds [**`Modal`**](docs/modal.md) component.

__tests__/shared/utils/__snapshots__/config.js.snap

@@ -0,0 +1,10 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Base test 1`] = `
+Config {
+  "SECRET": Object {
+    "dummySecretKey": "Dummy Secret Value",
+  },
+  "dummyKey": "Dummy Value",
+}
+`;

__tests__/shared/utils/config.js

@@ -1,32 +1,38 @@
 /* eslint-env browser */
 
 import _ from 'lodash';
+import config from 'config';
 import { isClientSide, isServerSide } from 'utils/isomorphy';
 
-const CLIENT_SIDE_CONFIG = {
-  TYPE: 'CLIENT_SIDE_CONFIG',
-};
-
-const SERVER_SIDE_CONFIG = {
-  TYPE: 'SERVER_SIDE_CONFIG',
-};
-
-jest.setMock('config', _.clone(SERVER_SIDE_CONFIG));
-
-beforeEach(() => {
-  jest.resetModules();
-  window.CONFIG = _.clone(CLIENT_SIDE_CONFIG);
-});
-
-afterEach(() => delete window.TRU_FRONT_END);
-
-test('Serves injected config at the client side', () => {
-  window.TRU_FRONT_END = true;
-  expect(isClientSide()).toBe(true);
-  expect(require('utils/config')).toEqual(CLIENT_SIDE_CONFIG);
+test('Base test', () => {
+  expect(config).toMatchSnapshot();
 });
 
-test('Serves node-config at the server side', () => {
-  expect(isServerSide()).toBe(true);
-  expect(require('utils/config')).toEqual(SERVER_SIDE_CONFIG);
+describe('Isomorphy behavior tests', () => {
+  const CLIENT_SIDE_CONFIG = {
+    TYPE: 'CLIENT_SIDE_CONFIG',
+  };
+
+  const SERVER_SIDE_CONFIG = {
+    TYPE: 'SERVER_SIDE_CONFIG',
+  };
+
+  beforeEach(() => {
+    jest.resetModules();
+    jest.setMock('config', _.clone(SERVER_SIDE_CONFIG));
+    window.CONFIG = _.clone(CLIENT_SIDE_CONFIG);
+  });
+
+  afterEach(() => delete window.TRU_FRONT_END);
+
+  test('Serves injected config at the client side', () => {
+    window.TRU_FRONT_END = true;
+    expect(isClientSide()).toBe(true);
+    expect(require('utils/config').default).toEqual(CLIENT_SIDE_CONFIG);
+  });
+
+  test('Serves node-config at the server side', () => {
+    expect(isServerSide()).toBe(true);
+    expect(require('utils/config').default).toEqual(SERVER_SIDE_CONFIG);
+  });
 });

package-lock.json

@@ -21,7 +21,7 @@ let serverUrl = isProdBuild() ? 'prod' : 'dev';
 serverUrl = `topcoder-react-utils/dist/${serverUrl}/server`;
 const server = utils.isomorphy.isServerSide() ? requireWeak(serverUrl) : null;
 
-module.exports = {
+export {
   actions,
   AppChunk,
   Avatar,
@@ -36,5 +36,6 @@ module.exports = {
   ScalableRect,
   reducers,
   server,
-  ...utils,
 };
+
+export * from 'utils';

src/index.js

@@ -11,6 +11,5 @@ import { isClientSide } from './isomorphy';
 import { requireWeak } from './webpack';
 
 /* eslint-disable global-require */
-// console.log('IS CLIENT SIDE', isClientSide());
-module.exports = isClientSide() ? window.CONFIG : requireWeak('config');
+export default isClientSide() ? window.CONFIG : requireWeak('config');
 /* eslint-enable global-require */

src/shared/utils/config.js

@@ -11,9 +11,7 @@ import { connect } from 'react-redux';
 import { isDevBuild } from './isomorphy';
 
 /* Auxiliary aliases. */
-module.exports.connect = connect;
-module.exports.createActions = createActions;
-module.exports.handleActions = handleActions;
+export { connect, createActions, handleActions };
 
 /**
  * Reduce multiple reducers into a single reducer from left to right.