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

Integration test for events API #4175

Closed
killme2008 opened this issue Jun 20, 2024 · 1 comment
Closed

Integration test for events API #4175

killme2008 opened this issue Jun 20, 2024 · 1 comment
Assignees
@paomian @shuiyisong
Labels
C-enhancement Category Enhancements O-end-to-end Found by E2E tests

Comments

@killme2008
Copy link
Contributor

What type of enhancement is this?

Tech debt reduction

What does the enhancement do?

As the title says. It's better to add integration tests for events API.

Refer to https://github.com/GreptimeTeam/greptimedb/blob/main/tests-integration/tests/http.rs

@shuiyisong @paomian

Implementation challenges

No response
@killme2008 killme2008 added the C-enhancement Category Enhancements label Jun 20, 2024
@killme2008 killme2008 added the O-end-to-end Found by E2E tests label Jun 20, 2024
@shuiyisong
Copy link
Contributor

I have added some tests in #4156. Please wait for the pr to be merged 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@paomian paomian

@shuiyisong shuiyisong

Labels
C-enhancement Category Enhancements O-end-to-end Found by E2E tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@killme2008 @paomian @shuiyisong