chore: update attribution

README.md

@@ -35,6 +35,8 @@ Geolocation records from [DB-IP](https://db-ip.com) under [CC 4.0 license](https
 Executable versions of DZGUI published as release binaries ship with various runtime dependencies and the Python interpreter built in.
 Users wishing to review the licenses to these components can inspect the `LICENSE` file located in the release tarball.
 
+Steam and the Steam logo are trademarks and/or registered trademarks of Valve Corporation in the U.S. and/or other countries.
+
 ## Disclaimer
 
 DZGUI is beta-quality software and is provided as-is.

dzgui/api/shortcuts.py

@@ -10,7 +10,7 @@ from pathlib import Path
 from typing import Any
 
 from dzgui.api.steam import find_user_id_32
-from dzgui.const.constants import APP_NAME, APP_NAME_LOWER, IMAGES_PATH
+from dzgui.const.constants import APP_NAME, APP_NAME_LOWER, GTK_APP_ID, IMAGES_PATH
 from dzgui.util.dirs import copy_dzgui_to_xdg_data, make_parents
 from dzgui.util.strings import unknown
 
@@ -158,7 +158,7 @@ class Shortcuts:
         NEW_ENTRY["DevkitGameID"] = ""
         NEW_ENTRY["DevkitOverrideAppID"] = 0
         NEW_ENTRY["LastPlayTime"] = 0
-        NEW_ENTRY["FlatpakAppID"] = ""
+        NEW_ENTRY["FlatpakAppID"] = GTK_APP_ID
 
         NEW_ENTRY["sortas"] = ""
         NEW_ENTRY["tags"] = {}

dzgui/api/steam.py

@@ -13,7 +13,6 @@ from warnings import deprecated
 
 from dzgui.api.acf import ACF
 from dzgui.init.prereqs import has_any_client
-from dzgui.init.proc import has_cmd
 from dzgui.const.constants import (
     APPID_DAYZ,
     APPID_DAYZ_EXP,
@@ -23,7 +22,6 @@ from dzgui.const.constants import (
     DEBIAN_STEAM_PATH,
     DEFAULT_STEAM_PATH,
     DOWNLOADS_PATH,
-    FLATPAK_CMD,
     FLATPAK_STEAM_PATH,
     LIBRARYFOLDERS_PATH,
     UBUNTU_STEAM_PATH,
@@ -84,8 +82,6 @@ def get_steam_paths() -> list[tuple[Path, str]]:
         }
 
         for path in DEFAULT_PATH, FLATPAK_PATH, UBUNTU_PATH, DEBIAN_PATH:
-            if path is FLATPAK_PATH and has_cmd(FLATPAK_CMD) is False:
-                continue
             if path.joinpath(VDF_PATH).is_file():
                 paths.append((path, human[path]))
     return paths
@@ -202,7 +198,6 @@ def load_to_menu(client: str, appid: int, name: str, mods: list[str]) -> int:
     """Loads to the menu screen with the selected mods; used for AFK/pre-joining"""
     concat = concat_mods(mods)
     client_args = concat_bash_args(client)
-    logger.debug(f"Setting client arguments to: {client_args}")
     params = [
         "-applaunch",
         str(appid),

dzgui/app_init.py

@@ -2,17 +2,12 @@ import logging
 import os
 import shutil
 
+from importlib import resources
 from pathlib import Path
 from typing import TYPE_CHECKING
-from warnings import deprecated
 
 
-from dzgui.const.constants import (
-    APP_NAME,
-    FLATPAK_ROOT_PATH,
-    STEAM_ICON,
-    STEAM_ICONS_PATH,
-)
+from dzgui.const.constants import APP_NAME, APP_NAME_LOWER, STEAM_ICON, ICONS_PATH
 from dzgui.const.enum import Preferences
 from dzgui.config.query import lookup
 from dzgui.config.userprefs import UserPrefs
@@ -23,7 +18,7 @@ from dzgui.init.migrate import (
     migrate_cols_file,
 )
 from dzgui.init.flock import lock_acquire
-from dzgui.init.prereqs import has_flatpak_client, has_steam_client
+from dzgui.init.prereqs import has_any_client
 from dzgui.strings import boot
 
 from dzgui.util.deck import is_steam_deck, is_game_mode
@@ -31,7 +26,7 @@ from dzgui.util.dirs import make_parents
 from dzgui.util.localize import set_locale
 from dzgui.util.map_count import test_map_count
 from dzgui.util.redact import RedactionFilter, REDACTION_PATTERNS
-from dzgui.util.strings import init, steam_icon_missing
+from dzgui.util.strings import init
 
 from dzgui.views.base import App
 from dzgui.views.dialogs.boot import BootWindow
@@ -104,26 +99,20 @@ def copy_bare_configs(config: "Path", resolution: "Path") -> tuple[bool, bool]:
     return (config_changed, state_changed)
 
 
-@deprecated("Currently unused")
-def load_steam_icon(config: Path) -> None:
+def load_steam_icon() -> None:
     import gi
 
     gi.require_version("Gtk", "3.0")
-    from gi.repository import Gtk  # noqa
-
-    default = lookup(config, Preferences.DEFAULT)
-    icon_path = Path(default).joinpath(STEAM_ICONS_PATH)
+    from gi.repository import Gtk # noqa
     try:
+        traversable = resources.files(APP_NAME_LOWER).joinpath(ICONS_PATH)
+        icons = Path(str(traversable))
+
         theme = Gtk.IconTheme.get_default()
         paths = theme.get_search_path()
-        theme.set_search_path(paths + [str(icon_path)])
-        paths = theme.get_search_path()
-        icons = theme.list_icons(None)
-        if STEAM_ICON not in icons:
-            logger.warning(steam_icon_missing)
+        theme.set_search_path(paths + [str(icons)])
     except Exception as e:
-        print(e)
-        logger.warning(e)
+        logger.critical(e)
 
 
 def load_gui(version: str, is_debug: bool) -> None:
@@ -153,7 +142,7 @@ def load_gui(version: str, is_debug: bool) -> None:
         migrate_cols_file(XDG.columns)
         copy_state_files(xdg_paths["XDG_STATE_HOME"])
         # TODO: add logging inside wizard
-        wizard = SetupWizard(_is_steam_deck, XDG, xdg_paths)
+        wizard = SetupWizard(_is_steam_deck, XDG)
 
         if not wizard.is_setup_complete():
             return
@@ -166,11 +155,7 @@ def load_gui(version: str, is_debug: bool) -> None:
         # TODO: sudo escalation dialog
         # count = get_map_count()
         # TODO: move into module
-        if FLATPAK_ROOT_PATH in lookup(XDG.config, Preferences.DEFAULT):
-            has_client = has_flatpak_client()
-        else:
-            has_client = has_steam_client()
-        if has_client is False:
+        if has_any_client() is False:
             EarlyAlertDialog(init.requires_steam)
 
     bootwin = BootWindow(XDG, version)
@@ -189,5 +174,6 @@ def load_gui(version: str, is_debug: bool) -> None:
         use_miles=use_miles,
     )
     print(boot.all_ok)
+    load_steam_icon()
     app = App(prefs)
     app.run(None)

dzgui/config/xdg.py

@@ -3,13 +3,7 @@ import os
 from pathlib import Path
 from dataclasses import dataclass
 
-from dzgui.const.constants import (
-    APP_NAME_LOWER,
-    DEBUG_LOG,
-    FLATPAK_ROOT_PATH,
-    SYSTEM_LOG,
-)
-from dzgui.init.proc import is_sandboxed
+from dzgui.const.constants import APP_NAME_LOWER, DEBUG_LOG, SYSTEM_LOG
 
 
 @dataclass
@@ -59,12 +53,10 @@ def get_xdg_paths() -> dict:
         "XDG_DATA_HOME": home / ".local" / "share",
         "XDG_CACHE_HOME": home / ".cache",
     }
+
     resolved_paths = {}
     for path in xdg_paths:
         real_path = os.environ.get(path)
-        if real_path is not None and is_sandboxed():
-            search_string = FLATPAK_ROOT_PATH + "/"
-            real_path = real_path.replace(search_string, "")
         if real_path is not None and is_writeable(real_path):
             new_path = Path(real_path)
         else:

dzgui/const/constants.py

@@ -51,7 +51,7 @@ INPUT_KEYBOARD = "input-keyboard-symbolic"
 LIST_ADD = "list-add-symbolic"
 REFRESH_ICON = "view-refresh-symbolic"
 SEARCH_ICON = "system-search-symbolic"
-STEAM_ICON = "steam_tray_mono"
+STEAM_ICON = "steam-tray-symbolic"
 VIEW_CONCEAL = "view-conceal-symbolic"
 VIEW_REVEAL = "view-reveal-symbolic"
 WARNING = "dialog-warning-symbolic"
@@ -80,6 +80,7 @@ DEBUG_LOG = f"{APP_NAME}_DEBUG.LOG"
 SYSTEM_LOG = f"{APP_NAME}_SYSTEM.LOG"
 CHANGELOG_PATH = "data/CHANGELOG.md"
 IMAGES_PATH = "data/images"
+ICONS_PATH = "data/images/gtk"
 
 HERO_PATH = "data/images/_hero.png"
 CSS_PATH = "data/app.css"
@@ -89,7 +90,6 @@ FLATPAK_ROOT_PATH = ".var/app/com.valvesoftware.Steam"
 FLATPAK_STEAM_PATH = FLATPAK_ROOT_PATH + "/data/Steam"
 UBUNTU_STEAM_PATH = ".steam/steam"
 DEBIAN_STEAM_PATH = ".steam/debian-installation"
-STEAM_ICONS_PATH = "public"
 
 LOG_FILTERS = ("CRITICAL", "WARNING", "INFO", "DEBUG")
 

dzgui/data/images/gtk/steam-tray-symbolic.svg

@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 640 640"><!--!Font Awesome Free 7.3.1 by @fontawesome - https://fontawesome.com License - https://fontawesome.com/license/free Copyright 2026 Fonticons, Inc.--><path d="M568 320C568 457 456.8 568 319.6 568C205.8 568 110 491.7 80.6 387.6L175.8 426.9C182.2 459 210.7 483.3 244.7 483.3C283.9 483.3 316.6 450.9 314.9 409.8L399.4 349.6C451.5 350.9 495.2 308.7 495.2 256.1C495.2 204.5 453.2 162.6 401.5 162.6C349.8 162.6 307.8 204.6 307.8 256.1L307.8 257.3L248.6 343C233.1 342.1 217.9 346.4 205.1 355.1L72 300.1C82.2 172.4 189.1 72 319.6 72C456.8 72 568 183 568 320zM227.7 448.3L197.2 435.7C202.8 447.3 212.5 456.5 224.4 461.5C251.3 472.7 282.2 459.9 293.4 433.1C298.8 420.1 298.9 405.8 293.5 392.8C288.1 379.8 278 369.6 265 364.2C252.1 358.8 238.3 359 226.1 363.6L257.6 376.6C277.4 384.8 286.8 407.5 278.5 427.3C270.2 447.2 247.5 456.5 227.7 448.3zM401.5 193.8C435.9 193.8 463.8 221.7 463.8 256.1C463.8 290.5 435.9 318.4 401.5 318.4C367.1 318.4 339.2 290.5 339.2 256.1C339.2 221.7 367.1 193.8 401.5 193.8zM401.6 302.8C427.4 302.8 448.4 281.8 448.4 256C448.4 230.2 427.4 209.2 401.6 209.2C375.8 209.2 354.8 230.2 354.8 256C354.8 281.8 375.8 302.8 401.6 302.8z"/></svg>

dzgui/init/proc.py

@@ -1,8 +1,7 @@
-import logging
-import os
 import psutil
-import shutil
 import subprocess
+import shutil
+import logging
 
 from dzgui.const.constants import (
     APP_NAME,
@@ -27,13 +26,7 @@ def is_steam_running(cmd: str) -> bool:
         raise TypeError("Not a valid Steam client selection")
 
 
-def is_sandboxed() -> bool:
-    return bool(os.environ.get("FLATPAK_ID"))
-
-
 def is_flatpak_steam_running() -> bool:
-    if is_sandboxed():
-        return True
     if has_cmd(FLATPAK_CMD) is False:
         return False
     proc = subprocess.check_output(

dzgui/strings/wizard.py

@@ -3,8 +3,6 @@ title_intro = "Welcome"
 blurb_intro = """
 This wizard will help you set up some common config options before launching the application.
 """
-error_sandbox = "Setup cannot be run from inside Flatpak sandbox.\nPlease launch the application directly."
-welcome = "Welcome to {0}!"
 
 ### SteamPathPage
 heading_steam_path = "Steam path"
@@ -82,17 +80,6 @@ checkbox_start_menu = (
 )
 checkbox_desktop_shortcut = ("Add a desktop shortcut", "Depends on the option above.")
 
-### Flatpak page
-heading_flatpak = "Flatpak sandbox overrides"
-blurb_flatpak = (
-    "You chose to add a shortcut directly to Flatpak Steam. "
-    "To launch DZGUI from Flatpak's sandbox, you must grant "
-    "permission to read from and write to the following locations."
-)
-tree_column = "Paths"
-overrides_applied = "Overrides applied."
-button_flatpak = "Grant permissions"
-
 ### Completion page
 heading_completion = "Setup complete"
 blurb_completion = "Configuration completed successfully. Click 'Close' to close this dialog and launch DZGUI."

dzgui/util/bash.py

@@ -1,12 +1,5 @@
 import shlex
 
-from dzgui.const.constants import STEAM_CMD
-from dzgui.init.proc import is_sandboxed
-
 
 def concat_bash_args(command: str) -> list[str]:
-    # NOTE: NSGs spawned from sandbox cannot access host application commands,
-    # so fallback to Steam internal URIs
-    if is_sandboxed():
-        return [STEAM_CMD]
     return shlex.split(command)

dzgui/util/flatpak.py

@@ -1,31 +0,0 @@
-import logging
-import subprocess
-
-from dzgui.const.constants import APP_NAME, FLATPAK_APPID, GTK_APP_ID
-
-logger = logging.getLogger(APP_NAME)
-
-
-def apply_overrides(xdg: dict[str, str]) -> None:
-    try:
-        # NOTE: D-Bus registration required for sandbox
-        args = [
-            "flatpak",
-            "--user",
-            "override",
-            f"--own-name={GTK_APP_ID}",
-            FLATPAK_APPID,
-        ]
-        subprocess.run([*args], capture_output=True)
-
-        for path in xdg.values():
-            args = [
-                "flatpak",
-                "--user",
-                "override",
-                f"--filesystem={path}",
-                FLATPAK_APPID,
-            ]
-            subprocess.run([*args], capture_output=True)
-    except Exception as e:
-        logger.critical(e)

dzgui/util/strings.py

@@ -297,7 +297,7 @@ class Options:
 init = Init(
     is_steam_running="Is Steam running? DZGUI must be run on top of Steam.",
     is_dayz_running="Is DayZ already running? DZGUI cannot launch DayZ if another process is using it.",
-    requires_steam="Your chosen client (Steam/Flatpak Steam) must be installed on the system.",
+    requires_steam="DZGUI requires that Steam or Flatpak Steam be installed on the system.",
 )
 
 mod_panel = ModPanelStrings(

dzgui/views/dialogs/wizard.py

@@ -12,24 +12,22 @@ from dzgui.const.constants import (
     APP_NAME,
     APP_NAME_LOWER,
     HERO_PATH,
-    FLATPAK_ROOT_PATH,
+    FLATPAK_STEAM_PATH,
     LEGACY_CONFIG_PATH,
 )
 from dzgui.const.boilerplate import config_boilerplate
 from dzgui.const.endpoints import STEAM_API_SETUP
 from dzgui.config import freedesktop
-from dzgui.init.proc import is_sandboxed, path_to_steam_cmd
+from dzgui.init.proc import path_to_steam_cmd
 from dzgui.init.migrate import migrate_legacy_conf
 from dzgui.managers.threading import call_on_thread, StoredFunc, ThreadingManager
 from dzgui.strings import wizard
 from dzgui.util._json import write_json
-from dzgui.util.flatpak import apply_overrides
 from dzgui.util.open_links import open_link_by_url
 from dzgui.util.css import add_class, load_css
 from dzgui.views.components.buttons import WebButton
 from dzgui.views.components.entry import APIEntry
 from dzgui.views.mixins.colorscheme import ColorAwareApp
-from dzgui.views.trees.tree_simple import SimpleTree
 
 import gi
 
@@ -47,8 +45,7 @@ class PageNum(Enum):
     STEAM_API = 4
     USER_PREFS = 5
     SHORTCUTS = 6
-    FLATPAK = 7
-    FINAL = 8
+    FINAL = 7
 
 
 class OptionalPageMixin:
@@ -147,7 +144,7 @@ class NotificationFrame(Gtk.Frame):
             margin_start=50,
             margin_end=50,
         )
-        wrapped = textwrap.fill(label, width=60)
+        wrapped = textwrap.fill(label, width=80)
         self.label = Gtk.Label(
             label=wrapped,
             justify=Gtk.Justification.CENTER,
@@ -161,7 +158,7 @@ class NotificationFrame(Gtk.Frame):
             add_class(self, "error-frame")
 
     def set_text(self, text: str) -> None:
-        wrapped = textwrap.fill(text, width=60)
+        wrapped = textwrap.fill(text, width=80)
         self.label.set_markup(wrapped)
 
 
@@ -246,23 +243,12 @@ class IntroductionPage(EnumeratedWizardPage):
     def __init__(self) -> None:
         super().__init__(
             enum=PageNum.INTRO,
-            heading=wizard.welcome.format(APP_NAME),
+            heading=f"Welcome to {APP_NAME}!",
             description=wizard.blurb_intro,
         )
         self.page_type = Gtk.AssistantPageType.INTRO
         self.set_title(wizard.title_intro)
 
-        self.error_box = NotificationFrame(wizard.error_sandbox, error=True)
-        self.add_start(self.error_box)
-
-        self.connect("map", self._on_map)
-
-    def _on_map(self, page: "EnumeratedWizardPage") -> None:
-        self.error_box.hide()
-        if is_sandboxed():
-            self.error_box.show()
-            EMITTER.emit("step_pending")
-
 
 class Heading(Gtk.Label):
     def __init__(self, label: str):
@@ -419,42 +405,6 @@ class PreferencesPage(EnumeratedWizardPage):
         EMITTER.emit("step_complete")
 
 
-class FlatpakOverridePage(EnumeratedWizardPage):
-    def __init__(self, xdg_paths: dict[str, str]) -> None:
-        super().__init__(
-            enum=PageNum.FLATPAK,
-            heading=wizard.heading_flatpak,
-            description=wizard.blurb_flatpak,
-        )
-        self.xdg_paths = xdg_paths
-        self.page_type = Gtk.AssistantPageType.INTRO
-
-        store = Gtk.ListStore(str)
-        for path in xdg_paths.values():
-            store.append((str(path),))
-        tree = SimpleTree(store, wizard.tree_column)
-        tree.set_halign(Gtk.Align.CENTER)
-
-        self.authorize_button = Gtk.Button(
-            label=wizard.button_flatpak, halign=Gtk.Align.CENTER
-        )
-        self.authorize_button.connect("clicked", self._on_authorize)
-
-        self.notice = NotificationFrame(wizard.overrides_applied)
-        self.connect("map", self._on_map)
-
-        for el in tree, self.authorize_button, self.notice:
-            self.add_start(el)
-
-    def _on_map(self, page: "EnumeratedWizardPage") -> None:
-        self.notice.hide()
-
-    def _on_authorize(self, button: Gtk.Button) -> None:
-        apply_overrides(self.xdg_paths)
-        self.notice.show()
-        EMITTER.emit("step_complete")
-
-
 class CompletionPage(EnumeratedWizardPage):
     def __init__(self) -> None:
         super().__init__(
@@ -468,7 +418,7 @@ class CompletionPage(EnumeratedWizardPage):
 
 
 class Assistant(Gtk.Assistant):
-    def __init__(self, is_deck: bool, XDG: "Xdg", xdg_paths: dict[str, str]):
+    def __init__(self, is_deck: bool, XDG: "Xdg"):
         super().__init__()
         if is_deck:
             # NOTE: deemed to be "safe" dimensions that exclude taskbar size
@@ -487,7 +437,6 @@ class Assistant(Gtk.Assistant):
         self.page_api = SteamValidationPage()
         self.page_prefs = PreferencesPage()
         self.page_shortcuts = ShortcutCreationPage(XDG.shortcut)
-        self.page_flatpak = FlatpakOverridePage(xdg_paths)
         self.page_completion = CompletionPage()
 
         self.set_forward_page_func(self._advance_page)
@@ -507,7 +456,6 @@ class Assistant(Gtk.Assistant):
             self.page_api,
             self.page_prefs,
             self.page_shortcuts,
-            self.page_flatpak,
             self.page_completion,
         ):
             # NOTE: skip config migration page if no legacy config file
@@ -528,10 +476,7 @@ class Assistant(Gtk.Assistant):
     def write_config(self, config: dict[str, Any]) -> None:
         write_json(config, self.config_path)
 
-    def get_completion_page(self) -> int:
-        return self.pages[self.page_completion]
-
-    def get_shortcut_page(self) -> int:
+    def get_final_page(self) -> int:
         return self.pages[self.page_shortcuts]
 
     def _advance_page(self, index: int) -> int:
@@ -544,7 +489,7 @@ class Assistant(Gtk.Assistant):
                     sp = page.migrated_conf["default_steam_path"]
                     self.page_shortcuts.set_steam_path(sp)
                     self.config_values = self.page_migration.migrated_conf
-                    return self.get_shortcut_page()
+                    return self.get_final_page()
             case SteamPathPage():
                 path = page.get_path_from_radio()
                 self.config_values["default_steam_path"] = path
@@ -561,22 +506,10 @@ class Assistant(Gtk.Assistant):
                 )
             case ShortcutCreationPage():
                 page.create_shortcuts()
-                if not self.needs_flatpak_override():
-                    return self.get_completion_page()
-            case FlatpakOverridePage():
-                pass
             case _:
                 raise AttributeError("Trying to advance a non-canonical page")
         return index + 1
 
-    def needs_flatpak_override(self) -> bool:
-        if (
-            self.page_shortcuts.is_steam_checkbox_active()
-            and FLATPAK_ROOT_PATH in self.config_values["default_steam_path"]
-        ):
-            return True
-        return False
-
     def is_setup_complete(self) -> bool:
         return self.setup_complete
 
@@ -692,6 +625,12 @@ class ShortcutCreationPage(OptionalPageMixin, EnumeratedWizardPage):  # type: ig
         self.add_start(self.checks_area)
         self.show_all()
         self.connect("map", self._on_map)
+        self.connect("map", self._toggle_checkbox)
+
+    def _toggle_checkbox(self, page: "EnumeratedWizardPage") -> None:
+        if FLATPAK_STEAM_PATH in str(self.steam_path):
+            self.steam_checkbox.set_active(False)
+            self.steam_checkbox.set_sensitive(False)
 
     def _on_start_menu_toggled(self, button: Gtk.CheckButton) -> None:
         state = button.get_active()
@@ -702,9 +641,6 @@ class ShortcutCreationPage(OptionalPageMixin, EnumeratedWizardPage):  # type: ig
     def set_steam_path(self, path: Path) -> None:
         self.steam_path = path
 
-    def is_steam_checkbox_active(self) -> bool:
-        return self.steam_checkbox.get_active()
-
     def create_shortcuts(self) -> None:
         # NOTE: best-effort, permissive even on failure (page is already marked as complete)
         if self.steam_checkbox.get_active():
@@ -773,10 +709,10 @@ class SteamPathPage(EnumeratedWizardPage):
 
 
 class SetupWizard(ColorAwareApp, Gtk.Application):  # type: ignore
-    def __init__(self, is_deck: bool, XDG: "Xdg", xdg_paths: dict[str, str]) -> None:
+    def __init__(self, is_deck: bool, XDG: "Xdg") -> None:
         super().__init__()
         GLib.set_prgname(APP_NAME)
-        self.win = Window(is_deck, XDG, xdg_paths)
+        self.win = Window(is_deck, XDG)
         Gtk.main()
 
     def is_setup_complete(self) -> int:
@@ -784,9 +720,9 @@ class SetupWizard(ColorAwareApp, Gtk.Application):  # type: ignore
 
 
 class Window(Gtk.Window):
-    def __init__(self, is_deck: bool, XDG: "Xdg", xdg_paths: dict[str, str]) -> None:
+    def __init__(self, is_deck: bool, XDG: "Xdg") -> None:
         super().__init__(title=APP_NAME, icon_name=APP_NAME)
-        self.assistant = Assistant(is_deck, XDG, xdg_paths)
+        self.assistant = Assistant(is_deck, XDG)
 
 
 class Emitter(GObject.GObject):

dzgui/views/pages/options.py

@@ -266,6 +266,7 @@ class FlatpakToggleField(ToggleField):
 
     def _on_radio_toggled(self, button: Gtk.RadioButton, context: Preferences) -> None:
         cmd = FLATPAK_RUN_CMD if self.radio1.get_active() else FLATPAK_SANDBOX
+        print(context)
         self.controller.update_config(context, cmd)
 
 

dzgui/views/trees/tree_simple.py

@@ -1,35 +0,0 @@
-from typing import TYPE_CHECKING, Union
-
-if TYPE_CHECKING:
-    from dzgui.model.model_factory import FastInsertListStore
-
-import gi
-
-gi.require_version("Gtk", "3.0")
-from gi.repository import Gtk  # noqa
-
-
-class SimpleTree(Gtk.ScrolledWindow):
-    """Simple scrollable 1-column tree"""
-
-    def __init__(
-        self, model: Union[Gtk.ListStore, "FastInsertListStore"], title: str
-    ) -> None:
-        super().__init__(
-            overlay_scrolling=False,
-            propagate_natural_width=True,
-            propagate_natural_height=True,
-        )
-
-        self.tree = Gtk.TreeView(model=model, enable_search=False)
-        renderer = Gtk.CellRendererText()
-        column = Gtk.TreeViewColumn(title, renderer, text=0)
-        column.set_sizing(Gtk.TreeViewColumnSizing.FIXED)
-        column.set_resizable(False)
-        column.set_sort_column_id(0)
-        self.tree.append_column(column)
-
-        self.add(self.tree)
-
-    def get_tree(self) -> Gtk.TreeView:
-        return self.tree

pyproject.toml

@@ -110,11 +110,10 @@ config-settings-package = { pygobject-stubs = { config = "Gtk3,Gdk3,Soup2" } }
 markers = [
     "apitest: checks remote endpoints",
     "config: config file keys/values",
-    "flatpak: depends on local flatpak installation",
+    "realconfig: depends on actual generated local config file",
     "mods: tests mod metadata/link creation",
     "pefile: validate PE files",
     "post_install: requires a completed installation",
-    "realconfig: depends on actual generated local config file",
     "redact: log redaction mechanisms",
     "slow: long-running tests",
     "webtest: depends on remote endpoint",

tests/test_flatpak_override.py

@@ -1,43 +0,0 @@
-import configparser
-import pytest
-
-from dzgui.const.constants import FLATPAK_APPID, GTK_APP_ID
-from dzgui.config.xdg import get_xdg_paths
-from dzgui.util.flatpak import apply_overrides
-
-PATHS_KEY = "Context"
-PATHS_SUBKEY = "filesystems"
-BUS_KEY = "Session Bus Policy"
-
-pytestmark = pytest.mark.flatpak
-
-
-@pytest.fixture
-def xdg_paths() -> dict[str, str]:
-    return get_xdg_paths()
-
-
-@pytest.fixture
-def override(xdg_paths) -> None:
-    apply_overrides(xdg_paths)
-
-
-@pytest.fixture
-def config(xdg_paths) -> configparser.ConfigParser:
-    conf_path = xdg_paths["XDG_DATA_HOME"].parent.joinpath(
-        f"flatpak/overrides/{FLATPAK_APPID}"
-    )
-    config = configparser.ConfigParser()
-    config.read(conf_path)
-    return config
-
-
-def test_session_bus(override, config) -> None:
-    bus = config[BUS_KEY][GTK_APP_ID]
-    assert bus == "own"
-
-
-def test_paths(override, xdg_paths, config) -> None:
-    expect = xdg_paths
-    paths = config[PATHS_KEY][PATHS_SUBKEY].split(";")
-    assert all(str(path) in paths for path in expect.values())

tests/test_flatpak_toggle.py

@@ -1,30 +0,0 @@
-import pytest
-
-from dzgui.const.constants import FLATPAK_RUN_CMD, FLATPAK_SANDBOX
-from dzgui.const.enum import Preferences
-from dzgui.views.pages.options import FlatpakToggleField
-
-
-class MockController:
-    def __init__(self) -> None:
-        self.command: str
-
-    def update_config(self, context: Preferences, cmd: str) -> None:
-        self.command = cmd
-
-    def get_command(self) -> str:
-        return self.command
-
-
-@pytest.mark.parametrize("index, expect", [(0, FLATPAK_RUN_CMD), (1, FLATPAK_SANDBOX)])
-def test_toggle(index: int, expect: str) -> None:
-    controller = MockController()
-    toggle = FlatpakToggleField(controller)
-    match index:
-        case 0:
-            toggle.radio2.set_active(True)
-            toggle.radio1.set_active(True)
-        case 1:
-            toggle.radio1.set_active(True)
-            toggle.radio2.set_active(True)
-    assert controller.get_command() == expect

tests/test_html_escaping.py

@@ -1,5 +1,3 @@
-import pytest
-
 from dzgui.api.servers import Details
 from dzgui.views.dialogs.servers import ServerDetailsDialog
 
@@ -23,7 +21,6 @@ class MockController:
         return None
 
 
-@pytest.mark.filterwarnings("ignore::DeprecationWarning")
 def test_html_escaping() -> None:
     name = "Test server"
     desc = "<Server description> Game & mods"
@@ -34,7 +31,5 @@ def test_html_escaping() -> None:
     controller = MockController()
     dialog = ServerDetailsDialog(controller, details)
     dialog.destroy()
-    while Gtk.events_pending():
-        Gtk.main_iteration()
 
     assert dialog.description.get_text() == desc

tests/test_process_detection.py

@@ -1,4 +1,3 @@
-import pytest
 import unittest
 from unittest.mock import patch
 
@@ -10,8 +9,6 @@ from dzgui.const.constants import (
 )
 from dzgui.init import proc
 
-pytestmark = pytest.mark.flatpak
-
 
 class TestSteamDetection(unittest.TestCase):
     def test_flatpak_detection_with_multiple_running_apps(self):