~martin-kalcok/juju-lint:unit-tests

Branch merges

Propose for merging

Merged into juju-lint:master at revision 3d7a202f2f005c8fc5e9df2e926b6b5001347458

Eric Chen: Approve on 2022-09-07

Gabriel Cocenza: Approve on 2022-09-06

BootStack Reviewers: Pending requested 2022-09-01

Diff: 2367 lines (+1793/-195)

17 files modified

jujulint/check_spaces.py (+1/-3)
jujulint/cli.py (+1/-1)
jujulint/cloud.py (+3/-4)
jujulint/k8s.py (+9/-19)
jujulint/lint.py (+31/-18)
jujulint/logging.py (+15/-19)
jujulint/openstack.py (+1/-5)
tests/conftest.py (+12/-2)
tests/requirements.txt (+1/-2)
tests/test_check_spaces.py (+347/-0)
tests/test_cli.py (+377/-3)
tests/test_cloud.py (+495/-112)
tests/test_jujulint.py (+225/-7)
tests/test_k8s.py (+29/-0)
tests/test_logging.py (+216/-0)
tests/test_openstack.py (+29/-0)
tox.ini (+1/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

e504493... by Martin Kalcok on 2022-09-06

Fix linting errors.

848196a... by Martin Kalcok on 2022-09-05

Merge remote-tracking branch 'upstream/master' into unit-tests

# Conflicts:
# tests/requirements.txt
# tests/test_jujulint.py

4bd3680... by Martin Kalcok on 2022-09-01

Mark code without test coverage as "no cover".

This is a temporary measure allowing us to set 100% requirement for code
coverage of all future changes. There will be a follow-up work to cover
currently untested code paths.

6ab4cba... by Martin Kalcok on 2022-09-01

Add more corner-case unit tests

8a8402d... by Martin Kalcok on 2022-09-01

Improve docstring of Linter.atoi() method

ea3f302... by Gabriel Cocenza on 2022-08-30

applied pattern of bootstack charms for make and tox commands.

- added pre-commit support.
- solved lint issues.
- removed debug flag to build snap. If the build fails, it should
  stop the CI and not open the shell from the snap.
- added an aggregator-kubernetes to solve pre-commit yaml check.
- changed tox and make commands to match scripts on the bootstack CI.

Closes-Bug: #1978739
Partial-Bug: #1978841

Reviewed-on: https://code.launchpad.net/~gabrielcocenza/juju-lint/+git/juju-lint/+merge/426219
Reviewed-by: Martin Kalcok <email address hidden>
Reviewed-by: Eric Chen <email address hidden>

5160caa... by Gabriel Cocenza on 2022-08-29

- removing pyproject-flake8

89f1ae1... by Gabriel Cocenza on 2022-08-01

applied pattern of bootstack charms for make and tox commands.

- added pre-commit support.
- solved lint issues.
- removed debug flag to build snap. If the build fails, it should
  stop the CI and not open the shell from the snap.
- added an aggregator-kubernetes to solve pre-commit yaml check.
- changed tox and make commands to match scripts on the bootstack CI.

Closes-Bug: #1978739
Partial-Bug: #1978841

d3dfcb2... by Martin Kalcok on 2022-08-26

Fix incompatibility of flake8 5.0 with some plugins

pycodestyle is inherent dependency of flake8

b5d6a15... by Martin Kalcok on 2022-08-26

Minor changes to tests/code based on review.