JS support for Kotest runner

Open
#186 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
38/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
javascript, kotlin
Domain
testing

Research direction

Start in selfie-lib by reviewing the five tests listed in jvmTest and their existing commonTest counterparts, then search selfie-runner-kotest for TODO: https://github.com/diffplug/selfie/issues/186. Move and implement the tests sequentially where possible, keeping RecordCallTest platform-specific if necessary, and move HarnessVerifyTest into commonTest; done means the JS runner supports these tests.

Written by the indexing model from the issue text.

Description

enhancement js

JS seems like the easiest next step, but PRs for native platforms are welcome too!

  • in selfie-lib, move tests from jvmTest to commonTest one by one and implement the missing part
    • most tests are already in commonTest, here is a suggested order for the 5 which are stuck in jvmTest
    • LineReaderTest, SnapshotValueReaderTest, SnapshotReaderTest, SnapshotFileTest (sequentially in one PR)
    • RecordCallTest in its own PR, perhaps needs to stay platform-specific
  • in selfie-runner-kotest search for TODO: https://github.com/diffplug/selfie/issues/186
  • move HarnessVerifyTest out of jvmTest and into commonTest
Dominant language
Kotlin
Stars
101
Forks
18
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from diffplug/selfie

All issues in diffplug/selfie

Similar issues

More Kotlin issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.