Skip to content

remove unused test stubs#431

Open
hippalectryon-0 wants to merge 1 commit intoQubesOS:mainfrom
hippalectryon-0:remove-test-stubs
Open

remove unused test stubs#431
hippalectryon-0 wants to merge 1 commit intoQubesOS:mainfrom
hippalectryon-0:remove-test-stubs

Conversation

@hippalectryon-0
Copy link

@hippalectryon-0 hippalectryon-0 commented Feb 20, 2026

Requires #424

Removed unused ./test-packages

@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.21%. Comparing base (277a03a) to head (ea5325c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #431      +/-   ##
==========================================
+ Coverage   76.20%   76.21%   +0.01%     
==========================================
  Files          53       53              
  Lines        9359     9359              
==========================================
+ Hits         7132     7133       +1     
+ Misses       2227     2226       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hippalectryon-0 hippalectryon-0 marked this pull request as ready for review March 13, 2026 04:39
@marmarek
Copy link
Member

Hm, it used to be used by pylint to check some of the external modules not directly available in the CI env. I see CI passed - does pylint now ignore external modules? I don't see qubesdb on the ignored-modules list and also I'm pretty sure it isn't installed in the CI env. What changed?
dbus stub indeed seems not used anymore.

@hippalectryon-0
Copy link
Author

What changed?

I don't remember making any intentional changes to that effect - I suppose it's been that way for some time now ?

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.

2 participants