Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added pattern test cases for detectors starting with Alphabet C #3564

Merged

Conversation

kashifkhan0771
Copy link
Contributor

@kashifkhan0771 kashifkhan0771 commented Nov 6, 2024

Description:

This PR separate the integration test cases and add pattern test cases for all detectors starting with alphabet c.

Tests Output:

Here's the beautiful output ❤️
pattern-c-tests01
pattern-c-tests02

Checklist:

  • Tests passing (make test-community)?
  • Lint passing (make lint this requires golangci-lint)?

@kashifkhan0771 kashifkhan0771 force-pushed the tests/pattern-test-cases-c branch from 4abea0e to cf90506 Compare November 7, 2024 11:05
},
want: nil,
wantErr: false,
name: "valid pattern - not found",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This particular detector secret is not being pushed due to some pre-commit push secret error.

@kashifkhan0771 kashifkhan0771 changed the title Added pattern test cases for detectors starting with Alphabet C [WIP] Added pattern test cases for detectors starting with Alphabet C Nov 7, 2024
@kashifkhan0771 kashifkhan0771 marked this pull request as ready for review November 7, 2024 15:15
@kashifkhan0771 kashifkhan0771 requested a review from a team as a code owner November 7, 2024 15:15
@zricethezav zricethezav merged commit 6921ffd into trufflesecurity:main Nov 7, 2024
13 checks passed
@kashifkhan0771 kashifkhan0771 deleted the tests/pattern-test-cases-c branch November 7, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants