fixed jest unit tests

Author: Colinh84

__tests__/shared/components/__snapshots__/Content.jsx.snap

@@ -374,6 +374,19 @@ exports[`Matches shallow shapshot 1`] = `
       </code>
       .
     </li>
+    <li>
+      <Link
+        replace={false}
+        to="/examples/tags/"
+      >
+        Tags
+      </Link>
+       - Demo/test of standard tags already available in the code, and customizable with help of 
+      <code>
+        react-css-themr
+      </code>
+      .
+    </li>
     <li>
       <Link
         replace={false}

__tests__/shared/components/challenge-listing/ChallengeCard/index.jsx

@@ -1,10 +1,12 @@
 import React from 'react';
+import ReactDOM from 'react-dom';
 import Renderer from 'react-test-renderer/shallow';
 import TU from 'react-dom/test-utils';
 import ChallengeCard from 'components/challenge-listing/ChallengeCard';
 import MockDate from 'mockdate';
 import { Provider } from 'react-redux';
 import mockReduxStore from 'redux-mock-store';
+import { Tag } from 'components/tags';
 
 const store = mockReduxStore()();
 
@@ -152,16 +154,16 @@ describe('render properly', () => {
 
   test('click', () => {
     instance = TU.renderIntoDocument((<Wrapper {...mockData4} />));
-    const tags = TU.findAllInRenderedTree(instance, item =>
-      item && item.className && item.className.match('technology'));
-    expect(tags).toHaveLength(4);
-    TU.Simulate.click(tags[0]);
+    const tags = TU.scryRenderedComponentsWithType(instance, Tag);
+    const el = ReactDOM.findDOMNode(tags[0]);
+    expect(tags).toHaveLength(4);    
+    TU.Simulate.click(el);
   });
 
   test('click + tag', () => {
     instance = TU.renderIntoDocument((<Wrapper {...mockData5} />));
-    const tags = TU.findAllInRenderedTree(instance, item =>
-      item && item.className && item.className.match('technology'));
-    TU.Simulate.click(tags[0]);
+    const tags = TU.scryRenderedComponentsWithType(instance, Tag);
+    const el = ReactDOM.findDOMNode(tags[0]);
+    TU.Simulate.click(el);
   });
 });

__tests__/shared/components/challenge-listing/Listing/Bucket.jsx

@@ -1,10 +1,12 @@
 import React from 'react';
+import ReactDOM from 'react-dom';
 import _ from 'lodash';
 import Renderer from 'react-test-renderer/shallow';
 import TU from 'react-dom/test-utils';
 import Bucket from 'components/challenge-listing/Listing/Bucket';
 import reduxStoreFactory from 'redux-mock-store';
 import { Provider } from 'react-redux';
+import { Tag } from 'components/tags';
 
 const store = reduxStoreFactory()();
 
@@ -176,14 +178,13 @@ class Wrapper extends React.Component {
 const instance = TU.renderIntoDocument((<Wrapper {...mockDatas[0]} />));
 
 test('setFilterState', () => {
-  const matches = TU.findAllInRenderedTree(instance, item =>
-    item && item.className && item.className.match('technology'));
+  const matches = TU.scryRenderedComponentsWithType(instance, Tag);
   expect(matches).toHaveLength(3);
-  TU.Simulate.click(matches[0]);
+  TU.Simulate.click(ReactDOM.findDOMNode(matches[0]));
   expect(setFilterState).toHaveBeenCalledTimes(1);
-  TU.Simulate.click(matches[1]);
+  TU.Simulate.click(ReactDOM.findDOMNode(matches[1]));
   expect(setFilterState).toHaveBeenCalledTimes(2);
-  TU.Simulate.click(matches[2]);
+  TU.Simulate.click(ReactDOM.findDOMNode(matches[2]));
   expect(setFilterState).toHaveBeenCalledTimes(3);
 });
 

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

@@ -232,6 +232,7 @@ class Tags extends React.Component {
         <Tag
           onClick={() => this.onClick(c.trim())}
           key={c}
+          role="button"
         >{c}</Tag>
       ));
     }

src/shared/components/tags/index.jsx

@@ -16,6 +16,8 @@ export function GenericTag({
   onClick,
   openNewTab,
   replace,
+  role,
+  tabIndex,
   theme,
   to,
 }) {
@@ -35,8 +37,8 @@ export function GenericTag({
     <a
       className={`${theme.tag}`}
       onClick={onClick}
-      role="button"
-      tabIndex={0}
+      role={role}
+      tabIndex={tabIndex}
     >{children}</a>
   );
 }
@@ -47,6 +49,8 @@ GenericTag.defaultProps = {
   onClick: null,
   openNewTab: false,
   replace: false,
+  role: "button",
+  tabIndex: 0,
   to: null,
 };
 
@@ -56,6 +60,8 @@ GenericTag.propTypes = {
   onClick: PT.func,
   openNewTab: PT.bool,
   replace: PT.bool,
+  role: PT.string,
+  tabIndex: PT.number,
   theme: PT.shape({
     tag: PT.string.isRequired,
   }).isRequired,