8000 0.21.0/fix(tests): fix goroutines leakage in integration tests by AlonZivony · Pull Request #4052 · aquasecurity/tracee · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

0.21.0/fix(tests): fix goroutines leakage in integration tests #4052

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 16, 2024
Merged

0.21.0/fix(tests): fix goroutines leakage in integration tests #4052

merged 1 commit into from
May 16, 2024

Conversation

AlonZivony
Copy link
Contributor

1. Explain what the PR does

Fix tests so they will make sure to close channels properly and clean goroutines.

This is a cherry pick of #4047

2. Explain how to test it

3. Other comments

@AlonZivony
fix(tests): fix goroutines leakage in integration tests (aquasecurity…
12f41be 
…#4047)

Fix tests so they will make sure to close channels properly and clean goroutines.

(cherry picked from commit cd7ed0d)
@AlonZivony AlonZivony requested a review from yanivagman May 16, 2024 14:32
yanivagman
yanivagman approved these changes May 16, 2024
View reviewed changes
Copy link
Collaborator
@yanivagman yanivagman left a comment

Choose a reason for hiding this comment

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

LGTM

@yanivagman yanivagman merged commit 1ba8a3d into aquasecurity:v0.21.0 May 16, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yanivagman yanivagman yanivagman approved these changes

Assignees

@AlonZivony AlonZivony

Labels
area/testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AlonZivony @yanivagman
0