Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

XdpAppInfo needs more test coverage #1555

Open
swick opened this issue Jan 6, 2025 · 0 comments
Open

XdpAppInfo needs more test coverage #1555

swick opened this issue Jan 6, 2025 · 0 comments
Labels
bug

Comments

@swick
Copy link
Collaborator

swick commented Jan 6, 2025

Operating System

Any

XDG Desktop Portal version

Git

XDG Desktop Portal version (Other)

No response

Desktop Environment

Other

Desktop Environment (Other)

No response

Expected Behavior

XdpAppInfo should be covered by our tests.

Current Behavior

There are basic unit tests but all the integration tests use a specific XdpAppInfo subclass and not the real-world host, flatpak and snap ones.

Simple improvement: add more unit tests for all the subclasses

More complex improvement: make the integration tests start a flatpak or snap environment that x-d-p can pick up

Steps to Reproduce

Anything else we should know?

No response
@swick swick added the bug label Jan 6, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
No open projects
Triage
Status: Needs Triage
Milestone
No milestone
Development

No branches or pull requests
1 participant
@swick