fix: guard missing getContext (#15103)

mccmrunal · wswebcreation · web-flow · commit f6ca37e8c10b · 2026-02-24T07:25:01.000+01:00
* fix: guard missing getContext * Revert "fix: guard missing getContext" This reverts commit 3f6de43. * fix: skip session manager for protocol stub --------- Co-authored-by: Wim Selles <wim@wswebcreation.nl>
diff --git a/packages/webdriverio/src/index.ts b/packages/webdriverio/src/index.ts
@@ -75,7 +75,14 @@ export const remote = async function (
     }
 
     instance.addLocatorStrategy = addLocatorStrategyHandler(instance)
-    await registerSessionManager(instance)
+
+    /**
+     * protocol stub sessions are used before the real session starts (e.g. spec filtering).
+     * They don't have full command coverage, so skip session manager initialization.
+     */
+    if (!isStub(options.automationProtocol)) {
+        await registerSessionManager(instance)
+    }
     return instance
 }
 
@@ -209,4 +216,3 @@ export const multiremote = async function (
     driver.addLocatorStrategy = addLocatorStrategyHandler(driver)
     return driver
 }
-
diff --git a/packages/webdriverio/tests/module.test.ts b/packages/webdriverio/tests/module.test.ts
@@ -6,8 +6,10 @@ import { validateConfig } from '@wdio/config'
 
 import detectBackend from '../src/utils/detectBackend.js'
 import { remote, multiremote, attach, Key, SevereServiceError } from '../src/index.js'
+import { registerSessionManager } from '../src/session/index.js'
 
 vi.mock('../src/utils/detectBackend', () => ({ default: vi.fn() }))
+vi.mock('../src/session/index.js', () => ({ registerSessionManager: vi.fn() }))
 vi.mock('@wdio/logger', () => import(path.join(process.cwd(), '__mocks__', '@wdio/logger')))
 vi.mock('webdriver', () => {
     const client = {
@@ -68,6 +70,7 @@ describe('WebdriverIO module interface', () => {
         vi.mocked(WebDriver.newSession).mockClear()
         vi.mocked(WebDriver.attachToSession).mockClear()
         vi.mocked(detectBackend).mockClear()
+        vi.mocked(registerSessionManager).mockClear()
     })
 
     it('should provide all exports', () => {
@@ -87,6 +90,7 @@ describe('WebdriverIO module interface', () => {
             const browser = await remote(options)
             expect(browser.sessionId).toBe('foobar-123')
             expect(logger.setLogLevelsConfig).toBeCalledWith(undefined, 'trace')
+            expect(registerSessionManager).toBeCalledTimes(1)
         })
 
         it('allows to propagate a modifier', async () => {
@@ -170,6 +174,21 @@ describe('WebdriverIO module interface', () => {
                 isWindowsApp: false,
                 isMacApp: false,
             })
+            expect(registerSessionManager).not.toBeCalled()
+        })
+
+        it('should not initialize session managers for protocol stub sessions', async () => {
+            const browser = await remote({
+                automationProtocol: './protocol-stub.js',
+                capabilities: {
+                    browserName: 'Safari',
+                    platformName: 'iOS',
+                    'appium:options': { automationName: 'XCUITest' }
+                }
+            })
+
+            expect(browser.isMobile).toBe(true)
+            expect(registerSessionManager).not.toBeCalled()
         })
 
         it('should use the element disable implicitWait exclusion list', async () => {

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,14 @@ export const remote = async function (`
`75`	`75`	`}`
`76`	`76`
`77`	`77`	`instance.addLocatorStrategy = addLocatorStrategyHandler(instance)`
`78`		`- await registerSessionManager(instance)`
	`78`	`+`
	`79`	`+ /**`
	`80`	`+ * protocol stub sessions are used before the real session starts (e.g. spec filtering).`
	`81`	`+ * They don't have full command coverage, so skip session manager initialization.`
	`82`	`+ */`
	`83`	`+ if (!isStub(options.automationProtocol)) {`
	`84`	`+ await registerSessionManager(instance)`
	`85`	`+ }`
`79`	`86`	`return instance`
`80`	`87`	`}`
`81`	`88`
`@@ -209,4 +216,3 @@ export const multiremote = async function (`
`209`	`216`	`driver.addLocatorStrategy = addLocatorStrategyHandler(driver)`
`210`	`217`	`return driver`
`211`	`218`	`}`
`212`		`-`