Remove unused code & tidy

src/mixins/graphql.js

@@ -1,6 +1,4 @@
-import { mapState } from 'vuex'
-
-/**
+/*
  * Copyright (C) NIWA & British Crown (Met Office) & Contributors.
  *
  * This program is free software: you can redistribute it and/or modify
@@ -17,15 +15,18 @@ import { mapState } from 'vuex'
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+import { mapState } from 'vuex'
+
 /**
  * A mixin that contains data used for a GraphQL subscription, such as the
  * query variables.
  *
- * To be used in Views that are bound to VueRouter routes that contain the
+ * To be used in Views that are bound to Vue-Router routes that contain the
  * :workflowName param.
  */
 export default {
   props: {
+    /** This is set by vue-router */
     workflowName: {
       type: String,
       required: true

src/router/index.js

@@ -90,15 +90,12 @@ router.beforeEach(async (to, from) => {
 
   // Set toolbar title:
   let title = to.name
-  let workflowName = null
   if (to.meta.toolbar) {
     // When a workflow is being displayed, we set the title to a
     // different value.
     title = to.params.workflowName
-    workflowName = to.params.workflowName
   }
   store.commit('app/setTitle', title)
-  store.commit('workflows/SET_WORKFLOW_NAME', workflowName)
   store.dispatch('setAlert', null)
 })
 

src/store/workflows.module.js

@@ -33,16 +33,6 @@ const state = () => ({
     $index: {},
     children: []
   },
-  /**
-   * This holds the name of the current workflow. This is set by VueRouter
-   * and is used to decide what's the current workflow. It is used in conjunction
-   * with the workflows/workflows (above) when finding the current workflow and
-   * using it, for instance, to create the GraphQL variables of a workflow
-   * view (see mixins used in the Tree View).
-   *
-   * @type {String}
-   */
-  workflowName: null
 })
 
 const getters = {
@@ -475,10 +465,6 @@ function remove (state, prunedID) {
 }
 
 const mutations = {
-  // the old callback methods
-  SET_WORKFLOW_NAME (state, data) {
-    state.workflowName = data
-  },
   // the new cylc tree methods
   CREATE: createTree,
   UPDATE: update,

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

@@ -48,7 +48,6 @@ describe('Toolbar component', () => {
         status: WorkflowState.RUNNING.name
       }
     ]
-    store.state.workflows.workflowName = 'test'
   })
 
   it('should mount the component', async () => {