fix(ios): ios13 statusbar height

Author: SvetoslavTsenov

lib/appium-driver.d.ts

@@ -271,7 +271,6 @@ export declare class AppiumDriver {
     getScreenViewPort(): IRectangle;
     /**
     * Android ONLY! Input key event via ADB.
-    * Must be combined with '--relaxed-security' appium flag. When not running in sauceLabs '--ignoreDeviceController' should be added too.
     * @param keyEvent The event number
     */
     adbKeyEvent(keyEvent: number | AndroidKeyEvent): Promise<void>;

lib/appium-driver.ts

@@ -11,7 +11,8 @@ import {
     DeviceController,
     IDevice,
     DeviceType,
-    AndroidController
+    AndroidController,
+    IOSController
 } from "mobile-devices-controller";
 import {
     addExt,
@@ -47,6 +48,7 @@ import { DeviceOrientation } from "./enums/device-orientation";
 import { NsCapabilities } from "./ns-capabilities";
 import { AutomationName } from "./automation-name";
 import { AndroidKeyEvent } from "mobile-devices-controller";
+import { setInterval } from "timers";
 
 export class AppiumDriver {
     private _defaultWaitTime: number = 5000;
@@ -215,10 +217,11 @@ export class AppiumDriver {
 
         let hasStarted = false;
         let retries = 10;
+        let sessionInfoDetails;
+
         while (retries > 0 && !hasStarted) {
             try {
                 let sessionInfo;
-                let sessionInfoDetails;
                 try {
                     if (args.sessionId || args.attachToDebug) {
                         const sessionInfos = JSON.parse(((await getSessions(args.port)) || "{}") + '');
@@ -311,7 +314,19 @@ export class AppiumDriver {
             await driver.updateSettings(appiumCapsFromConfig.settings);
         }
 
-        return new AppiumDriver(driver, wd, webio, args.driverConfig, args);
+        if (+sessionInfoDetails.statBarHeight === 0
+            && sessionInfoDetails.platformName.toLowerCase() === "ios"
+            && sessionInfoDetails.platformVersion.startsWith("13")) {
+            try {
+                const devicesInfos = IOSController.devicesDisplaysInfos();
+                const matches = devicesInfos.filter(d => sessionInfoDetails.deviceName.includes(d.deviceType));
+                const deviceType = matches[matches.length - 1];
+                args.device.viewportRect.y += deviceType.statBarHeight * deviceType.density;
+            } catch (error) { }
+        }
+
+        const appiumDriver = new AppiumDriver(driver, wd, webio, args.driverConfig, args);
+        return appiumDriver;
     }
 
     public async updateSettings(settings: any) {
@@ -1011,23 +1026,22 @@ export class AppiumDriver {
     public getScreenViewPort(): IRectangle {
         const rect = this.getScreenActualViewPort();
         if (rect
+            && this.isIOS
             && Object.getOwnPropertyNames(rect).length > 0
-            && this._args.appiumCaps.device.deviceScreenDensity) {
+            && this._args.device.deviceScreenDensity) {
             return <IRectangle>{
-                x: rect.x / this._args.appiumCaps.device.deviceScreenDensity,
-                y: rect.y / this._args.appiumCaps.device.deviceScreenDensity,
-                width: rect.width / this._args.appiumCaps.device.deviceScreenDensity,
-                height: rect.height / this._args.appiumCaps.device.deviceScreenDensity,
+                x: rect.x / this._args.device.deviceScreenDensity,
+                y: rect.y / this._args.device.deviceScreenDensity,
+                width: rect.width / this._args.device.deviceScreenDensity,
+                height: rect.height / this._args.device.deviceScreenDensity,
             }
         } else {
-            logError("Device's density is undefined!");
             return rect;
         }
     }
 
     /**
     * Android ONLY! Input key event via ADB.
-    * Must be combined with '--relaxed-security' appium flag. When not running in sauceLabs '--ignoreDeviceController' should be added too.
     * @param keyEvent The event number
     */
     public async adbKeyEvent(keyEvent: number | AndroidKeyEvent) {

lib/device-manager.d.ts

@@ -13,7 +13,7 @@ export declare class DeviceManager implements IDeviceManager {
     static getInstalledApps(device: IDevice): Promise<string[]>;
     static getDefaultDevice(args: INsCapabilities, deviceName?: string, token?: string, type?: DeviceType, platformVersion?: number): IDevice;
     private static convertViewportRectToIRectangle;
-    static applyAppiumSessionInfoDetails(args: INsCapabilities, sessionInfoDetails: any): IDevice;
+    static applyAppiumSessionInfoDetails(args: INsCapabilities, sessionInfoDetails: any): any;
     static setDontKeepActivities(args: INsCapabilities, driver: any, value: any): Promise<void>;
     static executeShellCommand(driver: any, commandArgs: {
         command: string;

lib/device-manager.ts

@@ -261,7 +261,7 @@ export class DeviceManager implements IDeviceManager {
                 if (args.device.name.includes(k)) {
                     args.device.config = {
                         density: args.device.config['density'] || v.density,
-                        offsetPixels: v.actionBarHeight
+                        offsetPixels: v.actionBarHeight * (args.device.config['density'] || v.density)
                     };
                 }
             });
@@ -289,7 +289,7 @@ export class DeviceManager implements IDeviceManager {
                 args.device.config['offsetPixels'] = AndroidController.calculateScreenOffset(args.device.config.density);
             }
 
-            if (!density) {
+            if (!density && !args.isIOS) {
                 await DeviceManager.getDensity(args, driver);
                 density = args.device.config.density
                 args.device.config['offsetPixels'] = AndroidController.calculateScreenOffset(args.device.config.density);

lib/parser.d.ts

@@ -1,2 +1,2 @@
 import { LogImageType } from "./enums/log-image-type";
-export declare const projectDir: string, projectBinary: string, pluginRoot: string, pluginBinary: string, port: number, verbose: boolean, appiumCapsLocation: string, testFolder: string, runType: string, isSauceLab: boolean, appPath: string, storage: string, testReports: string, devMode: boolean, ignoreDeviceController: boolean, wdaLocalPort: number, path: string, relaxedSecurity: boolean, cleanApp: boolean, attachToDebug: boolean, sessionId: string, startSession: boolean, capabilitiesName: string, imagesPath: string, startDeviceOptions: string, deviceTypeOrPlatform: string, device: import("mobile-devices-controller/lib/device").IDevice, driverConfig: any, logImageTypes: LogImageType[], appiumCaps: any;
+export declare const projectDir: string, projectBinary: string, pluginRoot: string, pluginBinary: string, port: number, verbose: boolean, appiumCapsLocation: string, testFolder: string, runType: string, isSauceLab: boolean, appPath: string, storage: string, testReports: string, devMode: boolean, ignoreDeviceController: boolean, wdaLocalPort: number, path: string, relaxedSecurity: boolean, cleanApp: boolean, attachToDebug: boolean, sessionId: string, startSession: boolean, capabilitiesName: string, imagesPath: string, startDeviceOptions: string, deviceTypeOrPlatform: string, device: any, driverConfig: any, logImageTypes: LogImageType[], appiumCaps: any;

package.json

@@ -36,7 +36,7 @@
     "frame-comparer": "^2.0.1",
     "glob": "^7.1.0",
     "inquirer": "^6.2.0",
-    "mobile-devices-controller":  "^5.0.0",
+    "mobile-devices-controller": "^5.1.0",
     "wd": "~1.11.3",
     "webdriverio": "~4.14.0",
     "yargs": "~12.0.5"