Tidy

Author: MetRonnie

src/components/cylc/table/Table.vue

@@ -60,14 +60,14 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
                   <Task
                     v-cylc-object="item.value.task"
                     :task="item.value.task.node"
-                    :startTime="((item.value.latestJob || {}).node || {}).startedTime"
+                    :startTime="item.value.latestJob?.node?.startedTime"
                   />
                 </div>
                 <div class="mr-1">
                   <Job
                     v-cylc-object="item.value.task"
                     :status="item.value.task.node.state"
-                    :previous-state="((item.value.previousJob || {}).node || {}).state"
+                    :previous-state="item.value.previousJob?.node?.state"
                   />
                 </div>
                 <div>{{ item.value.task.name }}</div>

src/store/workflows.module.js

@@ -80,7 +80,7 @@ const getters = {
     while (stack.length) {
       item = stack.pop()
       if (parentNodeTypes.includes(item.type)) {
-        // this is above "nodeTyoe" in the tree, look through its child nodes
+        // this is above "nodeType" in the tree, look through its child nodes
         stack.push(...item.children)
       } else if (
         item.type === nodeType &&
@@ -99,12 +99,11 @@ function createTree (state) {
   if (state.cylcTree) {
     return
   }
-  const tree = {
+  state.cylcTree = {
     $index: {},
-    id: '$root'
+    id: '$root',
+    children: [],
   }
-  tree.children = []
-  state.cylcTree = tree
   // console.log('@@')
 }
 

src/utils/tasks.js

@@ -20,7 +20,7 @@ import { TASK_OUTPUT_NAMES } from '@/model/TaskOutput.model'
 
 /**
  * States used when the parent is stopped.
- * @type {Array<TaskState>}
+ * @type {TaskState[]}
  */
 const isStoppedOrderedStates = [
   TaskState.SUBMIT_FAILED,
@@ -35,9 +35,9 @@ const isStoppedOrderedStates = [
 
 /**
  * Gives a single state, based on a list of states of children nodes.
- * @param childStates {Array<TaskState>} children nodes
- * @param isStopped {boolean} whether the parent node is stopped or not
- * @returns {string} a valid Task State name, or null if not found
+ * @param {TaskState[]} childStates children nodes
+ * @param {boolean} isStopped whether the parent node is stopped or not
+ * @returns {string} a valid Task State name, or empty string if not found
  * @link @see https://github.com/cylc/cylc-flow/blob/d66ae5c3ce8c749c8178d1cd53cb8c81d1560346/lib/cylc/task_state_prop.py
  */
 function extractGroupState (childStates, isStopped = false) {
@@ -51,10 +51,7 @@ function extractGroupState (childStates, isStopped = false) {
 }
 
 function latestJob (taskProxy) {
-  if (taskProxy && taskProxy.children && taskProxy.children.length > 0) {
-    return taskProxy.children[0].node
-  }
-  return null
+  return taskProxy?.children?.[0]?.node
 }
 
 /** Returns an array of task messages and custom outputs for a job node.

tests/unit/components/cylc/table/table.vue.spec.js

@@ -17,26 +17,18 @@
 
 import { mount } from '@vue/test-utils'
 import { createVuetify } from 'vuetify'
+import sinon from 'sinon'
 import { simpleTableTasks } from './table.data'
 import TaskState from '@/model/TaskState.model'
 import CylcObjectPlugin from '@/components/cylc/cylcObject/plugin'
 import Table from '@/components/cylc/table/Table.vue'
 import { VDataTable, VDataTableFooter } from 'vuetify/labs/VDataTable'
+import WorkflowService from '@/services/workflow.service'
 
 const $eventBus = {
   emit () {}
 }
-const $workflowService = {
-  register () {},
-  unregister () {},
-  subscribe () {},
-  introspection: Promise.resolve({
-    mutations: [
-      { args: [] }
-    ],
-    types: []
-  })
-}
+const $workflowService = sinon.createStubInstance(WorkflowService)
 
 const vuetify = createVuetify({
   components: { VDataTable, VDataTableFooter }

tests/unit/components/cylc/tree/tree.vue.spec.js

@@ -19,25 +19,17 @@
 import { nextTick } from 'vue'
 import { mount } from '@vue/test-utils'
 import { createVuetify } from 'vuetify'
+import sinon from 'sinon'
 import Tree from '@/components/cylc/tree/Tree.vue'
 import { simpleWorkflowTree4Nodes } from './tree.data'
 import CylcObjectPlugin from '@/components/cylc/cylcObject/plugin'
 import cloneDeep from 'lodash/cloneDeep'
+import WorkflowService from '@/services/workflow.service'
 
 const $eventBus = {
   emit () {}
 }
-const $workflowService = {
-  register () {},
-  unregister () {},
-  subscribe () {},
-  introspection: Promise.resolve({
-    mutations: [
-      { args: [] }
-    ],
-    types: []
-  })
-}
+const $workflowService = sinon.createStubInstance(WorkflowService)
 const vuetify = createVuetify()
 
 describe('Tree component', () => {

tests/unit/components/cylc/tree/treeitem.vue.spec.js

@@ -19,13 +19,15 @@
 import { mount } from '@vue/test-utils'
 import { Assertion } from 'chai'
 import { createVuetify } from 'vuetify'
+import sinon from 'sinon'
 import TreeItem from '@/components/cylc/tree/TreeItem.vue'
 import {
   simpleWorkflowNode,
   simpleCyclepointNode,
   simpleTaskNode
 } from './tree.data'
 import CylcObjectPlugin from '@/components/cylc/cylcObject/plugin'
+import WorkflowService from '@/services/workflow.service'
 
 /**
  * Helper function for expecting TreeItem to be expanded.
@@ -50,12 +52,7 @@ Assertion.addMethod('expanded', function () {
   )
 })
 
-const $workflowService = {
-  introspection: Promise.resolve({
-    mutations: [],
-    types: []
-  })
-}
+const $workflowService = sinon.createStubInstance(WorkflowService)
 const $eventBus = {
   emit: () => {}
 }

tests/unit/utils/tasks.spec.js

@@ -56,37 +56,33 @@ describe('tasks', () => {
       expect(extractGroupState([])).to.equal('')
     })
   })
-  describe('latestJob', () => {
-    it('should return the correct value for latestJob', () => {
-      const tests = [
-        {
-          taskProxy: null,
-          expected: null
-        },
-        {
-          taskProxy: {},
-          expected: null
-        },
-        {
-          taskProxy: {
-            children: []
-          },
-          expected: null
-        },
-        {
-          taskProxy: {
-            children: [
-              {
-                node: 1
-              }
-            ]
-          },
-          expected: 1
-        }
-      ]
-      tests.forEach(test => {
-        expect(latestJob(test.taskProxy)).to.equal(test.expected)
-      })
+  describe.each([
+    {
+      taskProxy: null,
+      expected: undefined
+    },
+    {
+      taskProxy: {},
+      expected: undefined
+    },
+    {
+      taskProxy: {
+        children: []
+      },
+      expected: undefined
+    },
+    {
+      taskProxy: {
+        children: [
+          { node: 'foo' },
+          { node: 'bar' },
+        ]
+      },
+      expected: 'foo'
+    }
+  ])('latestJob($taskProxy)', ({ taskProxy, expected }) => {
+    it(`returns ${expected}`, () => {
+      expect(latestJob(taskProxy)).to.equal(expected)
     })
   })
   describe('dtMean', () => {