ljnsn / coinapi-rest / 11309257501

92%

push

github

🐛 fix: sonar issues (#21)

* 🐛 fix: remove unused init

* ♻️  refactor: base operations class

* ♻️  refactor: use constant for exception message

* ♻️  refactor: generate url function

* ♻️  refactor: get query params

* 💚 ci: upload correct coverage file

* 💚 ci: simplify lint job and upload reports

* 💚 ci: combine coverage reports

* ♻️  refactor: get_deep_object_query_params

* 💚 ci: install coverage with toml extra

* 💚 ci: check reports

* 💚 ci: don't merge artifacts

* ♻️  refactor: serialize multipart form

* 💚 ci: fix combining coverage

* ♻️  refactor: serialize form data

* 💚 ci: remove lses

* ♻️  refactor: populate form

* ♻️  refactor: serialize header

* 🐛 fix: rename unused param

* 💚 ci: use only latest coverage

* 🔧 config(coverage): try another workaround

* 🔧 config(coverage): add workaround for source path issue

for the issue see [1]
for the workaround see [2]

[1]: https://github.com/nedbat/coveragepy/issues/578
[2]: https://github.com/LibraryOfCongress/concordia/pull/857/files

* 💚 ci: update triggers to run on PRs too

* ✅ test: add utils tests

* 🐛 fix: remove class vars on security client

* 💚 ci: remove always from sonarqube

* 🐛 fix: handle annotated file fields

* ✅ test(utils): add more tests

* ✅ test(base): add some tests

* ✅ test(utils): more tests

* ✅ test(utils): more coverage

* ✅ test(utils): more tests

138 of 168 branches covered (82.14%)

Branch coverage included in aggregate %.

307 of 364 new or added lines in 2 files covered. (84.34%)

2371 of 2547 relevant lines covered (93.09%)

5.59 hits per line