Honouring the skipped status (webdriverio#15232)

* honouring the skipped status

* review changes pt.1

Author: Bhargavi-BS

packages/wdio-browserstack-service/src/cli/frameworks/wdioMochaTestFramework.ts

@@ -168,13 +168,17 @@ export default class WdioMochaTestFramework extends TestFramework {
 
     loadTestResult(instance: TestFrameworkInstance, args: Record<string, unknown>) {
         const results = args.result as Frameworks.TestResult
-        const { error, passed } = results
+        const { error, passed, skipped } = results
         let result = 'passed'
         let failure: Array<unknown>|null = null
         let failureReason: string|null = null
         let failureType: string|null = null
         if (!passed) {
-            result = (error && error.message && error.message.includes('sync skip; aborting execution')) ? 'ignore' : 'failed'
+            if (skipped) {
+                result = 'skipped'
+            } else {
+                result = (error && error.message && error.message.includes('sync skip; aborting execution')) ? 'ignore' : 'failed'
+            }
             if (error && result !== 'skipped') {
                 failure = [{ backtrace: [removeAnsiColors(error.message), removeAnsiColors(error.stack || '')] }] // add all errors here
                 failureReason = removeAnsiColors(error.message)

packages/wdio-browserstack-service/src/cli/modules/automateModule.ts

@@ -100,14 +100,14 @@ export default class AutomateModule extends BaseModule {
     async onAfterTest(args: Record<string, unknown>) {
         this.logger.debug('onAfterTest: inside automate module after test hook!')
         const instace = args.instance as TestFrameworkInstance
-        const { error, passed } = args.result as { error: Error | null, passed: boolean }
+        const { error, passed, skipped } = args.result as { error: Error | null, passed: boolean, skipped?: boolean }
         const _failReasons: string[] = []
 
-        if (!passed) {
+        if (!passed && !skipped) {
             _failReasons.push((error && error.message) || 'Unknown Error')
         }
 
-        const status = passed ? 'passed' : 'failed'
+        const status = passed || skipped ? 'passed' : 'failed'
         const reason = _failReasons.length > 0 ? _failReasons.join('\n') : undefined
 
         const autoInstance = AutomationFramework.getTrackedInstance()

packages/wdio-browserstack-service/src/service.ts

@@ -402,8 +402,8 @@ export default class BrowserstackService implements Services.ServiceInstance {
     @PerformanceTester.Measure(PERFORMANCE_SDK_EVENTS.EVENTS.SDK_HOOK, { hookType: 'afterTest' })
     async afterTest(test: Frameworks.Test, context: never, results: Frameworks.TestResult) {
         this._specsRan = true
-        const { error, passed } = results
-        if (!passed) {
+        const { error, passed, skipped } = results
+        if (!passed && !skipped) {
             const testError = (error && error.message) || 'Unknown Error'
             this._failReasons.push(testError)
 

packages/wdio-types/src/Frameworks.ts

@@ -41,6 +41,7 @@ export interface TestResult {
     retries: TestRetries
     exception: string
     status: string
+    skipped?: boolean
 }
 
 export interface Results {