-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Code coverage #6173
Code coverage #6173
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #6173 +/- ##
==========================================
Coverage ? 80.88%
==========================================
Files ? 311
Lines ? 35078
Branches ? 6333
==========================================
Hits ? 28372
Misses ? 6463
Partials ? 243
|
@kirill-sizov, could you please review this PR? |
Dockerfile
Outdated
@@ -178,6 +178,12 @@ COPY --chown=${USER} wait-for-it.sh manage.py backend_entrypoint.sh ${HOME}/ | |||
COPY --chown=${USER} utils/ ${HOME}/utils | |||
COPY --chown=${USER} cvat/ ${HOME}/cvat | |||
|
|||
ARG COVERAGE_PROCESS_START | |||
RUN if [ "${COVERAGE_PROCESS_START}" ]; then \ | |||
python3 -m pip install --no-cache-dir coverage==7.2.3; \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need the line? It should be installed by 75 line in the file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK. Let's add the line in production.txt and add a comment that "to collect the coverage for rest api tests. It will be enabled only if COVERAGE_PROCESS_START env is defined." I believe it will be better for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
It was accidentally disabled in cvat-ai#6173.
It was accidentally disabled in #6173. Also, refactor the build steps to make the log easier to read.
Motivation and context
How has this been tested?
Checklist
develop
branch(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)
License
Feel free to contact the maintainers if that's a concern.