feat: Make settings api to accept string names (#1240)

Author: takeyaqa

src/main/java/io/appium/java_client/HasSettings.java

@@ -37,6 +37,18 @@ public interface HasSettings extends ExecutesMethod {
      * @param value   value of the setting.
      */
     default void setSetting(Setting setting, Object value) {
+        CommandExecutionHelper.execute(this, setSettingsCommand(setting.toString(), value));
+    }
+
+    /**
+     * Set a setting for this test session It's probably better to use a
+     * convenience function, rather than use this function directly. Try finding
+     * the method for the specific setting you want to change.
+     *
+     * @param setting Setting you wish to set.
+     * @param value   value of the setting.
+     */
+    default void setSetting(String setting, Object value) {
         CommandExecutionHelper.execute(this, setSettingsCommand(setting, value));
     }
 

src/main/java/io/appium/java_client/MobileCommand.java

@@ -469,9 +469,9 @@ public static ImmutableMap<String, Object> prepareArguments(String[] params,
         return new AbstractMap.SimpleEntry<>(GET_SETTINGS, ImmutableMap.of());
     }
 
-    public static Map.Entry<String, Map<String, ?>> setSettingsCommand(Setting setting, Object value) {
+    public static Map.Entry<String, Map<String, ?>> setSettingsCommand(String setting, Object value) {
         return new AbstractMap.SimpleEntry<>(SET_SETTINGS, prepareArguments("settings",
-                prepareArguments(setting.toString(), value)));
+                prepareArguments(setting, value)));
     }
 
     /**

src/test/java/io/appium/java_client/android/SettingTest.java

@@ -92,6 +92,17 @@ public class SettingTest extends BaseAndroidTest {
             .get(Setting.TRACK_SCROLL_EVENTS.toString()));
     }
 
+    @Test public void testSettingByString() {
+        assertEquals(true, driver.getSettings()
+                .get("shouldUseCompactResponses"));
+        driver.setSetting("shouldUseCompactResponses", false);
+        assertEquals(false, driver.getSettings()
+                .get("shouldUseCompactResponses"));
+        driver.setSetting("shouldUseCompactResponses", true);
+        assertEquals(true, driver.getSettings()
+                .get("shouldUseCompactResponses"));
+    }
+
     private void assertJSONElementContains(Setting setting, long value) {
         assertEquals(driver.getSettings().get(setting.toString()), value);
     }

src/test/java/io/appium/java_client/ios/SettingTest.java

@@ -83,5 +83,16 @@ public class SettingTest extends AppIOSTest {
             .get(Setting.KEYBOARD_PREDICTION.toString()));
     }
 
+    @Test public void testSettingByString() {
+        assertEquals(true, driver.getSettings()
+                .get("shouldUseCompactResponses"));
+        driver.setSetting("shouldUseCompactResponses", false);
+        assertEquals(false, driver.getSettings()
+                .get("shouldUseCompactResponses"));
+        driver.setSetting("shouldUseCompactResponses", true);
+        assertEquals(true, driver.getSettings()
+                .get("shouldUseCompactResponses"));
+    }
+
 
 }