fix: existing tests after refactor

.infra/rdev/values.yaml

@@ -2,7 +2,7 @@ stack:
   services:
     explorer:
       image:
-        tag: sha-44567a92
+        tag: sha-419fde2c
       replicaCount: 1
       env:
         # env vars common to all deployment stages

client/src/components/geneExpression/infoPanel/common/containerInfo/index.tsx → client/src/components/geneExpression/infoPanel/common/infoPanelContainer/index.tsx

@@ -1,7 +1,12 @@
 import React from "react";
 import { kebabCase } from "lodash";
 import { InfoContainer, InfoWrapper } from "../style";
-import { ErrorInfo, LoadingInfo, NoneSelected, ShowInfo } from "../loadingInfo";
+import {
+  ErrorInfo,
+  LoadingInfo,
+  NoneSelected,
+  ShowInfo,
+} from "../infoPanelParts";
 import { ExtendedInfoProps } from "../types";
 
 function ContainerInfo(props: ExtendedInfoProps) {
@@ -21,19 +26,20 @@ function ContainerInfo(props: ExtendedInfoProps) {
 
   const entityTag = kebabCase(entity);
 
+  const wrapperTestId = `${entity === "Gene" ? symbol : id}:${entityTag}-info`;
+
   return (
-    <InfoWrapper
-      id={`${entityTag}info_wrapper`}
-      data-testid={`${id}:${entityTag}-info`}
-    >
+    <InfoWrapper id={`${entityTag}info_wrapper`} data-testid={wrapperTestId}>
       <InfoContainer id={`${entityTag}-info`}>
         {/* Loading */}
         {(name || symbol) && !error && loading && (
           <LoadingInfo name={name} entity={entity} />
         )}
 
         {/* None Selected */}
-        {!entity && error === null && <NoneSelected entity={entity} />}
+        {name === "" && error === null && !loading && (
+          <NoneSelected entity={entity} />
+        )}
 
         {/* Error */}
         {error && <ErrorInfo name={name} entity={entity} />}

client/src/components/geneExpression/infoPanel/connect.ts

@@ -0,0 +1,32 @@
+import { ChangeEvent, useEffect, useState } from "react";
+import { AppDispatch } from "../../../reducers";
+
+function useConnect({
+  dispatch,
+  activeTab,
+}: {
+  dispatch: AppDispatch;
+  activeTab: string;
+}) {
+  const [value, setValue] = useState(1);
+
+  const handleTabsChange = (
+    _: ChangeEvent<Record<string, unknown>>,
+    tabsValue: unknown
+  ) => {
+    setValue(tabsValue as number);
+    dispatch({
+      type: "toggle active info panel",
+      activeTab:
+        tabsValue === 0 ? "Gene" : tabsValue === 1 ? "CellType" : "Dataset",
+    });
+  };
+
+  useEffect(() => {
+    setValue(activeTab === "Gene" ? 0 : activeTab === "CellType" ? 1 : 2);
+  }, [activeTab]);
+
+  return { value, handleTabsChange };
+}
+
+export default useConnect;

client/src/components/geneExpression/infoPanel/index.tsx

@@ -1,4 +1,4 @@
-import React, { useState, ChangeEvent, useEffect } from "react";
+import React from "react";
 import { connect } from "react-redux";
 import { AnchorButton, ButtonGroup } from "@blueprintjs/core";
 import { Tabs, Tab } from "czifui";
@@ -9,30 +9,15 @@ import {
   InfoPanelHeader,
   InfoPanelWrapper,
 } from "./style";
-import CellTypeInfo from "./cellTypeInfo";
-import GeneInfo from "./geneInfo";
-import DatasetInfo from "./datasetInfo";
+import CellTypeInfo from "./infoPanelCellType";
+import GeneInfo from "./infoPanelGene";
+import DatasetInfo from "./infoPanelDataset";
 import { Props, mapStateToProps } from "./types";
+import useConnect from "./connect";
 
 function InfoPanel(props: Props) {
   const { activeTab, dispatch, infoPanelMinimized, infoPanelHidden } = props;
-  const [value, setValue] = useState(1);
-
-  const handleTabsChange = (
-    _: ChangeEvent<Record<string, unknown>>,
-    tabsValue: unknown
-  ) => {
-    setValue(tabsValue as number);
-    dispatch({
-      type: "toggle active info panel",
-      activeTab:
-        tabsValue === 0 ? "Gene" : tabsValue === 1 ? "CellType" : "Dataset",
-    });
-  };
-
-  useEffect(() => {
-    setValue(activeTab === "Gene" ? 0 : activeTab === "CellType" ? 1 : 2);
-  }, [activeTab]);
+  const { value, handleTabsChange } = useConnect({ dispatch, activeTab });
 
   return (
     <InfoPanelWrapper

client/src/components/geneExpression/infoPanel/cellTypeInfo/index.tsx → client/src/components/geneExpression/infoPanel/infoPanelCellType/index.tsx

@@ -1,7 +1,7 @@
 import React from "react";
 import { connect } from "react-redux";
 import { Props, mapStateToProps } from "./types";
-import ContainerInfo from "../common/containerInfo";
+import ContainerInfo from "../common/infoPanelContainer";
 import { CELLGUIDE_URL } from "../common/constants";
 
 function CellTypeInfo(props: Props) {

client/src/components/geneExpression/infoPanel/geneInfo/index.tsx → client/src/components/geneExpression/infoPanel/infoPanelGene/index.tsx

@@ -1,7 +1,7 @@
 import React from "react";
 import { connect } from "react-redux";
 import { Props, mapStateToProps } from "./types";
-import ContainerInfo from "../common/containerInfo";
+import ContainerInfo from "../common/infoPanelContainer";
 
 function GeneInfo(props: Props) {
   const { geneInfo } = props;

client/src/reducers/controls.ts

@@ -54,8 +54,6 @@ const Controls = (
     activeTab: ActiveTab.Dataset,
     infoPanelHidden: true,
     infoPanelMinimized: false,
-    imageOpacity: 100,
-    dotOpacity: 100,
     unsMetadata: {
       imageWidth: 1955,
       imageHeight: 1955,
@@ -212,6 +210,7 @@ const Controls = (
         ...state,
         activeTab: action.activeTab,
         infoPanelHidden: false,
+        infoPanelMinimized: false,
       };
     }
     case "close info panel": {

server/common/utils/cell_type_info.py

@@ -1,6 +1,7 @@
-import requests
 from typing import Dict, List, TypedDict, cast
 
+import requests
+
 from server.common.constants import CELLGUIDE_BASE_URL