Build fixes

Author: jmgasper

.eslintrc

@@ -4,9 +4,12 @@
     "jsx-a11y/anchor-is-valid": false,
     "import/no-cycle": [2, { "maxDepth": 1 }],
     "react/forbid-prop-types": false,
-    "react/no-unknown-property": false
+    "react/no-unknown-property": false,
+    "react/jsx-no-bind": false,
+    "react/destructuring-assignment": false,
+    "react/no-array-index-key": false
   },
   "env": {
     "browser": true
-  }  
+  }
 }

__tests__/shared/components/tc-communities/ArticleCard.jsx

@@ -46,7 +46,7 @@ test('render properly', () => {
   TU.renderIntoDocument((
     <Wrapper
       title="How Does An IOS 10 LCD Work"
-      text={'<p>There are advances being made in science and technology everyday</p>'}
+      text="<p>There are advances being made in science and technology everyday</p>"
       imageSrc="/themes/wipro/home/news-01.jpg"
       link={{
         title: 'Read More',
@@ -67,7 +67,7 @@ test('render properly', () => {
   TU.renderIntoDocument((
     <Wrapper
       title="How Does An IOS 10 LCD Work"
-      text={'<p>There are advances being made in science and technology everyday</p>'}
+      text="<p>There are advances being made in science and technology everyday</p>"
       imageSrc="/themes/wipro/home/news-01.jpg"
       link={{
         title: 'Read More',

config/jest/setup.js

@@ -26,5 +26,15 @@ global.window.resizeTo = (width, height) => {
   global.window.innerHeight = height || global.window.innerHeight;
   global.window.dispatchEvent(new Event('resize'));
 };
+/* eslint-disable lines-between-class-members */
+/* eslint-disable no-empty-function */
+/* eslint-disable no-unused-vars */
+/* eslint-disable class-methods-use-this */
+/* eslint-disable no-useless-constructor */
+global.MutationObserver = class {
+  constructor(callback) {}
+  disconnect() {}
+  observe(element, initObject) {}
+};
 
 jest.mock('services/money');

package.json

@@ -208,7 +208,7 @@
     "eslint-plugin-import": "^2.13.0",
     "eslint-plugin-jest": "^21.17.0",
     "eslint-plugin-jsx-a11y": "^6.0.3",
-    "eslint-plugin-react": "^7.10.0",
+    "eslint-plugin-react": "^7.32.2",
     "file-loader": "^1.1.11",
     "happypack": "^5.0.1",
     "husky": "^4.2.1",

src/client/index.jsx

@@ -14,6 +14,7 @@ import {
 } from '@topcoder-platform/tc-auth-lib';
 import { actions, logger, errors } from 'topcoder-react-lib';
 import { client, redux, config } from 'topcoder-react-utils';
+// eslint-disable-next-line import/no-unresolved
 import { sprig } from '@sprig-technologies/sprig-browser';
 
 import './styles.scss';

src/shared/components/Contentful/ContentSlider/ContentSliderItem.jsx

@@ -41,6 +41,11 @@ function ContentSliderItem(props) {
   );
 }
 
+ContentSliderItem.defaultProps = {
+  spaceName: '',
+  environment: '',
+};
+
 ContentSliderItem.propTypes = {
   itemId: PT.string.isRequired,
   type: PT.string.isRequired,

src/shared/components/Contentful/SearchBar/SearchBar.jsx

@@ -203,6 +203,7 @@ export class SearchBarInner extends Component {
                       className={theme.articleLink}
                       href={(item.externalArticle && item.contentUrl) ? item.contentUrl : `${config.TC_EDU_BASE_PATH}${config.TC_EDU_ARTICLES_PATH}/${item.slug || item.title}`}
                       target={(item.externalArticle && item.contentUrl) ? '_blank' : '_self'}
+                      rel="noreferrer"
                       onClick={(e) => {
                         e.nativeEvent.stopImmediatePropagation();
                         e.stopPropagation();
@@ -254,6 +255,7 @@ export class SearchBarInner extends Component {
                       className={theme.articleLink}
                       href={(item.externalArticle && item.contentUrl) ? item.contentUrl : `${config.TC_EDU_BASE_PATH}${config.TC_EDU_ARTICLES_PATH}/${item.slug || item.title}`}
                       target={(item.externalArticle && item.contentUrl) ? '_blank' : '_self'}
+                      rel="noreferrer"
                       onClick={(e) => {
                         e.nativeEvent.stopImmediatePropagation();
                         e.stopPropagation();
@@ -300,6 +302,7 @@ export class SearchBarInner extends Component {
                       className={theme.forumLink}
                       href={(item.externalArticle && item.contentUrl) ? item.contentUrl : `${config.TC_EDU_BASE_PATH}${config.TC_EDU_ARTICLES_PATH}/${item.slug || item.title}`}
                       target={(item.externalArticle && item.contentUrl) ? '_blank' : '_self'}
+                      rel="noreferrer"
                     >
                       <span className={theme['cell-text']}>
                         {item.title}

src/shared/components/Leaderboard/PodiumSpot/index.jsx

@@ -1,3 +1,4 @@
+/* eslint-disable react/prop-types */
 /**
  * Leaderboard PodiumSpot
  *

src/shared/components/SubmissionPage/Submit/index.jsx

@@ -339,6 +339,7 @@ const filestackDataProp = PT.shape({
   container: PT.string.isRequired,
   challengeId: PT.string.isRequired,
   fileUrl: PT.string.isRequired,
+  fileType: PT.string,
 });
 
 /**

src/shared/components/challenge-listing/ChallengeCard/Status/index.jsx

@@ -155,6 +155,7 @@ export default function ChallengeStatus(props) {
         onClick={() => false}
         styleName="register-button"
         target={openChallengesInNewTabs ? '_blank' : undefined}
+        rel="noreferrer"
       >
         { timeDiff.late
           ? (

src/shared/components/challenge-listing/Tooltips/ProgressBarTooltip/index.jsx

@@ -175,7 +175,7 @@ Tip.defaultProps = {
 };
 
 Tip.propTypes = {
-  challenge: PT.shape({}).isRequired,
+  challenge: PT.object.isRequired,
   isLoaded: PT.bool,
 };
 

src/shared/containers/SubmissionPage.jsx

@@ -19,6 +19,7 @@ import { connect } from 'react-redux';
 import SubmissionsPage from 'components/SubmissionPage';
 import AccessDenied, { CAUSE as ACCESS_DENIED_REASON } from 'components/tc-communities/AccessDenied';
 import LoadingIndicator from 'components/LoadingIndicator';
+// eslint-disable-next-line import/no-unresolved
 import { sprig } from '@sprig-technologies/sprig-browser';
 
 export const Sprig = sprig.configure({

src/shared/routes/Communities/ChallengeListing.jsx

@@ -47,11 +47,9 @@ export default function ChallengeListingRoute({
               extraBucket={extraBucket}
               groupIds={meta.groupIds}
               hideSrm={hideSrm}
-
               /* TODO: This is hacky! A better, generic way to achieve it
                * should be adopted. */
               hideTcLinksInSidebarFooter={meta.communityId === 'wipro'}
-
               listingOnly={listingOnly}
               newChallengeDetails={newChallengeDetails}
               openChallengesInNewTabs={