Skip to content

Commit

Permalink
Merge pull request #42 from jjjermiah/39-add-code-coverage-to-gha
Browse files Browse the repository at this point in the history
build: Update Codecov action and add coverage badge
  • Loading branch information
jjjermiah committed Jan 21, 2024
2 parents 3bb7fc2 + 76d9fa8 commit 2114f3b
Show file tree
Hide file tree
Showing 9 changed files with 26 additions and 16 deletions.
16 changes: 10 additions & 6 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,16 +27,20 @@ jobs:
- name: Install poetry
uses: snok/install-poetry@v1

- name: Install package
run: poetry install

- name: Test with pytest
run: poetry run pytest
run: |
pip install -r requirements.txt
poetry install
pytest --cov --cov-report=xml
- name: Use Codecov to track coverage
uses: codecov/codecov-action@v2
uses: codecov/codecov-action@v3
with:
file: ./coverage.xml
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
files: ./coverage.xml
fail_ci_if_error: true # optional (default = false)
verbose: true # optional (default = false)
name: codecov-umbrella

- name: Build documentation
run: poetry run make html --directory docs/
Expand Down
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
[![PyTests](https://github.com/jjjermiah/NBIA-toolkit/actions/workflows/main.yml/badge.svg)](https://github.com/jjjermiah/NBIA-toolkit/actions/workflows/main.yml)
[![Documentation Status](https://readthedocs.org/projects/nbia-toolkit/badge/?version=latest)](https://nbia-toolkit.readthedocs.io/en/latest/?badge=latest)
[![codecov](https://codecov.io/gh/jjjermiah/NBIA-toolkit/graph/badge.svg?token=JKREY71D0R)](https://codecov.io/gh/jjjermiah/NBIA-toolkit)


# NBIA Toolkit
A python package that provides programmatic access to query and download images from the National Biomedical Imaging Archive (**NBIA**) and The Cancer Imaging Archive (**TCIA**) databases.
Expand Down
2 changes: 2 additions & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@ python-semantic-release = "^8.5.1"
requires = ["poetry-core"]
build-backend = "poetry.core.masonry.api"

# [tool.coverage]
# source = "src/nbiatoolkit"

# https://python-semantic-release.readthedocs.io/en/latest/configuration.html
[tool.semantic_release]
Expand Down
2 changes: 2 additions & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,5 @@ pytest-cov==4.0.0
ruff==v0.0.260
coverage-badge~=1.1.0
requests==2.31.0
pydicom==2.4.3
tqdm==4.66.1
10 changes: 5 additions & 5 deletions src/nbiatoolkit/nbia.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
from nbiatoolkit.auth import OAuth2
from nbiatoolkit.utils.nbia_endpoints import NBIA_ENDPOINTS
from nbiatoolkit.logger.logger import setup_logger
from nbiatoolkit.utils.md5 import validateMD5
from nbiatoolkit.dicomsort import DICOMSorter
from .auth import OAuth2
from .utils.nbia_endpoints import NBIA_ENDPOINTS
from .logger.logger import setup_logger
from .utils.md5 import validateMD5
from .dicomsort import DICOMSorter

import requests
from requests.exceptions import JSONDecodeError as JSONDecodeError
Expand Down
2 changes: 1 addition & 1 deletion tests/test_auth.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
# pytest -v -s

import pytest
from nbiatoolkit import OAuth2
from src.nbiatoolkit import OAuth2
import time
import requests

Expand Down
2 changes: 1 addition & 1 deletion tests/test_dicom_helpers.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
from nbiatoolkit.dicomsort.helper_functions import parseDICOMKeysFromFormat,sanitizeFileName,truncateUID
from src.nbiatoolkit.dicomsort.helper_functions import parseDICOMKeysFromFormat,sanitizeFileName,truncateUID
import pytest
###############################################################################
# parseDICOMKeysFromFormat
Expand Down
4 changes: 2 additions & 2 deletions tests/test_md5.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import os
from nbiatoolkit.utils.md5 import validateMD5, calculateMD5, MD5HashMismatchError
from nbiatoolkit.utils.nbia_endpoints import NBIA_ENDPOINTS
from src.nbiatoolkit.utils.md5 import validateMD5, calculateMD5, MD5HashMismatchError
from src.nbiatoolkit.utils.nbia_endpoints import NBIA_ENDPOINTS
import requests
import pytest
from tempfile import TemporaryDirectory
Expand Down
2 changes: 1 addition & 1 deletion tests/test_nbia.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
# pytest -v -n 8 tests/test_nbia.py::test_getCollections

import pytest
from nbiatoolkit import NBIAClient
from src.nbiatoolkit import NBIAClient
from tempfile import TemporaryDirectory
import os

Expand Down

0 comments on commit 2114f3b

Please sign in to comment.