[test] Remove prints & use set-up binary for `test_is_valid` in CLI test

pull/2/head
Daniel Van Der Maden 5 years ago
parent de40ab4b93
commit ee342392b6
  1. 8
      tests/cli-pyhmy/test_cli.py
  2. 1
      tests/util-pyhmy/test_util.py

@ -12,7 +12,6 @@ BINARY_FILE_PATH = f"{TEMP_DIR}/bin/cli_test_binary"
@pytest.fixture(scope="session", autouse=True) @pytest.fixture(scope="session", autouse=True)
def setup(): def setup():
print(f"Running test set-up for {os.path.realpath(__file__)}")
shutil.rmtree(TEMP_DIR, ignore_errors=True) shutil.rmtree(TEMP_DIR, ignore_errors=True)
os.makedirs(TEMP_DIR, exist_ok=True) os.makedirs(TEMP_DIR, exist_ok=True)
from pyhmy.util import download_cli from pyhmy.util import download_cli
@ -22,16 +21,13 @@ def setup():
@pytest.mark.run(order=0) @pytest.mark.run(order=0)
def test_is_valid(): def test_is_valid():
from pyhmy.util import download_cli
good_file_path = os.path.realpath(f"{TEMP_DIR}/test_is_valid/hmy")
bad_file_path = os.path.realpath(f"{TEMP_DIR}/test_is_valid/bad_hmy") bad_file_path = os.path.realpath(f"{TEMP_DIR}/test_is_valid/bad_hmy")
shutil.rmtree(Path(bad_file_path).parent, ignore_errors=True) shutil.rmtree(Path(bad_file_path).parent, ignore_errors=True)
os.makedirs(Path(bad_file_path).parent, exist_ok=True) os.makedirs(Path(bad_file_path).parent, exist_ok=True)
Path(bad_file_path).touch() Path(bad_file_path).touch()
download_cli(good_file_path, replace=True, verbose=False) assert os.path.exists(BINARY_FILE_PATH), "harmony cli did not download"
assert os.path.exists(good_file_path), "harmony cli did not download"
assert os.path.exists(bad_file_path), "did not create bad binary" assert os.path.exists(bad_file_path), "did not create bad binary"
assert cli.is_valid_binary(good_file_path) assert cli.is_valid_binary(BINARY_FILE_PATH)
assert not cli.is_valid_binary(bad_file_path) assert not cli.is_valid_binary(bad_file_path)

@ -14,7 +14,6 @@ TEMP_DIR = "/tmp/pyhmy-testing/test-util"
@pytest.fixture(scope="session", autouse=True) @pytest.fixture(scope="session", autouse=True)
def setup(): def setup():
print(f"Running test set-up for {os.path.realpath(__file__)}")
shutil.rmtree(TEMP_DIR, ignore_errors=True) shutil.rmtree(TEMP_DIR, ignore_errors=True)
os.makedirs(TEMP_DIR, exist_ok=True) os.makedirs(TEMP_DIR, exist_ok=True)

Loading…
Cancel
Save