add error case

appium · Nov 25, 2023 · 8012406 · 8012406
1 parent c1fef21
commit 8012406
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/extensions/permissions.js b/lib/extensions/permissions.js
@@ -127,7 +127,7 @@ async function setAccess (simctl, udid, bundleId, permissionsMapping) {
           resetPermissions.push(serviceName);
           break;
         default:
-          log.warn(`${serviceName} does not support ${permissionsMapping[serviceName]}. Please specify yes, no or unset.`);
+          log.errorAndThrow(`${serviceName} does not support ${permissionsMapping[serviceName]}. Please specify 'yes', 'no' or 'unset'.`);
       };
     }
   }

diff --git a/test/functional/simulator-e2e-specs.js b/test/functional/simulator-e2e-specs.js
@@ -356,6 +356,8 @@ describe('advanced features', function () {
       expect(await sim.setPermission('com.apple.Maps', 'location', 'yes')).should.not.be.rejected;
       expect(await sim.setPermission('com.apple.Maps', 'location', 'no')).should.not.be.rejected;
       expect(await sim.setPermission('com.apple.Maps', 'location', 'unset')).should.not.be.rejected;
+
+      expect(await sim.setPermission('com.apple.Maps', 'location', 'unsupported')).should.be.rejected;
     });
 
     it('should set and get with wix command', async function () {