Feature #83: Do not display set status for goals (#86)

* Do not display set status for goals

* Add tests to logger service

app/components/ItemActions/__tests__/presenters.test.js

@@ -50,6 +50,28 @@ describe('ItemActions presenters', () => {
     expect(window.console.error.mock.calls).toEqual([])
   })
 
+  test('do not display set status for goals', () => {
+    ownProps = { ...ownProps, type: 'goal' }
+
+    props = {
+      ...ownProps,
+      ...mapStateToProps(state, ownProps),
+      ...mapDispatchToProps(dispatch, ownProps)
+    }
+
+    expect(props.shouldDisplaySetStatus).toEqual(false)
+
+    ownProps = { ...ownProps, type: 'task' }
+
+    props = {
+      ...ownProps,
+      ...mapStateToProps(state, ownProps),
+      ...mapDispatchToProps(dispatch, ownProps)
+    }
+
+    expect(props.shouldDisplaySetStatus).toEqual(true)
+  })
+
   test('provide correct on delete callback', () => {
     ownProps = { ...ownProps, type: 'goal' }
     props = {

app/components/ItemActions/presenters.js

@@ -10,7 +10,7 @@ export const mapStateToProps = (state, ownProps) => {
     task: tasksSelectors.getById
   }[type](itemId)(state)
   const itemStatus = (item && item.status) || ''
-  const shouldDisplaySetStatus = tasksSelectors.isInnermost(state, itemId)
+  const shouldDisplaySetStatus = type === 'task' && tasksSelectors.isInnermost(state, itemId)
 
   return {
     shouldDisplaySetStatus,

app/services/logger/__tests__/logger.test.js

@@ -0,0 +1,23 @@
+/* eslint-env jest */
+
+import logger from 'ss/services/logger'
+
+describe('logger services', () => {
+  beforeEach(() => {
+    jest.spyOn(window.console, 'log')
+    window.console.log.mockImplementation(() => {})
+    window.console.log.mockClear()
+  })
+
+  test('log using console log', () => {
+    logger.log('a')
+
+    expect(window.console.log.mock.calls).toEqual([['a']])
+  })
+
+  test('support variable arguments', () => {
+    logger.log('a', 'b', 1, [], {})
+
+    expect(window.console.log.mock.calls).toEqual([['a', 'b', 1, [], {}]])
+  })
+})