Skip to content

Commit

Permalink
#14 #53 #78 Complete palette color selector unit tests.
Browse files Browse the repository at this point in the history
  • Loading branch information
@azurepolarbear
azurepolarbear committed Jun 3, 2024
1 parent 2207da3 commit a3c11df
Show file tree
Hide file tree
Showing 3 changed files with 116 additions and 27 deletions.
1 change: 0 additions & 1 deletion src/main/color/color-selector.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,6 @@ export abstract class ColorSelector {
return col;
}

// TODO - unit test
// TODO - documentation
// TODO - update release notes
public chooseBackgroundColor(chanceOfBlack: number,
Expand Down
136 changes: 113 additions & 23 deletions src/test/color/palette/palette-color-selector.test.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -18,27 +18,13 @@
import {Palette, PaletteColorSelector} from 'palette';
import {_0437F2, _0FFF4F, _121212, _7A00F5, _FF6BB5} from 'palette-colors';
import {Discriminators} from 'discriminator';

import {
checkForValidColorSelector,
checkForValidInOrderSelector,
checkForValidRandomSelector,
getColorsArray
} from "unit-test/shared";

// palette order, some colors, random
// palette order, 2 colors, not random
// palette order, 2 colors, random
// palette order, 1 color, not random
// palette order, 1 color, random

// not palette order, all colors, not random
// not palette order, all colors, random
// not palette order, some colors, not random
// not palette order, some colors, random
// not palette order, 2 colors, not random
// not palette order, 2 colors, random
// not palette order, 1 color, not random
// not palette order, 1 color, random
} from 'unit-test/shared';

const TEST_PALETTE_A: Palette = {
NAME: 'test A palette',
Expand Down Expand Up @@ -106,7 +92,6 @@ describe('palette color selector tests', (): void => {
);
});

// palette order, all colors, random
test('test palette color selector: palette order build, all colors, random order', (): void => {
const selector: PaletteColorSelector =
new PaletteColorSelector(
Expand All @@ -120,23 +105,128 @@ describe('palette color selector tests', (): void => {
checkForValidRandomSelector(selector,getColorsArray(TEST_PALETTE_A.COLORS), true);
});

test('test palette color selector: palette order build, 3 colors, not random order', (): void => {
const colorCount: number = 3;
test.each([
{count: 3},
{count: 2},
{count: 1}
])('$# test palette color selector: palette order build, $count color(s), not random order',
({count}): void => {
const colorCount: number = count;
const selector: PaletteColorSelector =
new PaletteColorSelector(
TEST_PALETTE_A,
true,
colorCount,
false
);
expect(selector).toBeTruthy();
checkForValidColorSelector(selector);
checkForValidInOrderSelector(
selector,
getColorsArray(TEST_PALETTE_A.COLORS),
true,
false,
colorCount
);
}
);

test.each([
{count: 3},
{count: 2},
{count: 1}
])('$# test palette color selector: palette order build, $count color(s), random order',
({count}): void => {
const colorCount: number = count;
const selector: PaletteColorSelector =
new PaletteColorSelector(
TEST_PALETTE_A,
true,
colorCount,
true
);
expect(selector).toBeTruthy();
checkForValidColorSelector(selector);
checkForValidRandomSelector(selector,getColorsArray(TEST_PALETTE_A.COLORS), false, colorCount);
}
);

//TODO - duplicate start

test('test palette color selector: random build, all colors, not random order', (): void => {
const selector: PaletteColorSelector =
new PaletteColorSelector(
TEST_PALETTE_A,
true,
colorCount,
false,
TEST_PALETTE_A.COLORS.length,
false
);
expect(selector).toBeTruthy();
checkForValidColorSelector(selector);
checkForValidInOrderSelector(
selector,
getColorsArray(TEST_PALETTE_A.COLORS),
true,
false,
colorCount
true
);
});

test('test palette color selector: random build, all colors, random order', (): void => {
const selector: PaletteColorSelector =
new PaletteColorSelector(
TEST_PALETTE_A,
false,
TEST_PALETTE_A.COLORS.length,
true
);
expect(selector).toBeTruthy();
checkForValidColorSelector(selector);
checkForValidRandomSelector(selector,getColorsArray(TEST_PALETTE_A.COLORS), true);
});

test.each([
{count: 3},
{count: 2},
{count: 1}
])('$# test palette color selector: random build, $count color(s), not random order',
({count}): void => {
const colorCount: number = count;
const selector: PaletteColorSelector =
new PaletteColorSelector(
TEST_PALETTE_A,
false,
colorCount,
false
);
expect(selector).toBeTruthy();
checkForValidColorSelector(selector);
checkForValidInOrderSelector(
selector,
getColorsArray(TEST_PALETTE_A.COLORS),
false,
false,
colorCount
);
}
);

test.each([
{count: 3},
{count: 2},
{count: 1}
])('$# test palette color selector: random build, $count color(s), random order',
({count}): void => {
const colorCount: number = count;
const selector: PaletteColorSelector =
new PaletteColorSelector(
TEST_PALETTE_A,
false,
colorCount,
true
);
expect(selector).toBeTruthy();
checkForValidColorSelector(selector);
checkForValidRandomSelector(selector,getColorsArray(TEST_PALETTE_A.COLORS), false, colorCount);
}
);
});
6 changes: 3 additions & 3 deletions src/test/shared/color-selector.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -86,9 +86,9 @@ export function checkForValidInOrderSelector(selector: ColorSelector,
for (let i: number = 0; i < colorCount; i++) {
const c: Color = selector.getColor();
const components: ColorComponents = colorToColorComponents(c);
const expected: Color = inOrderColors[i];
const expectedComponents: ColorComponents = colorToColorComponents(expected);
expect(components).toEqual(expectedComponents);
const expectedSelection: Color = inOrderColors[i];
const expectedSelectionComponents: ColorComponents = colorToColorComponents(expectedSelection);
expect(components).toEqual(expectedSelectionComponents);
}
} else {
for (let i: number = 0; i < colors.length * 2; i++) {
Expand Down

0 comments on commit a3c11df

Please sign in to comment.