Skip to content
This repository has been archived by the owner on Jun 7, 2024. It is now read-only.

Adapt integration tests to reuse the packed charm #50

Merged
merged 3 commits into from
Jul 21, 2023

Conversation

arturo-seijas
Copy link
Collaborator

Adapt the integration tests to reuse the exisintg charm artifact as per canonical/operator-workflows#162

@arturo-seijas arturo-seijas requested a review from a team as a code owner July 21, 2023 09:24
@github-actions
Copy link
Contributor

Test coverage for 3c4acd1

Name           Stmts   Miss Branch BrPart  Cover   Missing
----------------------------------------------------------
src/charm.py     190      0     68      1    99%   306->309
----------------------------------------------------------
TOTAL            190      0     68      1    99%

Static code analysis report

Run started:2023-07-21 09:56:14.109036

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 1133
  Total lines skipped (#nosec): 0
  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):

Copy link
Contributor

@gtrkiller gtrkiller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@arturo-seijas arturo-seijas merged commit 119b3ff into main Jul 21, 2023
@arturo-seijas arturo-seijas deleted the reuse-charm-in-tests branch July 21, 2023 12:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants