Open maximilien opened 4 years ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: maximilien
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/hold
@daisy-ycguo: The /test
command needs one or more targets.
The following commands are available to trigger jobs:
/test pull-knative-client-contrib-build-tests
/test pull-knative-client-contrib-unit-tests
/test pull-knative-client-contrib-integration-tests
Use /test all
to run all jobs.
/test all
@maximilien Hello, dr.max. Currently, the integration test runs nothing, because you don't have an e2e-tests.sh
under plugins/source-github/test
, just a local-e2e-tests.sh
. The integration test framework used by client-contrib is designed to call plugins/source-github/test/e2e-tests.sh
for the e2e test of every plugin. So please add a plugins/source-github/test/e2e-tests.sh
to your plugin. You may just copy the same of local-e2e-tests.sh
. Thanks.
Thanks @daisy-ycguo will do.
@maximilien: PR needs rebase.
@maximilien: The following tests failed, say /retest
to rerun all failed tests:
Test name | Commit | Details | Rerun command |
---|---|---|---|
pull-knative-client-contrib-integration-tests | 897f796167136bac559f8779b25d7b6c327b6bf8 | link | /test pull-knative-client-contrib-integration-tests |
pull-knative-client-contrib-build-tests | 897f796167136bac559f8779b25d7b6c327b6bf8 | link | /test pull-knative-client-contrib-build-tests |
pull-knative-client-contrib-unit-tests | 897f796167136bac559f8779b25d7b6c327b6bf8 | link | /test pull-knative-client-contrib-unit-tests |
Full PR test history. Your PR dashboard.
Includes:
Missing:
Fixes #10 and #34