Bugfix that allow clinvar export submission set deletion (#713).

Closes: #713
Related-Issue: #713
Projected-Results-Impact: none

HISTORY.rst

@@ -129,6 +129,7 @@ Full Change List
 - Changing ClinVar link-out to VCV entry instead of coordinates (#693).
 - Adding unit test for clinvar Vue app (#692).
 - Moving clinvar Vue app (#711).
+- Bugfix that allow clinvar export submission set deletion (#713).
 
 ------
 v1.2.0

clinvar_export/vueapp/src/api/clinvarExport.js

@@ -49,7 +49,6 @@ async function apiDelete(entity, uuid, appContext) {
       `Problem with request: ${response.status} ${response.statusText}`
     )
   }
-  return await response.json()
 }
 
 /**

clinvar_export/vueapp/src/store/modules/clinvarExport.js

@@ -645,9 +645,9 @@ export function extractVariantZygosity(smallVariant, individualUuids, state) {
 
   // See whether any individual is annotated as recessive.
   let anyRecessive = false
-  let variantAlleleCount = 0
-  let variantZygosity = 'not provided'
-  if (smallVariant) {
+  let variantAlleleCount = null
+  let variantZygosity = null
+  if (smallVariant !== null && smallVariant !== undefined) {
     let individual = null
     for (const individualUuid of individualUuids) {
       const currIndividual = state.individuals[individualUuid]
@@ -695,6 +695,12 @@ export function extractVariantZygosity(smallVariant, individualUuids, state) {
       }
     }
   }
+  if (variantAlleleCount === null) {
+    variantAlleleCount = 0
+  }
+  if (variantZygosity === null) {
+    variantZygosity = 'not provided'
+  }
   return { variantAlleleCount, variantZygosity }
 }
 

clinvar_export/vueapp/tests/unit/clinvarExport/api.spec.js

@@ -211,7 +211,7 @@ describe('api', () => {
       `${appContext.baseUrl}/${entityNoUnder}/`,
       expectedJsonDeleteArgs,
     ])
-    expect(res).toEqual({ fake: 'result' })
+    expect(res).toBeUndefined()
   })
 
   test('getUserAnnotations', async () => {