Skip to content

Commit 1cd8618

Browse files
committed
Clarify testing requirement
1 parent 4835a12 commit 1cd8618

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

README.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,10 @@ formats. You must include `valid`, `absoluteKeywordLocation`, and
1717
`instanceLocation`. Because we aren't implementing the Verbose output format,
1818
you don't need to support `annotations`/`annotation`. You don't need to support
1919
`error` because we're going to ignore messages from the implementation anyway.
20-
`keywordLocation` is optional because I can't see any reason we'd use it. You
21-
must include tests to show that your implementation works including coverage for
22-
all implemented keywords.
20+
`keywordLocation` is optional because I can't see any reason we'd use it. **You
21+
must include output format tests to show that your implementation works
22+
including coverage for all implemented keywords. This is in addition to the
23+
validation tests that I provided.**
2324

2425
To submit your qualification task, use `npm pack` and DM it to me. I will
2526
provide one and only one review for each candidate, so make sure you're ready

0 commit comments

Comments
 (0)