replicate when platform.name is absent in windows

duckduckgo · Aug 7, 2024 · f319e14 · f319e14
1 parent d369580
commit f319e14
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 16 deletions.
diff --git a/packages/special-pages/pages/duckplayer/app/index.js b/packages/special-pages/pages/duckplayer/app/index.js
@@ -56,6 +56,7 @@ export async function init (messaging, baseEnvironment) {
             })
 
     const settings = new Settings({})
+        .withPlatformName(baseEnvironment.platform)
         .withPlatformName(init.platform?.name)
         .withPlatformName(baseEnvironment.urlParams.get('platform'))
         .withFeatureState('pip', init.settings.pip)

diff --git a/packages/special-pages/pages/duckplayer/app/providers/SettingsProvider.jsx b/packages/special-pages/pages/duckplayer/app/providers/SettingsProvider.jsx
@@ -49,17 +49,7 @@ export function useOpenSettingsHandler() {
 }
 
 export function useSettingsUrl() {
-    const settings = useContext(SettingsContext).settings;
-    switch (settings.platform.name) {
-        case "windows":
-            return "duck://settings/duckplayer"
-        case "macos":
-            return "about:preferences/duckplayer"
-        default: {
-            console.warn("unreachable!")
-        }
-    }
-    return ''
+    return 'duck://settings/duckplayer'
 }
 
 export function useSettings() {

diff --git a/packages/special-pages/tests/page-objects/duck-player.js b/packages/special-pages/tests/page-objects/duck-player.js
@@ -59,7 +59,7 @@ export class DuckPlayerPage {
                 },
                 locale: 'en',
                 env: 'development',
-                platform: { name: this.platform.name }
+                platform: this.platform.name === 'windows' ? undefined : { name: this.platform.name }
             },
             /** @type {import('../../types/duckplayer.js').UserValues} */
             getUserValues: {
@@ -325,10 +325,7 @@ export class DuckPlayerPage {
             this.page.context().on('requestfailed', resolve)
         })
 
-        const expected = this.build.switch({
-            windows: () => 'duck://settings/duckplayer',
-            apple: () => 'about:preferences/duckplayer'
-        })
+        const expected = 'duck://settings/duckplayer'
 
         const openSettings = this.page.getByRole('link', { name: 'Open Settings' })
         expect(await openSettings.getAttribute('href')).toEqual(expected)