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

Implement unit tests for pipeline element 'MergeByTimeProcessor' #1895

Closed
Tracked by #1884
bossenti opened this issue Sep 9, 2023 · 1 comment · Fixed by #2023
Closed
Tracked by #1884

Implement unit tests for pipeline element 'MergeByTimeProcessor' #1895

bossenti opened this issue Sep 9, 2023 · 1 comment · Fixed by #2023
Assignees
@rahulbiswas876
Labels
good first issue Good for newcomers hacktoberfest Indicates an issue for hacktoberfest event 2023. testing Relates to any kind of test (unit test, integration, or E2E test).

Comments

@bossenti
Copy link
Contributor

bossenti commented Sep 9, 2023 •
edited
Loading

This issue is about implementing unit tests for our pipeline element MergeByTimeProcessor.
An example of how to write unit tests for pipeline elements can be found here.

Mentoring

As this issue is marked as a good first issue and for the hacktoberfest event: one of @dominikriemer, @tenthe, @RobertIndie, or @bossenti are happy to provide help for getting started, just tag (one of) them if you want to start working on this issue and need some help.
@bossenti bossenti mentioned this issue Sep 9, 2023
Open
20 tasks
@bossenti bossenti changed the title MergeByTimeProcessor Implement unit tests for pipeline element 'MergeByTimeProcessor' Sep 9, 2023
@bossenti bossenti added good first issue Good for newcomers testing Relates to any kind of test (unit test, integration, or E2E test). hacktoberfest Indicates an issue for hacktoberfest event 2023. labels Sep 9, 2023
@rahulbiswas876
Copy link
Contributor

rahulbiswas876 commented Sep 26, 2023 •
edited
Loading

Hi team,
I want to work on the issue.. Please assign it to me

@rahulbiswas876 rahulbiswas876 mentioned this issue Oct 13, 2023
Merged
@bossenti bossenti linked a pull request Oct 15, 2023 that will close this issue
[#1895]Implement unit tests for pipeline element 'MergeByTimeProcessor' #2023
Merged
bossenti pushed a commit that referenced this issue Oct 16, 2023
@rahulbiswas876
[#1895] Implement unit tests for pipeline element 'MergeByTimeProcess…
d251268 
…or' (#2023)

* FEAT: unit test for MergeByTimeProcessor class

* FEAT: unit test for MergeByTimeProcessor class

* REFACTOR: added License header

* REFACTOR: addressed review comments

* REFACTOR: .gitignore file

* REFACTOR: addressed review comments

* REFACTOR: fix checkstyle violations

---------

Co-authored-by: rahulbis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rahulbiswas876 rahulbiswas876

Labels
good first issue Good for newcomers hacktoberfest Indicates an issue for hacktoberfest event 2023. testing Relates to any kind of test (unit test, integration, or E2E test).
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#1895]Implement unit tests for pipeline element 'MergeByTimeProcessor' rahulbiswas876/streampipes
2 participants
@rahulbiswas876 @bossenti