Skip to content

Comments

Implemented End-2-End test for the gitleaks scanner (closes #273)#336

Merged
J12934 merged 7 commits intomainfrom
gitleaks-e2e
Mar 23, 2021
Merged

Implemented End-2-End test for the gitleaks scanner (closes #273)#336
J12934 merged 7 commits intomainfrom
gitleaks-e2e

Conversation

@twwd
Copy link
Contributor

@twwd twwd commented Mar 23, 2021

Description

Add a simple e2e test for the gitleaks scanner (closes #273)

This PR adds a integration test for the gitleaks scanner. It only scans commit ec0fe17 in this repository that introduced a dummy finding.

Along the way I add a .nvmrc file to ensure a consistent nodejs version and downgrade a lockfile that was already on version 2 (npm 7).

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure npm test runs for the whole project.
  • Make codeclimate checks happy

Copy link

@qlty-cloud-legacy qlty-cloud-legacy bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 6819 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@qlty-cloud-legacy qlty-cloud-legacy bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 6819 lines exceeds the maximum allowed for the inline comments feature.

Copy link

@qlty-cloud-legacy qlty-cloud-legacy bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 6819 lines exceeds the maximum allowed for the inline comments feature.

@J12934 J12934 merged commit 17a9be5 into main Mar 23, 2021
@J12934 J12934 deleted the gitleaks-e2e branch March 23, 2021 14:39
@rfelber rfelber added the testing Improvements or additions regarding the test setup label Mar 26, 2021
@rfelber rfelber changed the title Implemented end-2-end test for the gitleaks scanner (closes #273) Implemented End-2-End test for the gitleaks scanner (closes #273) Apr 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Improvements or additions regarding the test setup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🧪 Implement a end-2-end test for the gitleaks scanner

3 participants