Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: initial gotestsum support and larger refactoring #81

Merged
merged 14 commits into from
Jul 3, 2024

Conversation

fredrikaverpil
Copy link
Owner

@fredrikaverpil fredrikaverpil commented Jul 2, 2024

image

This is just me playing around with using gotestsum so output is readable when attaching to long-running test (hotkey a in the test summary window), like an entire test suite.

This can allow nice output to be shown (when attached) as tests are running. Under the hood, neotest-golang will still parse the JSON output.

To do:

  • Clean up and refactoring, so command arguments can be controlled nicely and intuitively.
  • If gotestsum is chosen as "test runner", verify it indeed is on $PATH or throw error (possibly fall back to go test).

@fredrikaverpil fredrikaverpil self-assigned this Jul 2, 2024
@fredrikaverpil fredrikaverpil force-pushed the feat/gotestsum branch 7 times, most recently from 5fb64d0 to 1190b88 Compare July 3, 2024 18:55
@fredrikaverpil fredrikaverpil force-pushed the feat/gotestsum branch 2 times, most recently from 7496da2 to 945332a Compare July 3, 2024 21:16
@fredrikaverpil fredrikaverpil force-pushed the feat/gotestsum branch 3 times, most recently from 5049ae5 to c1ffa9b Compare July 3, 2024 21:54
@fredrikaverpil fredrikaverpil changed the title feat(poc): gotestsum for readable output when attached feat: initial gotestsum support and larger refactoring Jul 3, 2024
@fredrikaverpil fredrikaverpil marked this pull request as ready for review July 3, 2024 22:04
@fredrikaverpil fredrikaverpil merged commit 8672472 into main Jul 3, 2024
9 checks passed
@fredrikaverpil fredrikaverpil deleted the feat/gotestsum branch July 3, 2024 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant