-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix tests #121
Fix tests #121
Conversation
if out: | ||
logger.debug(f"Stdout:\n{SEP_BEGIN}\n{out}\n{SEP_END}\nStdout finished") | ||
if err: | ||
logger.debug(f"Stderr:\n{SEP_BEGIN}\n{err}\n{SEP_END}\nStderr finished") |
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.
make tests less noisy
@@ -50,7 +51,7 @@ def project_dir() -> Iterator[Path]: | |||
|
|||
@pytest.fixture() | |||
def cli_runner(capfd: CaptureFixture[str], project_dir: Path) -> CLIRunner: | |||
def _run_cli(args: List[str]) -> CompletedProcess: # type: ignore | |||
def _run_cli(args: List[str]) -> "CompletedProcess[str]": |
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.
fix typing instead of ignoring it
@@ -855,9 +888,8 @@ def test_data_cp_from_cloud_to_storage( | |||
@pytest.fixture | |||
def disk(cli_runner: CLIRunner) -> Iterator[str]: | |||
# Create disk | |||
res = cli_runner(["neuro", "disk", "create", "1G"]) | |||
res = cli_runner(["neuro", "disk", "create", "100M"]) |
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.
no need to get 1G. These tests run in parallel, each run will require 1G, so they will conflict.
merging w/out review |
sleep 10
v20.10.16a1
->a3acb3c3dfeabbb01292c3a441f4bf03efd041e5
because it contains this part required by gcp. This will be changed in the following-up releases, so this change is temp.disk:
to get readytest_seldon_deploy_from_local
(see Uncomment image building in test_seldon_deploy_from_local #123)