optimus
optimus copied to clipboard
feat: get compiled job assets from JobRunInput api
proton PR:: https://github.com/odpf/proton/pull/155
Future plans : https://github.com/odpf/optimus/issues/443
Pull Request Test Coverage Report for Build 2838115849
- 76 of 105 (72.38%) changed or added relevant lines in 7 files are covered.
- 2 unchanged lines in 2 files lost coverage.
- Overall coverage decreased (-0.1%) to 75.816%
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
---|---|---|---|
compiler/job_asset_compiler.go | 22 | 23 | 95.65% |
compiler/job_input_render.go | 0 | 1 | 0.0% |
compiler/job_run_input_compiler.go | 13 | 14 | 92.86% |
service/monitoring_service.go | 5 | 10 | 50.0% |
api/handler/v1beta1/job_run.go | 33 | 54 | 61.11% |
<!-- | Total: | 76 | 105 |
Files with Coverage Reduction | New Missed Lines | % |
---|---|---|
job/external_dependency_resolver.go | 1 | 79.75% |
service/monitoring_service.go | 1 | 74.36% |
<!-- | Total: | 2 |
Totals | |
---|---|
Change from base Build 2823967747: | -0.1% |
Covered Lines: | 7408 |
Relevant Lines: | 9771 |
💛 - Coveralls
I think it is better to create another PR for deprecating the old admin build instance command and the corresponding api