deprecate UIAutomation for iOS (#1074)

* depreacte UIAutomation for iOS * fix review * added deprecation description
appium · Dec 24, 2018 · 0cb3fcb · 0cb3fcb
1 parent 1e80714
commit 0cb3fcb
Show file tree

Hide file tree

Showing 9 changed files with 38 additions and 3 deletions.
diff --git a/src/main/java/io/appium/java_client/FindsByIosUIAutomation.java b/src/main/java/io/appium/java_client/FindsByIosUIAutomation.java
@@ -22,6 +22,7 @@
 
 import java.util.List;
 
+@Deprecated
 public interface FindsByIosUIAutomation<T extends WebElement> extends FindsByFluentSelector<T> {
     /**
      * Method performs the searching for a single element by iOS UIAutomation selector

diff --git a/src/main/java/io/appium/java_client/MobileBy.java b/src/main/java/io/appium/java_client/MobileBy.java
@@ -69,6 +69,9 @@ protected String getLocatorString() {
      *
      * @param iOSAutomationText is iOS UIAutomation string
      * @return an instance of {@link io.appium.java_client.MobileBy.ByIosUIAutomation}
+     *
+     * @deprecated UIAutomation is going to get deprecated.
+     *        It is recommended to use XCUITest
      */
     public static By IosUIAutomation(final String iOSAutomationText) {
         return new ByIosUIAutomation(iOSAutomationText);

diff --git a/src/main/java/io/appium/java_client/pagefactory/HowToUseLocators.java b/src/main/java/io/appium/java_client/pagefactory/HowToUseLocators.java
@@ -55,7 +55,11 @@
      *
      * @return the strategy which defines how to use locators which are described by the
      * {@link iOSFindBy} annotation
+     *
+     * @deprecated UIAutomation is going to get deprecated.
+     *      It is recommended to use XCUITest
      */
+    @Deprecated
     LocatorGroupStrategy iOSAutomation() default LocatorGroupStrategy.CHAIN;
 
     /**

diff --git a/src/main/java/io/appium/java_client/pagefactory/iOSFindAll.java b/src/main/java/io/appium/java_client/pagefactory/iOSFindAll.java
@@ -29,8 +29,13 @@
  * of {@link io.appium.java_client.pagefactory.iOSBy} tags.
  * It will then search for all elements that match any criteria. Note that elements
  * are not guaranteed to be in document order.
+ *
+ * @deprecated UIAutomation is going to get deprecated.
+ *      Use {@link iOSXCUITFindAll} instead
+ *      It is recommended to use XCUITest
  */
-@Retention(RUNTIME) @Target({FIELD, TYPE})
+@Retention(RUNTIME)
+@Target({FIELD, TYPE})
 @Repeatable(iOSFindByAllSet.class)
 public @interface iOSFindAll {
     /**

diff --git a/src/main/java/io/appium/java_client/pagefactory/iOSFindBy.java b/src/main/java/io/appium/java_client/pagefactory/iOSFindBy.java
@@ -31,8 +31,13 @@
  * {@link org.openqa.selenium.support.PageFactory}
  * this allows users to quickly and easily create PageObjects.
  * using iOS UI selectors, accessibility, id, name, class name, tag and xpath
+ *
+ * @deprecated UIAutomation is going to get deprecated.
+ *      Use {@link iOSXCUITFindBy} instead
+ *      It is recommended to use XCUITest
  */
-@Retention(RUNTIME) @Target({FIELD, TYPE})
+@Retention(RUNTIME)
+@Target({FIELD, TYPE})
 @Repeatable(iOSFindBySet.class)
 public @interface iOSFindBy {
     /**

diff --git a/src/main/java/io/appium/java_client/pagefactory/iOSFindByAllSet.java b/src/main/java/io/appium/java_client/pagefactory/iOSFindByAllSet.java
@@ -10,6 +10,10 @@
 /**
  * Defines set of chained/possible locators. Each one locator
  * should be defined with {@link io.appium.java_client.pagefactory.iOSFindAll}
+ *
+ * @deprecated UIAutomation is going to get deprecated.
+ *       Use {@link iOSXCUITFindByAllSet} instead
+ *       It is recommended to use XCUITest
  */
 @Target(value = {TYPE, FIELD})
 @Retention(value = RUNTIME)

diff --git a/src/main/java/io/appium/java_client/pagefactory/iOSFindByChainSet.java b/src/main/java/io/appium/java_client/pagefactory/iOSFindByChainSet.java
@@ -10,6 +10,10 @@
 /**
  * Defines set of chained/possible locators. Each one locator
  * should be defined with {@link io.appium.java_client.pagefactory.iOSFindBys}
+ *
+ * @deprecated UIAutomation is going to get deprecated.
+ *      Use {@link iOSXCUITFindByChainSet} instead
+ *      It is recommended to use XCUITest
  */
 @Target(value = {TYPE, FIELD})
 @Retention(value = RUNTIME)

diff --git a/src/main/java/io/appium/java_client/pagefactory/iOSFindBySet.java b/src/main/java/io/appium/java_client/pagefactory/iOSFindBySet.java
@@ -26,6 +26,10 @@
 /**
  * Defines set of chained/possible locators. Each one locator
  * should be defined with {@link io.appium.java_client.pagefactory.iOSFindBy}
+ *
+ * @deprecated UIAutomation is going to get deprecated.
+ *        Use {@link iOSXCUITFindBySet} instead
+ *        It is recommended to use XCUITest
  */
 @Target(value = {TYPE, FIELD})
 @Retention(value = RUNTIME)

diff --git a/src/main/java/io/appium/java_client/pagefactory/iOSFindBys.java b/src/main/java/io/appium/java_client/pagefactory/iOSFindBys.java
@@ -27,8 +27,13 @@
 /**
  * Used to mark a field on a Page Object to indicate that lookup should use
  * a series of {@link io.appium.java_client.pagefactory.iOSBy} tags.
+ *
+ * @deprecated UIAutomation is going to get deprecated.
+ *      Use {@link iOSXCUITFindBys} instead
+ *      It is recommended to use XCUITest
  */
-@Retention(RUNTIME) @Target({FIELD, TYPE})
+@Retention(RUNTIME)
+@Target({FIELD, TYPE})
 @Repeatable(iOSFindByChainSet.class)
 public @interface iOSFindBys {
     /**