Remove support for custom x-kde-os attribute for screenshots

This is no longer needed now that appstream has an officially supported attribute for this.
PhuNH · Oct 2, 2023 · 4202d58 · 4202d58
1 parent 6df5130
commit 4202d58
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 6 deletions.
diff --git a/py_appstream/subcomponent.py b/py_appstream/subcomponent.py
@@ -190,13 +190,11 @@ def __init__(self):
         self.caption = {}
         self.thumbnails = []
         self.source = None
-        self.os = ''
         self.environment = ''
 
     def parse_tree(self, node, lang_code_func=None):
         """ Parse a <screenshot> object """
         self.default = node.get('type', '') == 'default'
-        self.os = node.get('x-kde-os', '')
         self.environment = node.get('environment', '')
         for c3 in node:
             if c3.tag == 'caption':

diff --git a/tests/test.appdata.xml b/tests/test.appdata.xml
@@ -108,7 +108,7 @@
             <caption xml:lang="x-test">xx"Mixing color of paint" activityxx</caption>
             <image width="1600" height="1040">https://gcompris.net/screenshots_qt/large/color_mix.png</image>
         </screenshot>
-        <screenshot environment="windows" x-kde-os="windows">
+        <screenshot environment="windows">
             <caption>Main view with room list, chat, and room information pane</caption>
             <image>https://cdn.kde.org/screenshots/neochat/NeoChat-Windows-Timeline.png</image>
         </screenshot>

diff --git a/tests/test.py b/tests/test.py
@@ -55,14 +55,11 @@ def test_screenshots(self):
         self.assertEqual({'C', 'ca'}, set(screenshots[2].caption.keys()))
         # environment attribute
         self.assertEqual('windows', self.component.screenshots[4].environment)
-        # x-kde-os attribute
-        self.assertEqual('windows', self.component.screenshots[4].os)
         # serialization
         self.assertIn('source-image', self.obj['Screenshots'][0])
         self.assertEqual('https://gcompris.net/screenshots_qt/large/color_mix.png',
                          self.obj['Screenshots'][3]['source-image']['url'])
         self.assertEqual('windows', self.obj['Screenshots'][4]['environment'])
-        self.assertEqual('windows', self.obj['Screenshots'][4]['os'])
 
     def test_provide(self):
         provide = self.component.provides