From 8979902b96e742d674afc886c92f1453b8d15b11 Mon Sep 17 00:00:00 2001
From: Abhineet Jain <abhineet@coder.com>
Date: Wed, 13 Jul 2022 18:23:22 +0000
Subject: [PATCH] fix: remove system user highlighting

---
 .../WorkspaceBuildStats.tsx                   |  6 ++---
 .../WorkspaceStats/WorkspaceStats.tsx         |  6 ++---
 .../WorkspacesTable/WorkspacesRow.tsx         |  4 ++--
 site/src/util/workspace.test.ts               | 19 ++++-----------
 site/src/util/workspace.ts                    | 23 ++++---------------
 5 files changed, 15 insertions(+), 43 deletions(-)

diff --git a/site/src/components/WorkspaceBuildStats/WorkspaceBuildStats.tsx b/site/src/components/WorkspaceBuildStats/WorkspaceBuildStats.tsx
index 63537102673ea..8aaebd635aa27 100644
--- a/site/src/components/WorkspaceBuildStats/WorkspaceBuildStats.tsx
+++ b/site/src/components/WorkspaceBuildStats/WorkspaceBuildStats.tsx
@@ -19,7 +19,7 @@ export const WorkspaceBuildStats: FC<WorkspaceBuildStatsProps> = ({ build }) =>
   const styles = useStyles()
   const theme = useTheme()
   const status = getDisplayWorkspaceBuildStatus(theme, build)
-  const initiatedBy = getDisplayWorkspaceBuildInitiatedBy(theme, build)
+  const initiatedBy = getDisplayWorkspaceBuildInitiatedBy(build)
 
   return (
     <div className={styles.stats}>
@@ -61,9 +61,7 @@ export const WorkspaceBuildStats: FC<WorkspaceBuildStatsProps> = ({ build }) =>
       <div className={styles.statsDivider} />
       <div className={styles.statItem}>
         <span className={styles.statsLabel}>Initiated by</span>
-        <span className={styles.statsValue}>
-          <span style={{ color: initiatedBy.color }}>{initiatedBy.initiatedBy}</span>
-        </span>
+        <span className={styles.statsValue}>{initiatedBy}</span>
       </div>
     </div>
   )
diff --git a/site/src/components/WorkspaceStats/WorkspaceStats.tsx b/site/src/components/WorkspaceStats/WorkspaceStats.tsx
index d1fb1a51e16c8..8a75ab5edf5eb 100644
--- a/site/src/components/WorkspaceStats/WorkspaceStats.tsx
+++ b/site/src/components/WorkspaceStats/WorkspaceStats.tsx
@@ -28,7 +28,7 @@ export const WorkspaceStats: FC<WorkspaceStatsProps> = ({ workspace }) => {
   const styles = useStyles()
   const theme = useTheme()
   const status = getDisplayStatus(theme, workspace.latest_build)
-  const initiatedBy = getDisplayWorkspaceBuildInitiatedBy(theme, workspace.latest_build)
+  const initiatedBy = getDisplayWorkspaceBuildInitiatedBy(workspace.latest_build)
 
   return (
     <WorkspaceSection title={Language.workspaceDetails} contentsProps={{ className: styles.stats }}>
@@ -63,9 +63,7 @@ export const WorkspaceStats: FC<WorkspaceStatsProps> = ({ workspace }) => {
       <div className={styles.statsDivider} />
       <div className={styles.statItem}>
         <span className={styles.statsLabel}>{Language.byLabel}</span>
-        <span className={styles.statsValue}>
-          <span style={{ color: initiatedBy.color }}>{initiatedBy.initiatedBy}</span>
-        </span>
+        <span className={styles.statsValue}>{initiatedBy}</span>
       </div>
       <div className={styles.statsDivider} />
       <div className={styles.statItem}>
diff --git a/site/src/components/WorkspacesTable/WorkspacesRow.tsx b/site/src/components/WorkspacesTable/WorkspacesRow.tsx
index c5cb2e17a1a62..978b61d469a2a 100644
--- a/site/src/components/WorkspacesTable/WorkspacesRow.tsx
+++ b/site/src/components/WorkspacesTable/WorkspacesRow.tsx
@@ -27,7 +27,7 @@ export const WorkspacesRow: FC<{ workspaceRef: WorkspaceItemMachineRef }> = ({ w
   const [workspaceState, send] = useActor(workspaceRef)
   const { data: workspace } = workspaceState.context
   const status = getDisplayStatus(theme, workspace.latest_build)
-  const initiatedBy = getDisplayWorkspaceBuildInitiatedBy(theme, workspace.latest_build)
+  const initiatedBy = getDisplayWorkspaceBuildInitiatedBy(workspace.latest_build)
   const workspacePageLink = `/@${workspace.owner_name}/${workspace.name}`
 
   return (
@@ -47,7 +47,7 @@ export const WorkspacesRow: FC<{ workspaceRef: WorkspaceItemMachineRef }> = ({ w
       </TableCellLink>
       <TableCellLink to={workspacePageLink}>
         <AvatarData
-          title={initiatedBy.initiatedBy}
+          title={initiatedBy}
           subtitle={dayjs().to(dayjs(workspace.latest_build.created_at))}
         />
       </TableCellLink>
diff --git a/site/src/util/workspace.test.ts b/site/src/util/workspace.test.ts
index bcc8154ca48aa..349ce1d841fe8 100644
--- a/site/src/util/workspace.test.ts
+++ b/site/src/util/workspace.test.ts
@@ -1,7 +1,6 @@
 import dayjs from "dayjs"
 import * as TypesGen from "../api/typesGenerated"
 import * as Mocks from "../testHelpers/entities"
-import { dark } from "../theme/theme"
 import {
   defaultWorkspaceExtension,
   getDisplayWorkspaceBuildInitiatedBy,
@@ -109,14 +108,13 @@ describe("util > workspace", () => {
   })
 
   describe("getDisplayWorkspaceBuildInitiatedBy", () => {
-    it.each<[TypesGen.WorkspaceBuild, string, string]>([
-      [Mocks.MockWorkspaceBuild, "#C1C1C1", "TestUser"],
+    it.each<[TypesGen.WorkspaceBuild, string]>([
+      [Mocks.MockWorkspaceBuild, "TestUser"],
       [
         {
           ...Mocks.MockWorkspaceBuild,
           reason: "autostart",
         },
-        "#7057FF",
         "system/autostart",
       ],
       [
@@ -124,17 +122,10 @@ describe("util > workspace", () => {
           ...Mocks.MockWorkspaceBuild,
           reason: "autostop",
         },
-        "#7057FF",
         "system/autostop",
       ],
-    ])(
-      `getDisplayWorkspaceBuildInitiatedBy(%p) returns color: %p, initiatedBy: %p`,
-      (build, color, initiatedBy) => {
-        expect(getDisplayWorkspaceBuildInitiatedBy(dark, build)).toEqual({
-          color: color,
-          initiatedBy: initiatedBy,
-        })
-      },
-    )
+    ])(`getDisplayWorkspaceBuildInitiatedBy(%p) returns %p`, (build, initiatedBy) => {
+      expect(getDisplayWorkspaceBuildInitiatedBy(build)).toEqual(initiatedBy)
+    })
   })
 })
diff --git a/site/src/util/workspace.ts b/site/src/util/workspace.ts
index 2d7df1fda88f4..d9f3bc963e92b 100644
--- a/site/src/util/workspace.ts
+++ b/site/src/util/workspace.ts
@@ -191,29 +191,14 @@ export const DisplayWorkspaceBuildInitiatedByLanguage = {
   autostop: "system/autostop",
 }
 
-export const getDisplayWorkspaceBuildInitiatedBy = (
-  theme: Theme,
-  build: TypesGen.WorkspaceBuild,
-): {
-  color: string
-  initiatedBy: string
-} => {
+export const getDisplayWorkspaceBuildInitiatedBy = (build: TypesGen.WorkspaceBuild): string => {
   switch (build.reason) {
     case "initiator":
-      return {
-        color: theme.palette.text.secondary,
-        initiatedBy: build.initiator_name,
-      }
+      return build.initiator_name
     case "autostart":
-      return {
-        color: theme.palette.secondary.dark,
-        initiatedBy: DisplayWorkspaceBuildInitiatedByLanguage.autostart,
-      }
+      return DisplayWorkspaceBuildInitiatedByLanguage.autostart
     case "autostop":
-      return {
-        color: theme.palette.secondary.dark,
-        initiatedBy: DisplayWorkspaceBuildInitiatedByLanguage.autostop,
-      }
+      return DisplayWorkspaceBuildInitiatedByLanguage.autostop
   }
 }