Skip to content

Add integration test for variant analysis monitor#1846

Merged
koesie10 merged 1 commit intomainfrom
koesie10/monitor-multiple-responses-test
Dec 7, 2022
Merged

Add integration test for variant analysis monitor#1846
koesie10 merged 1 commit intomainfrom
koesie10/monitor-multiple-responses-test

Conversation

@koesie10
Copy link
Copy Markdown
Member

@koesie10 koesie10 commented Dec 7, 2022

This integration test will check that the monitor will actually make multiple requests to the API and that it will trigger a download extension command for each repo that has finished scanning.

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

This integration test will check that the monitor will actually make
multiple requests to the API and that it will trigger a download
extension command for each repo that has finished scanning.
@koesie10 koesie10 added the secexp label Dec 7, 2022
@koesie10 koesie10 requested a review from a team as a code owner December 7, 2022 13:50
@elenatanasoiu
Copy link
Copy Markdown
Contributor

Like lightning ⚡

@koesie10 koesie10 merged commit 3dd0ef5 into main Dec 7, 2022
@koesie10 koesie10 deleted the koesie10/monitor-multiple-responses-test branch December 7, 2022 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants