Run all CI against a single fractal-server instance - Githubissues

fractal-analytics-platform / fractal-client

Command-line client for Fractal

https://fractal-analytics-platform.github.io/fractal-client

BSD 3-Clause "New" or "Revised" License

45 stars 1 forks source link

Run all CI against a single fractal-server instance #717

Closed ychiucco closed 4 days ago

ychiucco commented 1 week ago

closes #697 closes #719 closes #720 closes #721

Checklist before merging

[x] I added an appropriate entry to CHANGELOG.md

github-actions[bot] commented 1 week ago

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
fractal_client
authclient.py	$This PR doesn't change the number of statements in fractal_client/authclient.py, which is 78.$	$This PR adds 1 to the number of statements missing coverage in fractal_client/authclient.py, taking it from 2 to 3.$	$This PR removes 1.19 percentage points from the coverage rate in fractal_client/authclient.py, taking it from 97.61% (76/78) to 96.42% (75/78).$	$This PR does not seem to add statements to fractal_client/authclient.py.$
parser.py	$This PR removes 2 from the number of statements in fractal_client/parser.py, taking it from 196 to 194.$	$This PR doesn't change the number of statements missing coverage in fractal_client/parser.py, which is 0.$	$This PR doesn't change the coverage rate in fractal_client/parser.py, which is 100% (194/194).$	$In this PR, 3 new statements are added to fractal_client/parser.py, 3 of which are covered (100%).$
fractal_client/cmd
__init__.py	$This PR removes 5 from the number of statements in fractal_client/cmd/__init__.py, taking it from 228 to 223.$	$This PR adds 3 to the number of statements missing coverage in fractal_client/cmd/__init__.py, taking it from 0 to 3.$	$This PR removes 1.33 percentage points from the coverage rate in fractal_client/cmd/__init__.py, taking it from 100% (228/228) to 98.67% (220/223).$	$This PR does not seem to add statements to fractal_client/cmd/__init__.py.$
_aux_task_caching.py	$This PR doesn't change the number of statements in fractal_client/cmd/_aux_task_caching.py, which is 84.$	$This PR adds 1 to the number of statements missing coverage in fractal_client/cmd/_aux_task_caching.py, taking it from 0 to 1.$	$This PR removes 1.92 percentage points from the coverage rate in fractal_client/cmd/_aux_task_caching.py, taking it from 100% (84/84) to 98.07% (83/84).$	$In this PR, 1 new statements are added to fractal_client/cmd/_aux_task_caching.py, 1 of which are covered (100%).$
_job.py	$This PR doesn't change the number of statements in fractal_client/cmd/_job.py, which is 67.$	$This PR adds 2 to the number of statements missing coverage in fractal_client/cmd/_job.py, taking it from 6 to 8.$	$This PR removes 5.88 percentage points from the coverage rate in fractal_client/cmd/_job.py, taking it from 91.76% (61/67) to 85.88% (59/67).$	$This PR does not seem to add statements to fractal_client/cmd/_job.py.$
_task.py	$This PR removes 10 from the number of statements in fractal_client/cmd/_task.py, taking it from 122 to 112.$	$This PR removes 4 from the number of statements missing coverage in fractal_client/cmd/_task.py, taking it from 6 to 2.$	$This PR adds 3.23 percentage points to the coverage rate in fractal_client/cmd/_task.py, taking it from 94.94% (116/122) to 98.17% (110/112).$	$In this PR, 4 new statements are added to fractal_client/cmd/_task.py, 4 of which are covered (100%).$
Project Total

This report was generated by python-coverage-comment-action

tcompa commented 1 week ago

[x] go back to subprocess, without using PIPE (see https://github.com/fractal-analytics-platform/fractal-client/pull/717/commits/0c867b5052125d843347cbe16d1341c52493d578)
[x] local tests: print the stdout/stderr path from within the fixture, so that it's easily accessible
[x] github tests: if pytest fails, cat the full stdout/stderr files in the next GHA step - for inspection