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.

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

[FEATURE] Add more integ tests for exporter and reader #233

Open
ansjcy opened this issue Jan 31, 2025 · 0 comments
Open

[FEATURE] Add more integ tests for exporter and reader #233

ansjcy opened this issue Jan 31, 2025 · 0 comments
Labels
enhancement New feature or request

Comments

@ansjcy
Copy link
Member

ansjcy commented Jan 31, 2025

Is your feature request related to a problem?

Exporters and readers are critical components in Query Insights that require robust testing coverage. Recent incidents during release have shown gaps in our testing strategy. We've encountered several unforeseen blockers during releases due to errors in exporter logic. Notable examples include:

The impact is big, including but not limited to:

  • Release delays
  • Required manual intervention
  • Potential reliability issues in production
  • Increased burden on manual testing before merges

Ideally, we should have extensive integ tests for such critical component to avoid manual testing before merging changes.

What solution would you like?

Have extensive integ tests for exporters and readers

  • Validate core functionality
  • Test edge cases
  • Ensure reliability across different scenarios
  • Reduce dependency on manual testing
  • Prevent regression issues

What alternatives have you considered?

N/A

Do you have any additional context?

Add any other context or screenshots about the feature request here.

@ansjcy ansjcy added enhancement New feature or request untriaged and removed untriaged labels Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant