feat: cell type info and info panel refactor

.infra/rdev/values.yaml

@@ -2,7 +2,7 @@ stack:
   services:
     explorer:
       image:
-        tag: sha-c765d258
+        tag: sha-a6dc7df8
       replicaCount: 1
       env:
         # env vars common to all deployment stages

client/__tests__/e2e/cellxgeneActions.ts

@@ -602,43 +602,51 @@ export async function expandGene(
 
 export async function requestGeneInfo(gene: string, page: Page): Promise<void> {
   await page.getByTestId(`get-info-${gene}`).click();
-  await expect(page.getByTestId(`${gene}:gene-info`)).toBeTruthy();
+}
+
+export async function requestCellTypeInfo(cell: string, page: Page) {
+  await page.getByTestId(`cell_type:category-expand`).click();
+  await page.getByTestId(`get-info-cell_type-${cell}`).click();
 }
 
 export async function assertInfoPanelExists(
-  gene: string,
+  id: string,
+  infoType: string,
   page: Page
 ): Promise<void> {
-  await expect(page.getByTestId(`${gene}:gene-info`)).toBeTruthy();
-  await expect(page.getByTestId(`info-panel-header`)).toBeTruthy();
-  await expect(page.getByTestId(`min-info-panel`)).toBeTruthy();
+  await expect(
+    page.getByTestId(`${id}:${infoType}-info-wrapper`)
+  ).toBeVisible();
+  await expect(page.getByTestId(`info-panel-header`)).toBeVisible();
+  await expect(page.getByTestId(`min-info-panel`)).toBeVisible();
 
-  await expect(page.getByTestId(`gene-info-synonyms`).innerText).not.toEqual(
-    ""
-  );
+  await expect(
+    page.getByTestId(`${infoType}-info-synonyms`).innerText
+  ).not.toEqual("");
 
-  await expect(page.getByTestId(`gene-info-link`)).toBeTruthy();
+  await expect(page.getByTestId(`${infoType}-info-link`)).toBeTruthy();
 }
 
 export async function minimizeInfoPanel(page: Page): Promise<void> {
   await page.getByTestId("min-info-panel").click();
 }
 
 export async function assertInfoPanelIsMinimized(
-  gene: string,
+  id: string,
+  infoType: string,
   page: Page
 ): Promise<void> {
   const testIds = [
-    `${gene}:gene-info`,
+    `${id}:${infoType}-info-wrapper`,
     "info-panel-header",
     "max-info-panel",
     "close-info-panel",
   ];
 
   await tryUntil(
     async () => {
-      for (const id of testIds) {
-        const result = await page.getByTestId(id).isVisible();
+      for (const testId of testIds) {
+        const result = await page.getByTestId(testId).isVisible();
         await expect(result).toBe(true);
       }
     },
@@ -651,19 +659,20 @@ export async function closeInfoPanel(page: Page): Promise<void> {
 }
 
 export async function assertInfoPanelClosed(
-  gene: string,
+  id: string,
+  infoType: string,
   page: Page
 ): Promise<void> {
   const testIds = [
-    `${gene}:gene-info`,
+    `${id}:${infoType}-info-wrapper`,
     "info-panel-header",
     "min-info-panel",
     "close-info-panel",
   ];
   await tryUntil(
     async () => {
-      for (const id of testIds) {
-        const result = await page.getByTestId(id).isVisible();
+      for (const testId of testIds) {
+        const result = await page.getByTestId(testId).isVisible();
         await expect(result).toBe(false);
       }
     },

client/__tests__/e2e/e2e.test.ts

@@ -1,4 +1,4 @@
 /**
  * Smoke test suite that will be run in GHA
  * Tests included in this file are expected to be relatively stable and test core features
  *
@@ -52,6 +52,7 @@
   snapshotTestGraph,
   getAllCategories,
   selectLayout,
+  requestCellTypeInfo,
 } from "./cellxgeneActions";
 
 import { datasets } from "./data";
@@ -66,6 +67,7 @@
   conditionallyToggleSidePanel,
   goToPage,
   shouldSkipTests,
+  skipIfPbmcDataset,
   skipIfSidePanel,
   toggleSidePanel,
 } from "../util/helpers";
@@ -99,6 +101,7 @@
 
 // open gene info card
 const geneToRequestInfo = "SIK1";
+const cellToRequestInfo = "monocyte";
 
 const genesetDescriptionString = "fourth_gene_set: fourth description";
 const genesetToCheckForDescription = "fourth_gene_set";
@@ -1382,6 +1385,9 @@
               test("open info panel and hide/remove", async ({
                 page,
               }, testInfo) => {
+                skipIfSidePanel(graphTestId, MAIN_PANEL);
+                skipIfPbmcDataset(testDataset, DATASET);
+
                 await setup({ option, page, url, testInfo });
                 await addGeneToSearch(geneToRequestInfo, page);
 
@@ -1390,7 +1396,11 @@
                 await tryUntil(
                   async () => {
                     await requestGeneInfo(geneToRequestInfo, page);
-                    await assertInfoPanelExists(geneToRequestInfo, page);
+                    await assertInfoPanelExists(
+                      geneToRequestInfo,
+                      "gene",
+                      page
+                    );
                   },
                   { page }
                 );
@@ -1400,7 +1410,11 @@
                 await tryUntil(
                   async () => {
                     await minimizeInfoPanel(page);
-                    await assertInfoPanelIsMinimized(geneToRequestInfo, page);
+                    await assertInfoPanelIsMinimized(
+                      geneToRequestInfo,
+                      "gene",
+                      page
+                    );
                   },
                   { page }
                 );
@@ -1410,7 +1424,39 @@
                 await tryUntil(
                   async () => {
                     await closeInfoPanel(page);
-                    await assertInfoPanelClosed(geneToRequestInfo, page);
+                    await assertInfoPanelClosed(
+                      geneToRequestInfo,
+                      "gene",
+                      page
+                    );
+                  },
+                  { page }
+                );
+
+                await snapshotTestGraph(page, testInfo);
+
+                await tryUntil(
+                  async () => {
+                    await requestCellTypeInfo(cellToRequestInfo, page);
+                    await assertInfoPanelExists(
+                      cellToRequestInfo,
+                      "cell-type",
+                      page
+                    );
+                  },
+                  { page }
+                );
+
+                await snapshotTestGraph(page, testInfo);
+
+                await tryUntil(
+                  async () => {
+                    await minimizeInfoPanel(page);
+                    await assertInfoPanelIsMinimized(
+                      cellToRequestInfo,
+                      "cell-type",
+                      page
+                    );
                   },
                   { page }
                 );

client/__tests__/util/helpers.ts

@@ -291,6 +291,13 @@ export function skipIfSidePanel(graphTestId: string, MAIN_PANEL: string) {
   skip(graphTestId !== MAIN_PANEL, "This test is only for the main panel");
 }
 
+export function skipIfPbmcDataset(graphTestId: string, PBMC_DATASET: string) {
+  skip(
+    graphTestId === PBMC_DATASET,
+    "This test is only for the spatial dataset, since there's not cell_type data in pbmc3k.cxg dataset"
+  );
+}
+
 export function shouldSkipTests(
   graphTestId: string,
   SIDE_PANEL: string

client/src/actions/index.ts

@@ -179,6 +179,21 @@ async function fetchGeneInfo(
   return response;
 }
 
+interface CellTypeInfoAPI {
+  name: string;
+  summary: string;
+}
+/**
+ * Fetch cell type summary information
+ * @param cell human-readable name of cell type
+ */
+async function fetchCellTypeInfo(
+  cell: string
+): Promise<CellTypeInfoAPI | undefined> {
+  const response = await fetchJson<CellTypeInfoAPI>(`cellinfo?cell=${cell}`);
+  return response;
+}
+
 function prefetchEmbeddings(annoMatrix: AnnoMatrix) {
   /*
   prefetch requests for all embeddings
@@ -539,6 +554,7 @@ export default {
   checkExplainNewTab,
   navigateCheckUserState,
   selectDataset,
+  fetchCellTypeInfo,
   fetchGeneInfo,
   selectContinuousMetadataAction: selnActions.selectContinuousMetadataAction,
   selectCategoricalMetadataAction: selnActions.selectCategoricalMetadataAction,

client/src/analytics/events.ts

@@ -51,6 +51,7 @@ export enum EVENTS {
   EXPLORER_ADD_GENE_AND_SELECT_HISTOGRAM = "EXPLORER_ADD_GENE_AND_SELECT_HISTOGRAM",
   EXPLORER_SELECT_HISTOGRAM = "EXPLORER_SELECT_HISTOGRAM",
   EXPLORER_GENE_INFO_BUTTON_CLICKED = "EXPLORER_GENE_INFO_BUTTON_CLICKED",
+  EXPLORER_CELLTYPE_INFO_BUTTON_CLICKED = "EXPLORER_CELLTYPE_INFO_BUTTON_CLICKED",
   EXPLORER_IMAGE_SELECT = "EXPLORER_IMAGE_SELECT",
   EXPLORER_IMAGE_DESELECT = "EXPLORER_IMAGE_DESELECT",
   EXPLORER_RE_CENTER_EMBEDDING = "EXPLORER_RE_CENTER_EMBEDDING",

client/src/common/types/entities.ts

@@ -143,3 +143,30 @@ export interface PublisherMetadata {
   published_month: number;
   published_year: number;
 }
+
+export interface CellInfo {
+  cellId: string;
+  cellName: string;
+  cellDescription: string;
+  synonyms: string[];
+  references: string[];
+  error: string | null;
+  loading: boolean;
+}
+
+export interface GeneInfo {
+  gene: string | null;
+  geneName: string;
+  geneSummary: string;
+  geneSynonyms: string[];
+  geneUrl: string;
+  showWarningBanner: boolean;
+  infoError: string | null;
+  loading: boolean;
+}
+
+export enum ActiveTab {
+  Gene = "Gene",
+  Dataset = "Dataset",
+  CellType = "CellType",
+}