Skip to content

chore: Use pytest jubilant for charm integration tests#1053

Open
pedro-avalos wants to merge 5 commits into
mainfrom
use-pytest-jubilant
Open

chore: Use pytest jubilant for charm integration tests#1053
pedro-avalos wants to merge 5 commits into
mainfrom
use-pytest-jubilant

Conversation

@pedro-avalos
Copy link
Copy Markdown
Collaborator

@pedro-avalos pedro-avalos commented May 8, 2026

Description

  • This PR adds pytest-jubilant to charm integration tests
  • This helps in case someone wants to test locally by skipping setup or teardown steps
  • It also extracts the juju fixture out to the fixture provided by pytest-jubilant (which is used by other charms)
  • Lastly, the Juju log can now be dumped and stored as an artifact in the CI
  • I have also removed ops[testing] from integration testing dependencies, as I don't think it's actually needed for integration tests

Resolved issues

Documentation

Web service API changes

Tests

@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.63%. Comparing base (489ea52) to head (4e4933b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1053   +/-   ##
=======================================
  Coverage   77.63%   77.63%           
=======================================
  Files         117      117           
  Lines       12282    12282           
  Branches     1005     1005           
=======================================
  Hits         9535     9535           
  Misses       2525     2525           
  Partials      222      222           
Flag Coverage Δ *Carryforward flag
agent 75.78% <ø> (ø)
cli 91.85% <ø> (ø) Carriedforward from 489ea52
device 63.76% <ø> (ø) Carriedforward from 489ea52
server 87.75% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

Components Coverage Δ
Agent 75.78% <ø> (ø)
CLI 91.85% <ø> (ø)
Common ∅ <ø> (∅)
Device Connectors 63.76% <ø> (ø)
Server 87.75% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pedro-avalos pedro-avalos force-pushed the use-pytest-jubilant branch from e6056f5 to ba44d70 Compare May 8, 2026 16:12
@pedro-avalos pedro-avalos force-pushed the use-pytest-jubilant branch 2 times, most recently from ba44d70 to 4e4933b Compare May 8, 2026 16:16
@pedro-avalos pedro-avalos marked this pull request as ready for review May 8, 2026 16:16
Copy link
Copy Markdown
Contributor

@rene-oromtz rene-oromtz left a comment

Choose a reason for hiding this comment

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

Thank you for taking the time to add this! LGTM
Integration tests are still running but seems a GH issue assigning runners and setting up uv 🤔

Copy link
Copy Markdown
Collaborator

@ajzobro ajzobro left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants