generated from allen-cell-animated/github-boilerplate
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* change color handling module * organize and cleanup * update example * update tests * fix file name * name functions more consistently * remove setGeom * move renderer out of colorHandler * remove import * fix typo * live modes stop correctly * fix tests * add type for colorinfo * use type for ColorInfo * remove redundant function * rename for clarity
- Loading branch information
Showing
15 changed files
with
397 additions
and
472 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,101 @@ | ||
| /* eslint-disable @typescript-eslint/no-explicit-any */ | ||
| /* NOTE: this file is testing many private | ||
| methods and properties of VisGeometry.ts | ||
| which we do by using (visGeometry as any) | ||
| so we need to disable the eslint rule here | ||
| */ | ||
|
|
||
| import ColorHandler from "../visGeometry/ColorHandler"; | ||
| import { convertColorStringToNumber } from "../visGeometry/ColorHandler"; | ||
| import { Color } from "three"; | ||
|
|
||
| const initialColorData = ["#000000", "#000001", "#000002", "#000003"]; | ||
| const colorHandler = new ColorHandler(); | ||
|
|
||
| describe("ColorHandler", () => { | ||
| beforeEach(() => { | ||
| (colorHandler as any).updateColorArray(initialColorData); | ||
| }); | ||
| describe("updateColorArray", () => { | ||
| test("sets colorsData with 4 values for each color", () => { | ||
| expect((colorHandler as any).colorsData).toHaveLength( | ||
| initialColorData.length * 4 | ||
| ); | ||
| expect((colorHandler as any).numberOfColors).toBe( | ||
| initialColorData.length | ||
| ); | ||
| }); | ||
| }); | ||
| describe("convertDataColorIndexToId", () => { | ||
| test("it returns the index into the shorter, color as string, array", () => { | ||
| expect((colorHandler as any).convertDataColorIndexToId(0)).toBe(0); | ||
| expect((colorHandler as any).convertDataColorIndexToId(4)).toBe(1); | ||
| expect((colorHandler as any).convertDataColorIndexToId(8)).toBe(2); | ||
| expect((colorHandler as any).convertDataColorIndexToId(12)).toBe(3); | ||
| }); | ||
| test("if the index is outside the the length of colorsData, it loops to the beginning", () => { | ||
| expect((colorHandler as any).convertDataColorIndexToId(16)).toBe(0); | ||
| expect((colorHandler as any).convertDataColorIndexToId(20)).toBe(1); | ||
| expect((colorHandler as any).convertDataColorIndexToId(24)).toBe(2); | ||
| expect((colorHandler as any).convertDataColorIndexToId(28)).toBe(3); | ||
| }); | ||
| test("if the colorDataIndex is not divisible by 4, returns -1", () => { | ||
| expect((colorHandler as any).convertDataColorIndexToId(1)).toBe(-1); | ||
| expect((colorHandler as any).convertDataColorIndexToId(2)).toBe(-1); | ||
| expect((colorHandler as any).convertDataColorIndexToId(3)).toBe(-1); | ||
| }); | ||
| }); | ||
| describe("getColorDataIndex", () => { | ||
| test("it returns the index into the colorData array of an existing color", () => { | ||
| for (let i = 0; i < initialColorData.length; i++) { | ||
| const colorNumber = convertColorStringToNumber( | ||
| initialColorData[i] | ||
| ); | ||
| const newColor = [ | ||
| ((colorNumber & 0x00ff0000) >> 16) / 255.0, | ||
| ((colorNumber & 0x0000ff00) >> 8) / 255.0, | ||
| ((colorNumber & 0x000000ff) >> 0) / 255.0, | ||
| 1.0, | ||
| ]; | ||
| expect((colorHandler as any).getColorDataIndex(newColor)).toBe( | ||
| i * 4 | ||
| ); | ||
| } | ||
| }); | ||
| test("it returns -1 if the color is not found", () => { | ||
| const colorNumber = convertColorStringToNumber("#000004"); | ||
| const newColor = [ | ||
| ((colorNumber & 0x00ff0000) >> 16) / 255.0, | ||
| ((colorNumber & 0x0000ff00) >> 8) / 255.0, | ||
| ((colorNumber & 0x000000ff) >> 0) / 255.0, | ||
| 1.0, | ||
| ]; | ||
| expect((colorHandler as any).getColorDataIndex(newColor)).toBe(-1); | ||
| }); | ||
| }); | ||
|
|
||
| describe("getColorById", () => { | ||
| test("it returns the color as a string for a valid colorId", () => { | ||
| for (let i = 0; i < initialColorData.length; i++) { | ||
| const expectedColor = new Color(initialColorData[i]); | ||
| const actualColor = (colorHandler as any).getColorById(i); | ||
| expect(actualColor.r).toBeCloseTo(expectedColor.r, 0.0001); | ||
| expect(actualColor.g).toBeCloseTo(expectedColor.g, 0.0001); | ||
| expect(actualColor.b).toBeCloseTo(expectedColor.b, 0.0001); | ||
| } | ||
| }); | ||
| test("it returns the first color for an invalid colorId", () => { | ||
| const actualColor = (colorHandler as any).getColorById(-1); | ||
|
|
||
| expect(actualColor).toEqual((colorHandler as any).getColorById(0)); | ||
| }); | ||
| test("it loops around if the id is out of range", () => { | ||
| expect((colorHandler as any).getColorById(4)).toEqual( | ||
| (colorHandler as any).getColorById(0) | ||
| ); | ||
| expect((colorHandler as any).getColorById(5)).toEqual( | ||
| (colorHandler as any).getColorById(1) | ||
| ); | ||
| }); | ||
| }); | ||
| }); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.