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

test: add FE tests for new starter project templates #6772

Merged
merged 6 commits into from
Feb 24, 2025

Conversation

Cristhianzl
Copy link
Member

@Cristhianzl Cristhianzl commented Feb 23, 2025

This pull request includes the addition of multiple integration tests for various project components. Each test is designed to verify the functionality of specific features by simulating user interactions and checking the expected outcomes. The most important changes include the addition of tests for the Financial Report Parser, Gmail Agent, Graph Rag, Meeting Summary, and News Aggregator.

Integration tests added:

…rch Translation Loop functionality in the frontend to ensure proper functionality and behavior.
@Cristhianzl Cristhianzl self-assigned this Feb 23, 2025
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Feb 23, 2025
@github-actions github-actions bot added the test Changes to tests label Feb 23, 2025
…nder feature

📝 (Price Deal Finder.spec.ts): Document test cases and setup for Price Deal Finder feature
♻️ (Research Translation Loop.spec.ts): Refactor test setup and add missing test case for Research Translation Loop feature
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Feb 24, 2025
@github-actions github-actions bot added test Changes to tests and removed test Changes to tests labels Feb 24, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 24, 2025
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Feb 24, 2025
@github-actions github-actions bot added test Changes to tests and removed test Changes to tests labels Feb 24, 2025
@Cristhianzl Cristhianzl changed the title test: Add Research Translation Loop starter template for FE testing test: Add FE tests to new starter projects Feb 24, 2025
@Cristhianzl Cristhianzl changed the title test: Add FE tests to new starter projects test: add FE tests for new starter project templates Feb 24, 2025
Copy link

codspeed-hq bot commented Feb 24, 2025

CodSpeed Performance Report

Merging #6772 will improve performances by 41.47%

Comparing cz/test-research-translation (9fe18a2) with main (05935ed)

Summary

⚡ 1 improvements
✅ 13 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
test_build_flow_invalid_job_id 11.3 ms 8 ms +41.47%

@github-actions github-actions bot added test Changes to tests and removed test Changes to tests labels Feb 24, 2025
@github-actions github-actions bot added test Changes to tests and removed test Changes to tests labels Feb 24, 2025
@Cristhianzl Cristhianzl added this pull request to the merge queue Feb 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Feb 24, 2025
@github-actions github-actions bot added test Changes to tests and removed test Changes to tests labels Feb 24, 2025
@Cristhianzl Cristhianzl added this pull request to the merge queue Feb 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 24, 2025
@Cristhianzl Cristhianzl added this pull request to the merge queue Feb 24, 2025
Merged via the queue into main with commit 5e40c5f Feb 24, 2025
33 of 34 checks passed
@Cristhianzl Cristhianzl deleted the cz/test-research-translation branch February 24, 2025 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XXL This PR changes 1000+ lines, ignoring generated files. test Changes to tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants