Skip to content

Commit

Permalink
(HP-1728) re-order imports
Browse files Browse the repository at this point in the history
  • Loading branch information
george42-ctds committed Jan 21, 2025
1 parent c54ab0e commit 3e26e14
Show file tree
Hide file tree
Showing 6 changed files with 14 additions and 9 deletions.
8 changes: 4 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ The [VLMD docs](heal/vlmd/README.md) describe how to use the SDK for extracting

### Run tests

```
```bash
poetry run pytest -vv tests
```

Expand All @@ -35,20 +35,20 @@ reference the git repo.
As an example, `pip install` can be called from the command line for getting
the master branch of the `heal-platform-sdk`,

```
```bash
pip install -e git+https://github.com/uc-cdis/heal-platform-sdk.git#egg=heal
```

or a particular branch, eg,

```
```bash
pip install -e git+https://github.com/uc-cdis/heal-platform-sdk.git@my-branch#egg=heal
```

The specification can also be listed in requirements.txt file
(with, say, a tag specification of 0.1.0)

```
```bash
pip install -e git+https://github.com/uc-cdis/[email protected]#egg=heal
```

Expand Down
1 change: 0 additions & 1 deletion heal/cli/extract.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import click

from cdislogging import get_logger

from heal.vlmd.extract.extract import vlmd_extract
Expand Down
3 changes: 2 additions & 1 deletion heal/cli/heal_cli.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
import click
import logging

import cdislogging
import click

import heal.cli.vlmd as vlmd


Expand Down
1 change: 0 additions & 1 deletion heal/cli/validate.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import click

from cdislogging import get_logger

from heal.vlmd.validate.validate import vlmd_validate
Expand Down
1 change: 1 addition & 0 deletions tests/conftest.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import json

import pytest

from heal.vlmd.config import (
Expand Down
9 changes: 7 additions & 2 deletions tests/test_cli.py
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
import os

from click.testing import CliRunner
import pytest

import heal.vlmd.file_utils as file_utils
import heal.cli.heal_cli as cli_module
import heal.vlmd.file_utils as file_utils


def test_help():
"""Test the help menu"""
runner = CliRunner()
expected_text = "HEAL-Platform SDK Command Line Interface"
expected_commands = ["vlmd Commands for VLMD"]
Expand All @@ -19,6 +19,7 @@ def test_help():


def test_vlmd_help():
"""Test the VLMD submenu"""
runner = CliRunner()
expected_text = "Commands for VLMD"
expected_commands = [
Expand All @@ -33,6 +34,7 @@ def test_vlmd_help():


def test_extract_help():
"""Test the extract submenu"""
runner = CliRunner()
expected_text = "Extract HEAL-compliant VLMD file from input file"
expected_commands = [
Expand All @@ -47,6 +49,7 @@ def test_extract_help():


def test_extract(tmp_path):
"""Test the cli extract"""
runner = CliRunner()
input_file = "tests/test_data/vlmd/valid/vlmd_valid.csv"
expected_output_file = file_utils.get_output_filepath(
Expand All @@ -62,6 +65,7 @@ def test_extract(tmp_path):


def test_validate_help():
"""Test the validate submenu"""
runner = CliRunner()
expected_text = "Validate VLMD input file"
expected_commands = [
Expand All @@ -75,6 +79,7 @@ def test_validate_help():


def test_validate(tmp_path):
"""Test the cli validation"""
runner = CliRunner()
input_file = "tests/test_data/vlmd/valid/vlmd_valid.json"
result = runner.invoke(
Expand Down

0 comments on commit 3e26e14

Please sign in to comment.