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.

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

Add Dedup tests (dedup_test.cc) to the unit test project + fix some dedup tests #348

Open
slahabar opened this issue Jun 9, 2021 · 0 comments
Open

Add Dedup tests (dedup_test.cc) to the unit test project + fix some dedup tests #348

slahabar opened this issue Jun 9, 2021 · 0 comments
Labels
bug Something isn't working

Comments

@slahabar
Copy link
Contributor

slahabar commented Jun 9, 2021

Add
Existing Dedup tests (dedup_test.cc) is not added to the unit test project.

Fix
And the below test are failing on the master branch.
action_dict_builder
compression_transformer
dedup_disable_compression
dedup_enable_compression

Improve
Add test for checking app_id in the generic event for dedup.
@slahabar slahabar changed the title Add Dedup tests (dedup_test.cc) to the unit test project + some dedup tests are failing Add Dedup tests (dedup_test.cc) to the unit test project + fix some dedup tests Jun 9, 2021
@slahabar slahabar mentioned this issue Jun 9, 2021
Merged
@jackgerrits jackgerrits added the bug Something isn't working label Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jackgerrits @slahabar