Skip to content

Commit

Permalink
Modify 'getAllKatas works' test to reduce friction when working on ne…
Browse files Browse the repository at this point in the history
…w katas (#1160)

This is meant to make it easier for people authoring on new katas.
  • Loading branch information
cesarzc authored Feb 16, 2024
1 parent 139a8e7 commit 66fe9a0
Showing 1 changed file with 2 additions and 8 deletions.
10 changes: 2 additions & 8 deletions npm/test/basics.js
Original file line number Diff line number Diff line change
Expand Up @@ -272,15 +272,9 @@ async function validateKata(
}
}

test("all katas work", async () => {
test("getAllKatas works", async () => {
const katas = await getAllKatas();
// N.B. If you update the expected katas count, make sure to add a validation test for your newly added kata.
const expectedKatasCount = 9;
assert.equal(
katas.length,
expectedKatasCount,
`Expected ${expectedKatasCount} katas, but found ${katas.length} katas`,
);
assert.ok(katas.length > 0, "katas should not be empty");
});

test("getting_started kata is valid", async () => {
Expand Down

0 comments on commit 66fe9a0

Please sign in to comment.