canonical / discourse-k8s-operator

discourse-k8s-operator - charm repository.
Apache License 2.0
7 stars 5 forks source link

Fix bash to enable plugins in the fixture in the integration tests #243

Closed javierdelapuente closed 6 months ago

javierdelapuente commented 6 months ago

Overview

The code for enabling plugins in the integrations tests (just after deploying discourse) was not working.

The code has been updated (it is a bash command that runs a rake task in one of the units). Also an assert has been added, so if the command fails, the fixture fails.

Rationale

Juju Events Changes

Module Changes

Library Changes

Checklist

github-actions[bot] commented 6 months ago

Test coverage for c8476141f3d4f4fb03bfafac2eb3a7657171e2c1

Name              Stmts   Miss Branch BrPart  Cover   Missing
-------------------------------------------------------------
src/charm.py        321     27     78     12    90%   186, 194-195, 207, 335->343, 376->381, 393, 575-577, 582-583, 595-597, 602-603, 615-617, 640-642, 684->exit, 694->exit, 724-730, 756->exit, 770-771, 781->exit, 795
src/database.py      30      1      8      1    95%   56
-------------------------------------------------------------
TOTAL               351     28     86     13    91%

Static code analysis report

Run started:2024-05-02 10:26:20.065875

Test results:
    No issues identified.

Code scanned:
    Total lines of code: 2047
    Total lines skipped (#nosec): 3
    Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
    Total issues (by severity):
        Undefined: 0
        Low: 0
        Medium: 0
        High: 0
    Total issues (by confidence):
        Undefined: 0
        Low: 0
        Medium: 0
        High: 0
Files skipped (0):