May 22, 2023 · May 11, 2023 · May 12, 2023 · May 12, 2023 · May 12, 2023 · May 12, 2023
diff --git a/site/src/testHelpers/entities.ts b/site/src/testHelpers/entities.ts
  }),
 }

 export const MockExperiments: TypesGen.Experiment[] = ["workspace_actions", "moons"]
 export const MockExperiments: TypesGen.Experiment[] = [
  "workspace_actions",
  "moons",
 ]

 export const MockAuditLog: TypesGen.AuditLog = {
  id: "fbd2116a-8961-4954-87ae-e4575bd29ce0",
diff --git a/site/src/testHelpers/localstorage.ts b/site/src/testHelpers/localstorage.ts
 export const localStorageMock = () => {
  const store = {} as Record<string, string>;
  const store = {} as Record<string, string>

  return {
    getItem: (key: string):string =>{
      return store[key];
    getItem: (key: string):string =>{
      return store[key]
    },
    setItem: (key:string, value:string)=> {
      store[key] = value;
    setItem: (key:string, value:string)=> {
      store[key] = value
    },
    clear: ()=> {
    clear: ()=> {
      Object.keys(store).forEach((key) => {
        delete store[key];
      });
        delete store[key]
      })
    },
    removeItem: (key: string) => {
      delete store[key]
    },
    removeItem: (key:string) => {
      delete store[key];
    }
  }
 }

 Object.defineProperty(window, "localStorage", { value: localStorageMock() });
 Object.defineProperty(window, "localStorage", { value: localStorageMock() })
Original file line number	Diff line number	Diff line change
Expand Up		@@ -1374,7 +1374,10 @@ export const MockEntitlementsWithScheduling: TypesGen.Entitlements = {
		}),
		}

		export const MockExperiments: TypesGen.Experiment[] = ["workspace_actions", "moons"]
		export const MockExperiments: TypesGen.Experiment[] = [
		"workspace_actions",
		"moons",
		]

		export const MockAuditLog: TypesGen.AuditLog = {
		id: "fbd2116a-8961-4954-87ae-e4575bd29ce0",
Expand Down
Original file line number	Diff line number	Diff line change
		@@ -1,22 +1,22 @@
		export const localStorageMock = () => {
		const store = {} as Record<string, string>;
		const store = {} as Record<string, string>

		return {
		getItem: (key: string):string =>{
		return store[key];
		getItem: (key: string):string =>{
		return store[key]
		},
		setItem: (key:string, value:string)=> {
		store[key] = value;
		setItem: (key:string, value:string)=> {
		store[key] = value
		},
		clear: ()=> {
		clear: ()=> {
		Object.keys(store).forEach((key) => {
		delete store[key];
		});
		delete store[key]
		})
		},
		removeItem: (key: string) => {
		delete store[key]
		},
		removeItem: (key:string) => {
		delete store[key];
		}
		}
		}

		Object.defineProperty(window, "localStorage", { value: localStorageMock() });
		Object.defineProperty(window, "localStorage", { value: localStorageMock() })
Copy link Collaborator BrunoQuaresmaMay 12, 2023 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Since this is being executed when the module is imported, maybe can be interesting to move all this code into the test setup file and mock it globally. We are doing that for a few default window interfaces. Copy link MemberAuthor EmyrkMay 12, 2023 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. maybe, I just wanted to avoid putting things in the global if my test was the only one using it atm.