Add passwordless-related tests to testplan.
* Include passwordless in testplan
* Update .github/ISSUE_TEMPLATE/testplan.md
Co-authored-by: Lisa Kim <lisa@goteleport.com>
Removed automatic creation of backport Pull Requests due to limitations
within GitHub Actions on recursive creation of workflows not allowing
the "Check" workflow to run upon automation creation of the Pull
Request.
As a temporary workaround, prepare the backport branch and provide a
link the user can click on to create the backport Pull Request.
* Move test plan to an issue template
* Update test plan with new desktop access features
* Add Binaries compatibility to test plan (#10518)
* Updates clipboard support tests (#10521)
* Adds MFA tests for Desktop
* makes browser instructions more precise
Co-authored-by: Jakub Nyckowski <jakub.nyckowski@goteleport.com>
Co-authored-by: Isaiah Becker-Mayer <isaiah@goteleport.com>
* Add more lint coverage
golanglint-ci doesn't pick up subdirectories with their own go.mod
which left certain directories unlinted. To get around this we can
run golanglint-ci directly against those submodules.
The bot runs code from the `master` branch even for the stable branches
that still use vendored dependencies, so we should keep ignoring the
`vendor/` directory in the bot, or we might miscategorize PRs.
In a3d85ab removed reviewers from jobs that don't require it and did not
update parsing logic. Instead pass reviewers to all jobs as it is not a
secret and will be inlined into actions/bot soon.
Code reviewers are split into code owners and code reviewers. Each PR is
now randomly assigned a reviewers from the code owner and reviewer set.
Review checking makes sure that a reviewer from each set has approved
the PR.
This new feature in Go 1.17 automatically restores the environment
variable to its previous value when a test ends, making it simpler
to set up the environment for tests and less likely that we accidentally
leave behind global state.
Also convert some of the remaining uses of check to standard Go tests.