~woutervb/various/+git/snapstore-proxy-charm:github_actions

Branch merges

Propose for merging

Merged into ~woutervb/various/+git/snapstore-proxy-charm:main at revision 53fa1514e0e02e48936e2af1352188bbcbb349d2

Przemysław Suliga (community): Approve on 2022-01-05

Wouter van Bommel: Pending requested 2021-12-13

Diff: 80 lines (+67/-0)

2 files modified

.github/workflows/buildcharm.yaml (+26/-0)
.github/workflows/tests.yaml (+41/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

5d2c6f9... by Wouter van Bommel on 2022-01-06

Replace test with lint as coverage also tests

3e7c00e... by Wouter van Bommel on 2021-12-13

Added github workflow

* Tests are run automatically on creation of a PR / merge
* On a release a charm will be build (todo push to charmhub)

138b0e7... by Wouter van Bommel on 2021-12-10

Added status action and refactored test

Refactored test of the charm in logical subsets, to ease expansion and
keep grouped tests together.

Added an status action, that will return the result of the proxy status
command.

Fixed a dependency issue in the Makefile, which prevented the charm from
being rebuild on changes to the actions.yaml file.

Added the action to the documentation.

19a8db2... by Wouter van Bommel on 2021-12-09

General improvements

* Some log files have been improved to be more explicit on what is going
on
* Renamed the db interface to db-admin, as that is the one we need
* Allow the reset of the username / password after registration (closes
SN-363)
* Don't allow the update of the domain to proceed, once the proxy is
registered

7975b3f... by Wouter van Bommel on 2022-01-06

Minor tweaks after rebasing against coverage branch

Based on review feedback, ensured that linting and tests succeed. There
was a double named test function.

f6f6e28... by Wouter van Bommel on 2021-12-09

General improvements

* Some log files have been improved to be more explicit on what is going
on
* Renamed the db interface to db-admin, as that is the one we need
* Allow the reset of the username / password after registration (closes
SN-363)
* Don't allow the update of the domain to proceed, once the proxy is
registered

046eaf5... by Wouter van Bommel on 2021-12-08

Ensure 100% coverage

Added some missing tests to ensure everything is covered.

1f1f808... by Wouter van Bommel on 2022-01-06

Tweak based on review feedback

Ensure that coverage itself fails if coverage is not 100%
Fixed a linting error

9c9bbb2... by Wouter van Bommel on 2021-12-08

Ensure 100% coverage

Added some missing tests to ensure everything is covered.

1f87662... by Wouter van Bommel on 2022-01-06

Fix a unittest that failed with None as default value