feat(config): default app permissions (iOS)

noomorph · noomorph · commit 8de310edbd57 · 2022-07-19T11:23:51.000+03:00
diff --git a/detox/index.d.ts b/detox/index.d.ts
@@ -178,6 +178,7 @@ declare global {
             bundleId?: string;
             build?: string;
             launchArgs?: Record<string, any>;
+            permissions?: DevicePermissions;
         }
 
         interface DetoxAndroidAppConfig {
@@ -195,6 +196,7 @@ declare global {
             build?: string;
             testBinaryPath?: string;
             launchArgs?: Record<string, any>;
+            permissions?: DevicePermissions;
         }
 
         type DetoxBuiltInDeviceConfig =
diff --git a/detox/src/configuration/composeAppsConfig.js b/detox/src/configuration/composeAppsConfig.js
@@ -68,6 +68,7 @@ function composeAppsConfigFromPlain(opts) {
         bundleId: localConfig.bundleId,
         build: localConfig.build,
         launchArgs: localConfig.launchArgs,
+        permissions: localConfig.permissions,
       };
       break;
     default:
diff --git a/detox/src/configuration/composeAppsConfig.test.js b/detox/src/configuration/composeAppsConfig.test.js
@@ -56,6 +56,9 @@ describe('composeAppsConfig', () => {
         binaryPath: 'path/to/app',
         bundleId: 'com.example.app',
         build: 'echo OK',
+        permissions: {
+          calendar: 'YES',
+        },
         launchArgs: {
           hello: 'world',
         }
@@ -65,6 +68,18 @@ describe('composeAppsConfig', () => {
     it.each([
       ['ios.none', 'ios.app'],
       ['ios.simulator', 'ios.app'],
+    ])('should infer type and app properties for %j', (deviceType, appType) => {
+      deviceConfig.type = deviceType;
+      expect(compose()).toEqual({
+        default: {
+          ...localConfig,
+          type: appType,
+          device: undefined,
+        },
+      });
+    });
+
+    it.each([
       ['android.attached', 'android.apk'],
       ['android.emulator', 'android.apk'],
       ['android.genycloud', 'android.apk'],
@@ -75,6 +90,7 @@ describe('composeAppsConfig', () => {
           ...localConfig,
           type: appType,
           device: undefined,
+          permissions: undefined,
         },
       });
     });
diff --git a/detox/src/devices/runtime/RuntimeDevice.js b/detox/src/devices/runtime/RuntimeDevice.js
@@ -1,3 +1,5 @@
+const _ = require('lodash');
+
 const DetoxRuntimeError = require('../../errors/DetoxRuntimeError');
 const debug = require('../../utils/debug'); // debug utils, leave here even if unused
 const { traceCall } = require('../../utils/trace');
@@ -55,6 +57,7 @@ class RuntimeDevice {
     this._emitter = eventEmitter;
     this._errorComposer = runtimeErrorComposer;
 
+    /** @type {Detox.DetoxAppConfig | null} */
     this._currentApp = null;
     this._currentAppLaunchArgs = new LaunchArgsEditor();
     this._processes = {};
@@ -332,8 +335,13 @@ class RuntimeDevice {
       this._createPayloadFileAndUpdatesParamsObject('userActivity', 'detoxUserActivityDataURL', params, baseLaunchArgs);
     }
 
-    if (params.permissions) {
-      await this.deviceDriver.setPermissions(bundleId, params.permissions);
+    const permissions = {
+      ...(this._currentApp ? this._currentApp['permissions'] : null),
+      ...params.permissions,
+    };
+
+    if (!_.isEmpty(permissions)) {
+      await this.deviceDriver.setPermissions(bundleId, permissions);
     }
 
     if (params.disableTouchIndicators) {
diff --git a/detox/src/devices/runtime/RuntimeDevice.test.js b/detox/src/devices/runtime/RuntimeDevice.test.js
@@ -430,10 +430,23 @@ describe('Device', () => {
     });
 
     it(`(relaunch) with permissions should send trigger setpermissions before app starts`, async () => {
-      const device = await aValidDevice();
+      const device = await aValidDevice({
+        appsConfig: {
+          default: {
+            permissions: {
+              calendar: 'NO',
+              camera: 'YES',
+            },
+          },
+        },
+      });
+
       await device.relaunchApp({ permissions: { calendar: 'YES' } });
 
-      expect(driverMock.driver.setPermissions).toHaveBeenCalledWith(bundleId, { calendar: 'YES' });
+      expect(driverMock.driver.setPermissions).toHaveBeenCalledWith(bundleId, {
+        calendar: 'YES',
+        camera: 'YES',
+      });
     });
 
     it('with languageAndLocale should launch app with a specific language/locale', async () => {
diff --git a/detox/test/e2e/detox.config.js b/detox/test/e2e/detox.config.js
@@ -16,6 +16,13 @@ const launchArgs = {
   micro: 'soft',
 };
 
+const defaultPermissions = {
+  calendar: 'YES',
+  notifications: 'YES',
+  camera: 'YES',
+  photos: 'YES',
+};
+
 /** @type {Detox.DetoxConfig} */
 const config = {
   testRunner: 'nyc jest',
@@ -56,13 +63,15 @@ const config = {
       binaryPath: 'ios/build/Build/Products/Debug-iphonesimulator/example.app',
       build: 'set -o pipefail && xcodebuild -workspace ios/example.xcworkspace -UseNewBuildSystem=YES -scheme example_ci -configuration Debug -sdk iphonesimulator -derivedDataPath ios/build -quiet',
       bundleId: 'com.wix.detox-example',
+      permissions: defaultPermissions,
     },
 
     'ios.release': {
       type: 'ios.app',
       name: 'example',
       binaryPath: 'ios/build/Build/Products/Release-iphonesimulator/example.app',
       build: 'set -o pipefail && export CODE_SIGNING_REQUIRED=NO && export RCT_NO_LAUNCH_PACKAGER=true && xcodebuild -workspace ios/example.xcworkspace -UseNewBuildSystem=YES -scheme example_ci -configuration Release -sdk iphonesimulator -derivedDataPath ios/build -quiet',
+      permissions: defaultPermissions,
     },
 
     'android.debug': {
diff --git a/docs/APIRef.Configuration.md b/docs/APIRef.Configuration.md
@@ -202,7 +202,11 @@ The format of Detox config allows you to define inside it multiple app configs i
     "ios.debug": {
       "type": "ios.app",
       "binaryPath": "ios/build/Build/Products/Debug-iphonesimulator/example.app",
-      "build": "xcodebuild -project ios/example.xcodeproj -scheme example -configuration Debug -sdk iphonesimulator -derivedDataPath ios/build"
+      "build": "xcodebuild -project ios/example.xcodeproj -scheme example -configuration Debug -sdk iphonesimulator -derivedDataPath ios/build",
+      "permissions": {
+        /** @see {Detox.DevicePermissions} */
+        notifications: 'YES',
+      },
     },
     "android.release": {
       "type": "android.apk",
@@ -218,14 +222,15 @@ The format of Detox config allows you to define inside it multiple app configs i
 
 An app config can have the following params:
 
-| Configuration Params | Details                                                                                                                                                                                                             |
-| -------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
-| `type`               | Mandatory property to discern app types: `ios.app`, `android.apk`.                                                                                                                                                  |
-| `name`               | Use only when working with multiple apps within the same configuration. See an example below.                                                                                                                       |
-| `binaryPath`         | Relative path to the ipa/app/apk due to be tested (make sure you build the app in a project relative path)                                                                                                          |
-| `build`              | **\[optional]** Build command (normally an `xcodebuild` command you use to build your app), which can be called later using Detox CLI tool as a convenience.                                                        |
-| `testBinaryPath`     | (optional, Android only): relative path to the test app (apk)                                                                                                                                                       |
-| `launchArgs`         | **\[optional]** An object specifying arguments (key-values pairs) to pass through into the app, upon launching on the device. For more info, refer to the dedicated [launch-arguments guide](APIRef.LaunchArgs.md). |
+| Configuration Params | Details                                                                                                                                                                                                               |
+|----------------------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
+| `type`               | Mandatory property to discern app types: `ios.app`, `android.apk`.                                                                                                                                                    |
+| `name`               | Use only when working with multiple apps within the same configuration. See an example below.                                                                                                                         |
+| `build`              | **\[optional]** Build command (normally an `xcodebuild` command you use to build your app), which can be called later using Detox CLI tool as a convenience.                                                          |
+| `binaryPath`         | Relative path to the ipa/app/apk due to be tested (make sure you build the app in a project relative path)                                                                                                            |
+| `testBinaryPath`     | **\[optional, Android only]**: relative path to the test app (apk)                                                                                                                                                    |
+| `launchArgs`         | **\[optional]** An object specifying arguments (key-values pairs) to pass through into the app, upon launching on the device. For more info, refer to the dedicated [launch-arguments guide](APIRef.LaunchArgs.md).   |
+| `permissions`        | **\[optional, iOS only]** An object specifying default [runtime permissions](APIRef.DeviceObjectAPI.md#permissions-ios-only) to give before launching the app.                                                        |
 
 To work with multiple apps within the same configuration you should be giving each app its name, e.g.:
 
diff --git a/docs/APIRef.DeviceObjectAPI.md b/docs/APIRef.DeviceObjectAPI.md
@@ -137,6 +137,8 @@ await device.launchApp({permissions: {calendar: 'YES'}});
 
 Detox uses [AppleSimUtils](https://github.com/wix/AppleSimulatorUtils) to implement this functionality for iOS simulators. Read about the different types of permissions and how to set them in AppleSimUtils' documentation and by checking out Detox’s [own test suite](https://github.com/wix/Detox/tree/a9a09246c05733f6b91cfcc0dba05a4714abca92/detox/test/e2e/13.permissions.test.js).
 
+As a reference, you can also use this [table of possible permissions](#permissions-ios-only).
+
 ##### 3. `url`—Launching with URL
 
 Launches the app with the specified URL to test your app’s deep link handling mechanism.
@@ -528,3 +530,32 @@ Exposes [`UiAutomator`’s `UiDevice` API](https://developer.android.com/referen
 **This is not a part of the official Detox API**, it may break and change whenever an update to `UiDevice` or `UiAutomator` Gradle dependencies (`androidx.test.uiautomator:uiautomator`) is introduced.
 
 [`UiDevice`’s autogenerated code](https://github.com/wix/Detox/tree/a9a09246c05733f6b91cfcc0dba05a4714abca92/detox/src/android/espressoapi/UIDevice.js)
+
+### Permissions (iOS only)
+
+When you are [configuring the apps](APIRef.Configuration.md#apps-configurations) list in your
+Detox config file or [launching the app](#2-permissionsset-runtime-permissions-ios-only),
+you can give certain permissions beforehand to it to avoid this inconvenient modal:
+
+![iOS permissions modal](img/ios-permissions.png)
+
+So, here is the exhaustive permissions list that Detox currently supports for iOS:
+
+| Permission    | Values                                                  |
+|---------------|---------------------------------------------------------|
+| calendar      | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| camera        | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| contacts      | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| faceid        | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| health        | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| homekit       | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| location      | `'always' &#124; 'inuse' &#124; 'never' &#124; 'unset'` |
+| medialibrary  | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| microphone    | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| motion        | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| notification  | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| photos        | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| reminders     | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| siri          | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| speech        | `'YES' &#124; 'NO' &#124; 'unset'`                      |
+| userTracking  | `'YES' &#124; 'NO' &#124; 'unset'`                      |
diff --git a/docs/img/ios-permissions.png b/docs/img/ios-permissions.png

Original file line number	Diff line number	Diff line change
`@@ -178,6 +178,7 @@ declare global {`
`178`	`178`	`bundleId?: string;`
`179`	`179`	`build?: string;`
`180`	`180`	`launchArgs?: Record<string, any>;`
	`181`	`+ permissions?: DevicePermissions;`
`181`	`182`	`}`
`182`	`183`
`183`	`184`	`interface DetoxAndroidAppConfig {`
`@@ -195,6 +196,7 @@ declare global {`
`195`	`196`	`build?: string;`
`196`	`197`	`testBinaryPath?: string;`
`197`	`198`	`launchArgs?: Record<string, any>;`
	`199`	`+ permissions?: DevicePermissions;`
`198`	`200`	`}`
`199`	`201`
`200`	`202`	`type DetoxBuiltInDeviceConfig =`