From 253e0c6712f66447fdfbdda70a236cc6a2a08eb3 Mon Sep 17 00:00:00 2001
From: Manuel Holtgrewe <manuel.holtgrewe@bih-charite.de>
Date: Tue, 27 Sep 2022 14:12:38 +0200
Subject: [PATCH] WIP

---
 .../components/SubmissionCaseListEntry.vue    |  19 +-
 .../clinvarExport/SubmissionCaseList.spec.js  |   2 +-
 .../SubmissionCaseListEntry.spec.js           | 228 ++++++++++++++++++
 .../unit/data/firstSubmissionIndividual.json  |   2 +-
 4 files changed, 247 insertions(+), 4 deletions(-)
 create mode 100644 varfish/vueapp/tests/unit/clinvarExport/SubmissionCaseListEntry.spec.js

diff --git a/varfish/vueapp/src/components/SubmissionCaseListEntry.vue b/varfish/vueapp/src/components/SubmissionCaseListEntry.vue
index 20ccbd6bc..dc373a002 100644
--- a/varfish/vueapp/src/components/SubmissionCaseListEntry.vue
+++ b/varfish/vueapp/src/components/SubmissionCaseListEntry.vue
@@ -2,7 +2,7 @@
   <b-card header-tag="header">
     <template #header>
       <h4 class="mb-0 ml-0">
-        Case &raquo;{{ individual.name }}&laquo;
+        Case &raquo;{{ individual ? individual.name : 'null' }}&laquo;
         <b-button-group class="float-right pb-0 mb-0">
           <b-button
             size="sm"
@@ -246,6 +246,9 @@ function generateVuexVuelidateWrappers(keys) {
       key,
       {
         get() {
+          if (!this.value) {
+            return null
+          }
           return this.submissionIndividuals[this.value.sodar_uuid][key]
         },
         set(value) {
@@ -296,6 +299,9 @@ export default {
 
     phenotypes: {
       get() {
+        if (!this.value) {
+          return []
+        }
         return this.submissionIndividuals[this.value.sodar_uuid].phenotypes
       },
       set(value) {
@@ -309,6 +315,9 @@ export default {
 
     citations: {
       get() {
+        if (!this.value) {
+          return ''
+        }
         const submissionIndividual =
           this.submissionIndividuals[this.value.sodar_uuid]
         return submissionIndividual.citations.join(' ')
@@ -326,7 +335,7 @@ export default {
     },
 
     individual() {
-      if (!this.individuals) {
+      if (!this.value || !this.individuals) {
         return null
       } else {
         return this.individuals[this.value.individual]
@@ -373,6 +382,9 @@ export default {
      * @return {boolean} whether moving into the given direction is currently disabled
      */
     isMoveDisabled(up) {
+      if (!this.value) {
+        return true
+      }
       const other = this.value.sort_order + (up ? -1 : 1)
       return (
         other < 0 ||
@@ -411,6 +423,9 @@ export default {
      * Validate form input state for the given property.
      */
     validateState(name) {
+      if (!this.value) {
+        return null
+      }
       const submissionIndividual =
         this.submissionIndividuals[this.value.sodar_uuid]
       this.updateSubmissionIndividual({
diff --git a/varfish/vueapp/tests/unit/clinvarExport/SubmissionCaseList.spec.js b/varfish/vueapp/tests/unit/clinvarExport/SubmissionCaseList.spec.js
index 5fcfaf6ca..e71c97da6 100644
--- a/varfish/vueapp/tests/unit/clinvarExport/SubmissionCaseList.spec.js
+++ b/varfish/vueapp/tests/unit/clinvarExport/SubmissionCaseList.spec.js
@@ -29,7 +29,7 @@ localVue.use(Vuex)
 // Mock out the clinvarExport API
 jest.mock('@/api/clinvarExport')
 
-describe('SubmissionEditor.vue', () => {
+describe('SubmissionCaseList.vue', () => {
   let store
   let actions
 
diff --git a/varfish/vueapp/tests/unit/clinvarExport/SubmissionCaseListEntry.spec.js b/varfish/vueapp/tests/unit/clinvarExport/SubmissionCaseListEntry.spec.js
new file mode 100644
index 000000000..1d71ed444
--- /dev/null
+++ b/varfish/vueapp/tests/unit/clinvarExport/SubmissionCaseListEntry.spec.js
@@ -0,0 +1,228 @@
+import { createLocalVue, mount } from '@vue/test-utils'
+import BootstrapVue from 'bootstrap-vue'
+import Vue from 'vue'
+import Vuex from 'vuex'
+
+import clinvarExportApi from '@/api/clinvarExport'
+import SubmissionCaseListEntry from '@/components/SubmissionCaseListEntry.vue'
+import {
+  actions,
+  mutations,
+  WizardState,
+} from '@/store/modules/clinvarExport.js'
+
+import { copy } from '../../testUtils.js'
+import {
+  clinvarExportEmptyState,
+  firstIndividual,
+  firstOrganisation,
+  firstSubmission,
+  firstSubmissionIndividual,
+  firstSubmissionSet,
+  firstSubmitter,
+  firstSubmittingOrg,
+  rawAppContext,
+  secondIndividual,
+  secondSubmissionIndividual,
+} from '../fixtures.js'
+
+// Set up extended Vue constructor
+const localVue = createLocalVue()
+localVue.use(BootstrapVue)
+localVue.use(Vuex)
+
+// Mock out the clinvarExport API
+jest.mock('@/api/clinvarExport')
+
+describe('SubmissionCaseListEntry.vue', () => {
+  let store
+
+  beforeAll(() => {
+    // Disable warnings
+    jest.spyOn(console, 'warn').mockImplementation(jest.fn())
+  })
+
+  beforeEach(() => {
+    // Setup relevant store/state fragment
+    const clinvarExport = {
+      namespaced: true,
+      actions,
+      mutations,
+      state: () => copy(clinvarExportEmptyState),
+    }
+    store = new Vuex.Store({
+      modules: {
+        clinvarExport,
+      },
+    })
+    store.state.clinvarExport.appContext = copy(rawAppContext)
+  })
+
+  // In these tests we consider the simple case of having one submission
+  // set only and to initially be in the submission set state
+  let organisation1
+  let submitter1
+  let submittingOrg1
+  let submissionSet1
+  let submission1
+  let submissionIndividual1
+  let submissionIndividual2
+  let individual1
+  let individual2
+  const setupSimpleCase = () => {
+    organisation1 = copy(firstOrganisation)
+    Vue.set(
+      store.state.clinvarExport.organisations,
+      organisation1.sodar_uuid,
+      organisation1
+    )
+    submitter1 = copy(firstSubmitter)
+    Vue.set(
+      store.state.clinvarExport.submitters,
+      submitter1.sodar_uuid,
+      submitter1
+    )
+    submittingOrg1 = copy(firstSubmittingOrg)
+    Vue.set(
+      store.state.clinvarExport.submittingOrgs,
+      submittingOrg1.sodar_uuid,
+      submittingOrg1
+    )
+    submission1 = copy(firstSubmission)
+    Vue.set(
+      store.state.clinvarExport.submissions,
+      submission1.sodar_uuid,
+      submission1
+    )
+    Vue.set(store.state.clinvarExport, 'currentSubmission', submission1)
+    submissionSet1 = copy(firstSubmissionSet)
+    Vue.set(store.state.clinvarExport, 'wizardState', WizardState.submissionSet)
+    Vue.set(
+      store.state.clinvarExport.submissionSets,
+      submissionSet1.sodar_uuid,
+      submissionSet1
+    )
+    submissionIndividual1 = copy(firstSubmissionIndividual)
+    Vue.set(
+      store.state.clinvarExport.submissionIndividuals,
+      submissionIndividual1.sodar_uuid,
+      submissionIndividual1
+    )
+    submissionIndividual2 = copy({
+      ...secondSubmissionIndividual,
+      submission: submission1.sodar_uuid,
+      sort_order: 1,
+    })
+    Vue.set(
+      store.state.clinvarExport.submissionIndividuals,
+      submissionIndividual2.sodar_uuid,
+      submissionIndividual2
+    )
+    Vue.set(
+      store.state.clinvarExport.currentSubmission,
+      'submission_individuals',
+      [submissionIndividual1.sodar_uuid, submissionIndividual2.sodar_uuid]
+    )
+    individual1 = copy(firstIndividual)
+    Vue.set(
+      store.state.clinvarExport.individuals,
+      individual1.sodar_uuid,
+      individual1
+    )
+    individual2 = copy(secondIndividual)
+    Vue.set(
+      store.state.clinvarExport.individuals,
+      individual2.sodar_uuid,
+      individual2
+    )
+    Vue.set(store.state.clinvarExport, 'submissionSetList', [submissionSet1])
+    Vue.set(store.state.clinvarExport, 'currentSubmissionSet', submissionSet1)
+    Vue.set(store.state.clinvarExport, 'wizardState', WizardState.submissions)
+  }
+
+  afterEach(() => {
+    Object.keys(clinvarExportApi).forEach((method) =>
+      clinvarExportApi[method].mockClear()
+    )
+  })
+
+  const testSetup = async (siNo) => {
+    await setupSimpleCase()
+
+    const wrapper = mount(
+      {
+        data: () => {
+          return {
+            submissionIndividual:
+              siNo === 2 ? submissionIndividual2 : submissionIndividual1,
+          }
+        },
+        template:
+          '<div><submission-case-list-entry ref="listEntry" v-model="submissionIndividual">' +
+          '</submission-case-list-entry></div>',
+        components: { SubmissionCaseListEntry },
+      },
+      {
+        store,
+        localVue,
+      }
+    )
+    return wrapper.vm.$refs.listEntry
+  }
+
+  test('check computed properties', async () => {
+    const submissionCaseListEntry = await testSetup(1)
+
+    expect(submissionCaseListEntry.hpoTermsLoading).toBe(false)
+
+    expect(submissionCaseListEntry.appContext).toEqual(
+      store.state.clinvarExport.appContext
+    )
+    expect(submissionCaseListEntry.currentSubmission).toEqual(
+      store.state.clinvarExport.currentSubmission
+    )
+    expect(submissionCaseListEntry.individuals).toEqual(
+      store.state.clinvarExport.individuals
+    )
+    expect(submissionCaseListEntry.submissionIndividuals).toEqual(
+      store.state.clinvarExport.submissionIndividuals
+    )
+
+    expect(submissionCaseListEntry.phenotypes).toEqual([
+      { term_id: 'HP:1234567', term_name: 'Something' },
+    ])
+    expect(submissionCaseListEntry.citations).toEqual('PMID:12345')
+    expect(submissionCaseListEntry.individual).toEqual(individual1)
+  })
+
+  test('check funcions', async () => {
+    const submissionCaseListEntry = await testSetup(1)
+
+    expect(
+      submissionCaseListEntry.getHpoTermLabel({
+        term_id: 'HP:xxx',
+        term_name: 'short',
+      })
+    ).toEqual('HP:xxx - short')
+    expect(
+      submissionCaseListEntry.getHpoTermLabel({
+        term_id: 'HP:xxx',
+        term_name: 'longlonglonglonglong',
+      })
+    ).toEqual('HP:xxx - longlonglo...')
+  })
+
+  test('check isMoveDisabled()', async () => {
+    const submissionCaseListEntry = await testSetup(1)
+
+    expect(submissionCaseListEntry.isMoveDisabled(true)).toBe(true)
+    expect(submissionCaseListEntry.isMoveDisabled(false)).toBe(false)
+  })
+
+  test('check isMoveDisabled()', async () => {
+    const submissionCaseListEntry = await testSetup(2)
+
+    expect(submissionCaseListEntry.isMoveDisabled(true)).toBe(false)
+    expect(submissionCaseListEntry.isMoveDisabled(false)).toBe(true)
+  })
+})
diff --git a/varfish/vueapp/tests/unit/data/firstSubmissionIndividual.json b/varfish/vueapp/tests/unit/data/firstSubmissionIndividual.json
index 8976fd6c2..6b8e991aa 100644
--- a/varfish/vueapp/tests/unit/data/firstSubmissionIndividual.json
+++ b/varfish/vueapp/tests/unit/data/firstSubmissionIndividual.json
@@ -2,7 +2,7 @@
   "sodar_uuid": "99999999-9999-9999-9999-999999999999",
   "date_created": "2020-11-09 13:37",
   "date_modified": "2020-11-09 13:37",
-  "sort_order": 1,
+  "sort_order": 0,
   "individual": "88888888-8888-8888-8888-888888888888",
   "submission": "66666666-6666-6666-6666-666666666666",
   "phenotypes": [{ "term_id": "HP:1234567", "term_name": "Something" }],