Add showDiff to PropTypes. Switch if/else class assignment to ternary

Susurrus-LLC · Dec 31, 2018 · 75fcd02 · 75fcd02
1 parent f11d493
commit 75fcd02
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/routes/Morph/MorphResults.js b/src/routes/Morph/MorphResults.js
@@ -21,11 +21,9 @@ const MorphResults = props => {
     }
 
     const classes = result => {
-      if (result.changed && props.showDiff) {
-        return classNames(props.styles.outText, props.styles.changed)
-      } else {
-        return props.styles.outText
-      }
+      return result.changed && props.showDiff
+        ? classNames(props.styles.outText, props.styles.changed)
+        : props.styles.outText
     }
 
     return resultsArr.length > 0 ? (
@@ -91,7 +89,8 @@ MorphResults.propTypes = {
       output: PropTypes.string.isRequired,
       changed: PropTypes.bool.isRequired
     })
-  )
+  ),
+  showDiff: PropTypes.bool.isRequired
 }
 
 export default injectSheet(sharedResultsStyles)(MorphResults)
diff --git a/src/routes/Morph/MorphResults.test.js b/src/routes/Morph/MorphResults.test.js
@@ -20,6 +20,7 @@ describe('MorphResults', () => {
         styles={classes}
         outputFormat={defData.outputFormat}
         results={results}
+        showDiff={defData.showDiff}
       />,
       div
     )