Test numeric annotations or mouse not eligible for pathways

Author: eweitz

app/javascript/components/explore/ExploreDisplayTabs.jsx

@@ -523,7 +523,7 @@ export default function ExploreDisplayTabs({
     queries = exploreParams.genes
   }
 
-  console.log('in ExploreDisplayTabs')
+  // eslint-disable-next-line no-unused-vars
   const [_, selectedAnnotation] = getSelectedClusterAndAnnot(exploreInfo, exploreParams)
 
   return (

app/javascript/components/explore/StudyGeneField.jsx

@@ -64,7 +64,7 @@ function getQueriesFromSearchOptions(newQueryArray, speciesList, selectedAnnotat
 }
 
 /** Indicate whether pathway view should be available for this study */
-function getIsEligibleForPathwayExplore(speciesList, selectedAnnotation) {
+export function getIsEligibleForPathwayExplore(speciesList, selectedAnnotation) {
   const isEligibleForPathwayExplore = (
     speciesList.length === 1 && speciesList[0] === 'Homo sapiens' &&
     selectedAnnotation.type === 'group' &&
@@ -104,12 +104,10 @@ export default function StudyGeneField({
 }) {
   const [inputText, setInputText] = useState('')
 
-  console.log('selectedAnnotation', selectedAnnotation)
   const includePathways = getIsEligibleForPathwayExplore(speciesList, selectedAnnotation)
   const rawSuggestions = getAutocompleteSuggestions(inputText, allGenes, includePathways)
   const searchOptions = getSearchOptions(rawSuggestions, speciesList, selectedAnnotation)
 
-
   let enteredQueryArray = []
   if (inputText.length === 0 && queries && queries.length > 0) {
     const queriesSearchOptions = getSearchOptions(queries, speciesList, selectedAnnotation)
@@ -232,7 +230,7 @@ export default function StudyGeneField({
     if (queries.join(',') !== queryArray.map(opt => opt.value).join(',')) {
       // the genes have been updated elsewhere -- resync
       const queriesSearchOptions = getSearchOptions(queries, speciesList, selectedAnnotation)
-      const newQueryArray = getQueryArrayFromSearchOptions(queriesSearchOptions, speciesList)
+      const newQueryArray = getQueryArrayFromSearchOptions(queriesSearchOptions, speciesList, selectedAnnotation)
       setQueryArray(newQueryArray)
       setInputText('')
       setNotPresentQueries(new Set([]))

test/js/explore/study-gene-keyword.test.js

@@ -3,7 +3,7 @@ import React from 'react'
 import { render, waitFor, screen, fireEvent } from '@testing-library/react'
 import '@testing-library/jest-dom/extend-expect'
 
-import StudyGeneField, { getIsInvalidQuery } from 'components/explore/StudyGeneField'
+import StudyGeneField, { getIsInvalidQuery, getIsEligibleForPathwayExplore } from 'components/explore/StudyGeneField'
 import * as UserProvider from '~/providers/UserProvider'
 import { interestingNames, interactionCacheCsn1s1 } from './../visualization/pathway.test-data'
 
@@ -67,7 +67,10 @@ describe('Search query display text', () => {
       })
 
     const { container } = render(
-      <StudyGeneField queries={[]} queryFn={() => {}} allGenes={['PTEN']} speciesList={['Homo sapiens']} />
+      <StudyGeneField
+        queries={[]} queryFn={() => {}} allGenes={['PTEN']}
+        speciesList={['Homo sapiens']} selectedAnnotation={{ type: 'group' }}
+      />
     )
 
     // Find the input field inside react-select
@@ -97,5 +100,26 @@ describe('Search query display text', () => {
     const pathwayIsInvalid = getIsInvalidQuery('CSN1S1', ['PTEN'])
     expect(pathwayIsInvalid).toBe(false)
   })
+
+  it('determines if view is eligible for pathway exploration', async () => {
+    jest
+      .spyOn(UserProvider, 'getFeatureFlagsWithDefaults')
+      .mockReturnValue({
+        show_pathway_expression: true
+      })
+
+    // Confirm mouse is not eligible
+    const isMouseEligible = getIsEligibleForPathwayExplore(['Mus musculus'], { type: 'group' })
+    expect(isMouseEligible).toBe(false)
+
+    // Confirm numeric annotation is not eligible
+    const isNumericAnnotationEligible = getIsEligibleForPathwayExplore(['Homo sapiens'], { type: 'numeric' })
+    expect(isNumericAnnotationEligible).toBe(false)
+
+    // Confirm group annotation for human is eligible
+    const isHumanGroupAnnotationEligible = getIsEligibleForPathwayExplore(['Homo sapiens'], { type: 'group' })
+    expect(isHumanGroupAnnotationEligible).toBe(true)
+  })
+
 })