Skip to content

feat: add test cases from upstream bluefin issue triage#193

Merged
castrojo merged 1 commit into
projectbluefin:mainfrom
clubanderson:feat/12-upstream-bluefin-test-cases
Jun 19, 2026
Merged

feat: add test cases from upstream bluefin issue triage#193
castrojo merged 1 commit into
projectbluefin:mainfrom
clubanderson:feat/12-upstream-bluefin-test-cases

Conversation

@clubanderson

Copy link
Copy Markdown
Contributor

Summary

Implements the 4 P1 test cases from upstream ublue-os/bluefin issue triage (#12):

1. gnome-software high CPU after close (bluefin#4471)

File: tests/software/features/flatpak.feature

Opens Bazaar, closes it, waits 30s, then checks ps -C gnome-software -o %cpu= is below 5%. Uses a named constant GNOME_SOFTWARE_CPU_THRESHOLD_PCT in the step definition.

2. btrfs compression not enabled on root (bluefin#4655)

File: tests/system/features/filesystem.feature (new)

Checks findmnt / -o OPTIONS for compress mount option when root is btrfs. Gracefully skips on non-btrfs filesystems (e.g. ext4 in CI VMs).

3. ujust devmode re-enable prompt (bluefin#4209)

File: tests/developer/features/devmode.feature (new)

Runs ujust devmode (with just fallback) and asserts the output does not contain phrases like "would you like to enable" or "enable developer mode" — the bug was that it prompted to enable when already active.

4. flatpak repair after Fedora upgrade (bluefin#4403)

File: tests/smoke/features/flatpak_health.feature (new)

Verifies both flatpak repair --user and flatpak repair --system exit 0. Uses the existing Last command output stripped contains step from the smoke suite.

Ref: #12

Test plan

  • python3 -m py_compile passes on all modified/new step files
  • Feature files parse correctly with behave --dry-run
  • Existing smoke/software/developer/system suite scenarios unaffected
  • New scenarios tagged with upstream issue numbers for traceability

Add 4 P1 test cases from upstream ublue-os/bluefin issue triage (projectbluefin#12):

1. bluefin#4471 — gnome-software high CPU after close: opens Bazaar,
   closes it, waits 30s, asserts CPU usage < 5%. Added to
   tests/software/features/flatpak.feature with a custom step that
   parses ps CPU output against a named threshold constant.

2. bluefin#4655 — btrfs compression not enabled on root: new
   tests/system/features/filesystem.feature checks findmnt mount
   options for 'compress' when root is btrfs. Skips gracefully on
   non-btrfs filesystems.

3. bluefin#4209 — ujust devmode prompts when already enabled: new
   tests/developer/features/devmode.feature runs ujust devmode and
   asserts the output does not contain re-enable prompts. Falls back
   to plain 'just' if ujust is not on PATH.

4. bluefin#4403 — flatpak repair after Fedora upgrade: new
   tests/smoke/features/flatpak_health.feature verifies both
   'flatpak repair --user' and '--system' exit cleanly.

Ref: projectbluefin#12
Signed-off-by: unknown <unknown@users.noreply.github.com>

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM — well-structured test/infra addition, clean code.

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good upstream issue triage integration. Merge after #202 (foundational pipeline). LGTM.

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved via org-wide review.

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved via org-wide review.

@castrojo
castrojo merged commit 91c8cd0 into projectbluefin:main Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants