[v6] Fix platform/device determination (#610)

* Fix dependency

* Fix 582 issue

* Fix 582 issue

* Fix 582 issue

* Fix 582 issue

* Fix 582 issue

* Fix 582 issue

* Fix 582 issue

* Upgrade eslint packages

* Update dependencies

* Fix 582 issue

* Resolve comments

* Resolve comments

* Change ConfigCapabilities to DesiredCapabilitiesExtended

* Resolve comments

* Resolve comments

* Resolve issue (device/platform info)

---------

Co-authored-by: Hanna Tarasevich <htarasevich@deloitte.com>

Author: HannaTarasevich

src/metadata.ts

@@ -23,11 +23,11 @@ export class Metadata {
                 // Fixes: https://github.com/webdriverio-community/wdio-cucumberjs-json-reporter/issues/73
                 ? data.capabilities['cjson:metadata'] as cjson_metadata
                 // Fallback
-                : (browser as WebdriverIOExtended)?.requestedCapabilities?.cjson_metadata
+                : (browser as WebdriverIOExtended)?.requestedCapabilities?.['cjson:metadata'] as cjson_metadata
             : {}
-        const metadata: cjson_metadata = (currentConfigCapabilities as W3CCapabilitiesExtended)?.cjson_metadata
+        const metadata: cjson_metadata = (currentConfigCapabilities as W3CCapabilitiesExtended)?.['cjson:metadata']
             || w3cCaps // When an app is used to test
-            || (optsCaps as DesiredCapabilitiesExtended)?.cjson_metadata // devtools
+            || (optsCaps as DesiredCapabilitiesExtended)?.['cjson:metadata'] // devtools
             || {} as cjson_metadata
 
         // When an app is used to test

src/types/wdio.ts

@@ -13,27 +13,27 @@ import type { cjson_metadata } from '../types'
 import { W3CCapabilities } from '@wdio/types/build/Capabilities'
 
 export interface W3CCapabilitiesExtended extends W3CCapabilities {
-    cjson_metadata?: cjson_metadata
+    'cjson:metadata'?: cjson_metadata
     app?: string
 }
 
 export interface DesiredCapabilitiesExtended extends WebdriverIO.Capabilities {
-    cjson_metadata?: cjson_metadata
+    'cjson:metadata'?: cjson_metadata
     app?: string
 }
 
 export interface TestrunnerExtended extends Options.Testrunner {
     jsonFolder?: string
     language?: string
-    cjson_metadata?: cjson_metadata
+    'cjson:metadata'?: cjson_metadata
     logFile?: string
     stdout?: boolean
     writeStream?: WriteStream
 }
 export interface WebdriverIOExtended extends Options.WebdriverIO {
     app?: string
     requestedCapabilities?: {
-        cjson_metadata?: cjson_metadata
+        'cjson:metadata'?: cjson_metadata
         w3cCaps: {
             alwaysMatch?: {
                 foo?: true

tests/__mocks__/mocks.ts

@@ -60,7 +60,7 @@ export const FULL_RUNNER_STATS: RunnerStatsExtended = {
     cid: '0-0',
     capabilities:
         <DesiredCapabilitiesExtended>{
-            cjson_metadata: {} as cjson_metadata,
+            'cjson:metadata': {} as cjson_metadata,
             acceptInsecureCerts: false,
             acceptSslCerts: false,
             applicationCacheEnabled: false,
@@ -154,7 +154,7 @@ export const FULL_RUNNER_STATS: RunnerStatsExtended = {
         capabilities: {},
         jsonFolder: '',
         language: 'en',
-        cjson_metadata: {
+        'cjson:metadata': {
             app: {
                 name: 'test',
                 version: '1'
@@ -196,7 +196,7 @@ export const WDIO6_RUNNER_STATS: RunnerStatsExtended = {
     config: <TestrunnerExtended>{
         jsonFolder: '',
         language: 'en',
-        cjson_metadata: {} as cjson_metadata,
+        'cjson:metadata': {} as cjson_metadata,
         logFile: '',
         stdout: true,
         writeStream: {} as WriteStream,

tests/metadata.test.ts

@@ -259,7 +259,7 @@ describe('metadata', () => {
 
         it('should return app metadata based on the current.config.capabilities[\'cjson:metadata\'].app', () => {
             // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-            (FULL_RUNNER_STATS.capabilities as W3CCapabilitiesExtended).cjson_metadata!.app = {
+            (FULL_RUNNER_STATS.capabilities as W3CCapabilitiesExtended)['cjson:metadata']!.app = {
                 name: 'mock-appName',
                 version: 'mock-appVersion',
             }
@@ -274,7 +274,7 @@ describe('metadata', () => {
             expect(determinePlatformVersionSpy).toHaveBeenCalledTimes(1)
 
             // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-            delete (FULL_RUNNER_STATS.capabilities as W3CCapabilitiesExtended).cjson_metadata!.app
+            delete (FULL_RUNNER_STATS.capabilities as W3CCapabilitiesExtended)['cjson:metadata']!.app
             determineAppDataSpy.mockClear()
         })
 
@@ -300,7 +300,7 @@ describe('metadata', () => {
                     requestedCapabilities: {
                         w3cCaps: {
                             alwaysMatch: {
-                                cjson_metadata: {},
+                                'cjson:metadata': {},
                             },
                         },
                     },
@@ -323,7 +323,7 @@ describe('metadata', () => {
                     },
                     capabilities: {
                         browserName: 'chrome',
-                        cjson_metadata: {},
+                        'cjson:metadata': {},
                     },
                 } as WebdriverIOExtended,
             }
@@ -341,7 +341,7 @@ describe('metadata', () => {
                 options: {
                     capabilities: {
                         browserName: 'chrome',
-                        cjson_metadata: {},
+                        'cjson:metadata': {},
                     },
                 } as WebdriverIOExtended,
             }