Remove kleur

[fabnguess]

No description provided.

[PierreDemailly]

We need to add export type { Color } from "./colorTypes.js"; in index.ts otherwise Color is internal only.
Also, maybe types/color.ts would be better ? Or simply types.ts, this is the way we process types in our projects (for instance: https://github.com/NodeSecure/scanner/blob/master/workspaces/scanner/src/types.ts)

[fraxken]

We removed the assignment to colors property so nothing work as expected

[PierreDemailly]

Hi @fabnguess, ping in case you forgot there is a change needed here 😋 (this.#color is never assigned)

[fabnguess]

Hi @PierreDemailly ,

I'm having some difficulty with my unit tests. Currently I'm trying to mock the display of the spinner frames to verify that the color is correctly applied. Here is the code I wrote. Could you tell me if it is correct and give me advice on how to improve it?

describe("spinner frame rendering", () => {
  it("should apply the correct color to the spinner frame", async () => {
    const spin = new Spinner({ verbose: false });

    spin.start("Loading...");

    const originalWrite = process.stdout.write;
    let output = "";
    process.stdout.write = (chunk) => {
      output += chunk;
      return true;
    };

    process.stdout.write = originalWrite;

   await once(spin, "start", { signal: AbortSignal.timeout(100) });

    const ESC = "\x1b";
    assert.match(output, new RegExp(`${ESC}\\[34m`));
  });
});

[PierreDemailly]

Since stream is public, maybe we can set a mock ?

Spinner/src/Spinner.class.ts

Line 53 in 2a6d7a7

public stream: TTY.WriteStream = process.stdout;

Spinner/src/Spinner.class.ts

Line 155 in 2a6d7a7

this.stream.write(line);

[fabnguess]

Okay, I'll work on that.