bazelbuild / bazelbuild/bazelisk

Add tests for --migrate

Open
#187 0 comments 0 reactions 0 assignees View on GitHub
P2 type: feature request
Dominant language
Go
Stars
2.7k
Forks
424
PR merge metrics
No merged PRs in 30d

Description

As seen in #185 the entire --migrate workflow is untested, and thus a breakage went by unnoticed.
Unfortunately #186 demonstrates that adding a test is not straightforward, and ideally we'd have a fast unit test that doesn't send any HTTP requests to GitHub.

Contributor guide

Open the contributing guide

Research direction

Start by tracing the --migrate workflow in the Bazelisk repository and read issues #185 and #186 for the observed breakage and testing constraints. Identify the existing workflow tests or entry points, then determine how a fast unit test can cover --migrate without sending HTTP requests to GitHub; done means the workflow is covered and the test remains network-free.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
cli, testing
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.