agentleFS
1Shot-API1Shot-API/skills/unit-tests/SKILL.md

unit-tests

Skill1 starsChanged 7 days agoMITView on GitHub

Authors Jest unit tests for TypeScript services using testdouble, neverthrow ResultAsync, and a structured mocks class. Use when writing or refactoring unit tests, adding test coverage, or when the user asks for tests matching project conventions (Arrange/Act/Assert, explicit mocks, negative-only td.verify).

16 other projects use a version of this.

Adopted by

Most recently taken up by

Used with

Projects with it also use auto-perf-optimize, chat-customizations-editor and 10 more.

Other files people use alongside this one.

Files found in the same public projects as a version of this one, more often than they turn up anywhere else. Each project counts once, large collections count for less, and this file's own repository is left out.
  • MicrosoftSkill

    auto-perf-optimize

    Run agent-driven VS Code performance or memory investigations. Use when asked to launch Code OSS, automate a VS Code scenario, run the Chat memory smoke runner, capture renderer heap snapshots, take workflow screenshots, compare run summaries, or drive a repeatable scenario before heap-snapshot analysis.

    Also in 5 projects that use this

  • MicrosoftSkill

    chat-customizations-editor

    Use when working on the Chat Customizations editor — the management UI for agents, skills, instructions, hooks, prompts, MCP servers, and plugins.

    Also in 5 projects that use this

  • OpenCortexIDESkill

    add-policy

    Use when adding, modifying, or reviewing VS Code configuration policies. Covers the full policy lifecycle from registration to export to platform-specific artifacts. Run on ANY change that adds a `policy:` field to a configuration property.

    Also in 4 projects that use this

  • +9 more from microsoft

The file

We split this skill into the rules it gives an agent

6 rules. 0 are common, 1 is distinctive, and 5 are very specific to this project.

  • 0

    Common

    Most of the 15 edited versions we read keep these.

  • 1

    Distinctive

    This author's own choices. Worth a look.

  • 5

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  1. Distinctive:

    Follow this structure so tests stay readable, explicit, and easy to maintain.

    line 17
  2. Very specific:

    Use real or fake values deliberately: prefer values that match production shape (UUIDs, 0x-prefixed addresses) so type constructors and …

    line 47
  3. Very specific:

    Create one class named after the system under test, e.g.

    line 51
  4. Very specific:

    Avoid td.matchers.anything() in baseline stubs unless the API is truly opaque.

    line 77
  5. Very specific:

    Use td.verify(..., { times: 0 }) when the test’s purpose is to prove something did not run (e.g.

    line 125
  6. Very specific:

    When verify is required, still prefer specific arguments where feasible.

    line 127

“Common” is measured: kept by at least 6 in 10 of the 15 edited versions of this file we could read. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

Unit tests (service / business layer)

Follow this structure so tests stay readable, explicit, and easy to maintain. Reference implementations (in the oneshot monorepo):

  • packages/signer/test/unit/business/SignerService.test.ts — parameterized mocks class, explicit Redis key matching, getInstance().
  • packages/EVMBaseChain/test/unit/business/EscrowWalletService.test.ts — shared CommonValues, heavy domain setup in Arrange, factoryService().

Install this skill (consumer / global)

npx skills add 1Shot-API/skills/unit-tests -g -a claude-code -y

When to use this skill

  • Writing or refactoring Jest unit tests for TypeScript services in the oneshot monorepo
  • Adding coverage that should match project conventions (testdouble, neverthrow ResultAsync)
  • The user asks for tests matching Arrange/Act/Assert, explicit mocks, or negative-only td.verify

When NOT to use this skill

  • Frontend/portal component tests that do not use the service mocks class pattern
  • Integration or e2e tests outside the Jest + testdouble service layer style
  • Relayer (1shot-okx-relayer) tests that use bun:test instead of Jest/testdouble

1. File-level constants

At the top of the test file (after imports), declare named constants used across tests:

  • Stable IDs and branded types (UserId, BusinessId, addresses) so every test uses the same fake identity unless a case needs a different value.
  • Shared primitives reused in mocks (e.g. JWT strings, chain enum) when they are not test-specific.

Use real or fake values deliberately: prefer values that match production shape (UUIDs, 0x-prefixed addresses) so type constructors and validators behave like production.

2. ClassUnderTestMocks class

Create one class named after the system under test, e.g. SignerServiceMocks, EscrowWalletServiceMocks.

  • Constructor parameters: Pass anything that should vary the baseline behavior across tests (e.g. userId, EService for the chain service token). Tests that need the same baseline use the same arguments; divergent tests pass different constructor args or override in Arrange (see below).

3. Dependencies

The mocks class owns every dependency the class-under-test needs:

  • td.object<Interface>() for each injected port (repository, config provider, Redis, etc.).
  • Public fields so tests and overrides can reference them (mocks.kmsRepository, mocks.redisClient, …).

Wire the real constructor in a single factory method:

  • Name it getInstance(), factoryService(), or another clear name; one place constructs new ClassUnderTest(...).

4. Baseline = happy path in the constructor

In the constructor (or an init() called from it), set td.when(...).thenReturn(...) for the default, success-path behavior:

  • getConfig() → okAsync a real-enough config object.
  • getRedisClient() → okAsync the mock Redis client.
  • Any stubbed remote call that most tests expect to succeed.

Prefer explicit arguments in td.when():

  • Pass concrete CreateSigningKeyParams-like values, exact RedisKeys.* arrays, or td.matchers.contains([...]) with real key strings when the implementation builds a known list.
  • Avoid td.matchers.anything() in baseline stubs unless the API is truly opaque; the tighter the match, the earlier wrong calls fail.

Complex or repeated payloads (resolved user context, config blobs) can be built in protected helpers or factory methods on the mocks class.

5. Public members and factories

Expose on the mocks class:

  • Data the SUT needs from outside: e.g. ruc, config, prebuilt SignerConfig.
  • Factory methods for verbose objects (makeRucForUserAndChainService, etc.) when multiple tests need the same shape with small variations.

6. Test structure: Arrange / Act / Assert

Each test(...) (or it(...)) uses three sections with comments:

test("descriptive name", async () => {
	// Arrange
	const mocks = new SomeServiceMocks(/* baseline params */);
	// Optional: td.when overrides for this test only
	const svc = mocks.getInstance();

	// Act
	const result = await svc.someMethod(...);

	// Assert
	expect(result.isErr()).toBe(true);
	// ...
});

7. Arrange

  • Instantiate ClassUnderTestMocks with the right constructor args.
  • Apply overrides after construction: extra td.when(mocks.dep.method(explicitArgs)).thenReturn(...) for this test only (e.g. kill switch returns ["on"]).
  • End Arrange with const sut = mocks.getInstance() (or factoryService()).

8. Act

  • Only calls on the system under test (and sometimes one obvious preparatory call if the scenario requires it). No assertions in Act.

9. Assert

  • Primary: expect(...) on the method result — for ResultAsync / neverthrow, result.isOk(), result.isErr(), _unsafeUnwrap(), _unsafeUnwrapErr(), toBeInstanceOf, deep equality on unwrapped values.
  • Avoid td.verify for happy-path call counts unless the team explicitly wants interaction testing for that case.

10. td.verify — negative guarantees only

Use td.verify(..., { times: 0 }) when the test’s purpose is to prove something did not run (e.g. KMS not called after validation failure, or Redis kill switch short-circuits before repository access).

When verify is required, still prefer specific arguments where feasible; use anything() only for parameters that are intentionally opaque or unstable.

Naming consistency

  • Mocks class: {ClassUnderTest}Mocks (e.g. SignerServiceMocks).
  • Factory: getInstance() or factoryService() — pick one per file and stay consistent.
  • Fix typos in dependency field names (repository not repsitory) in new tests.

Checklist for a new test file

  • Constants block for shared IDs / tokens / addresses
  • Single mocks class with all td.object dependencies
  • Constructor establishes happy-path td.when with explicit values where possible
  • getInstance / factoryService builds the real service
  • Each test has // Arrange, // Act, // Assert
  • Assert focuses on expect on results; td.verify only for must-not-happen side effects

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.