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

test: Add RAG and other Python UT coverage to the codecov report #815

Merged
merged 3 commits into from
Jan 10, 2025
Merged

test: Add RAG and other Python UT coverage to the codecov report #815

merged 3 commits into from
Jan 10, 2025

Conversation

helayoty
Copy link
Collaborator

@helayoty helayoty commented Jan 10, 2025 •
edited
Loading

Reason for Change:

Add RAG and other Python code UT coverage to the codecov report to ensure that the RAG server changes for consistency are tested and the code coverage is reported accurately.

Requirements

  • added unit tests and e2e tests (if applicable).

Issue Fixed:

Notes for Reviewers:

@codecov Codecov
Copy link

codecov bot commented Jan 10, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

see 20 files with indirect coverage changes

@helayoty helayoty force-pushed the helayoty/add-rag-ut-cov branch from d37bd8a to 30cc213 Compare January 10, 2025 02:50
@helayoty helayoty force-pushed the helayoty/add-rag-ut-cov branch from 30cc213 to 0357092 Compare January 10, 2025 02:57
@helayoty helayoty force-pushed the helayoty/add-rag-ut-cov branch from 0357092 to d5d5d7d Compare January 10, 2025 02:58
@helayoty helayoty force-pushed the helayoty/add-rag-ut-cov branch from 18a0b32 to 3186f31 Compare January 10, 2025 03:13
@helayoty
Add RAG UT coverage to the codecov report
9d18512 
Signed-off-by: Heba Elayoty <heelayot@microsoft.com>
@helayoty
add an empty codecov.yml to show the UT coverage for each PR
65deb9a 
Signed-off-by: Heba Elayoty <heelayot@microsoft.com>
@helayoty helayoty force-pushed the helayoty/add-rag-ut-cov branch from 3186f31 to 679335f Compare January 10, 2025 03:24
@helayoty helayoty force-pushed the helayoty/add-rag-ut-cov branch from 679335f to e462067 Compare January 10, 2025 03:33
@helayoty
use pytest to generate coverage
d016927 
Signed-off-by: Heba Elayoty <heelayot@microsoft.com>
@helayoty helayoty force-pushed the helayoty/add-rag-ut-cov branch from e462067 to d016927 Compare January 10, 2025 03:42
@helayoty helayoty changed the title test: Add RAG UT coverage to the codecov report test: Add RAG and other Python UT coverage to the codecov report Jan 10, 2025
@Fei-Guo Fei-Guo merged commit b4d9a85 into main Jan 10, 2025
8 of 9 checks passed
@Fei-Guo Fei-Guo deleted the helayoty/add-rag-ut-cov branch January 10, 2025 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Fei-Guo Fei-Guo Fei-Guo approved these changes

@ishaansehgal99 ishaansehgal99 Awaiting requested review from ishaansehgal99 ishaansehgal99 is a code owner

@zhuangqh zhuangqh Awaiting requested review from zhuangqh zhuangqh is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@helayoty @Fei-Guo