-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
updated and fixed typeform detectors #3769
updated and fixed typeform detectors #3769
Conversation
@@ -86,40 +89,6 @@ func TestTypeform_FromChunk(t *testing.T) { | |||
wantErr: false, | |||
wantVerificationErr: false, | |||
}, | |||
{ | |||
name: "found, would be verified if not for timeout", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even with 1 Microsecond timeout this test case was failing with no error 😄 (🥲) So I removed them.
c5b0d9d
to
ec892be
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Single non-blocking comment, but otherwise LGTM! Thanks for getting this cleaned up. ❤️
3fe6f2b
to
d4cc844
Compare
* updated and fixed typeform detectors * fixed linter issue * added tags back
Description:
This PR improve overall logic of TypeForm detectors and fixes it's integration test cases.
Checklist:
make test-community
)?make lint
this requires golangci-lint)?