Checkstyle issue fix

Author: TikhomirovSergey

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

@@ -17,6 +17,7 @@
 package io.appium.java_client;
 
 import com.google.common.collect.ImmutableMap;
+
 import org.openqa.selenium.By;
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.WebDriverException;

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

@@ -17,6 +17,7 @@
 package io.appium.java_client;
 
 import com.google.common.collect.ImmutableMap;
+
 import org.openqa.selenium.By;
 import org.openqa.selenium.WebDriverException;
 import org.openqa.selenium.WebElement;

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

@@ -22,7 +22,7 @@
 
 public interface ExecutesMethod {
     /**
-     * Executes JSONWP command and returns a response
+     * Executes JSONWP command and returns a response.
      *
      * @param driverCommand a JSONWP command
      * @param parameters map of command parameters
@@ -31,7 +31,7 @@ public interface ExecutesMethod {
     Response execute(String driverCommand, Map<String, ?> parameters);
 
     /**
-     * Executes JSONWP command and returns a response
+     * Executes JSONWP command and returns a response.
      *
      * @param driverCommand a JSONWP command
      * @return a result response

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

@@ -20,10 +20,10 @@
 import static io.appium.java_client.MobileCommand.INSTALL_APP;
 import static io.appium.java_client.MobileCommand.IS_APP_INSTALLED;
 import static io.appium.java_client.MobileCommand.LAUNCH_APP;
-import static io.appium.java_client.MobileCommand.prepareArguments;
-import static io.appium.java_client.MobileCommand.RESET;
 import static io.appium.java_client.MobileCommand.REMOVE_APP;
+import static io.appium.java_client.MobileCommand.RESET;
 import static io.appium.java_client.MobileCommand.RUN_APP_IN_BACKGROUND;
+import static io.appium.java_client.MobileCommand.prepareArguments;
 
 import com.google.common.collect.ImmutableMap;
 

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

@@ -20,6 +20,7 @@
 import static io.appium.java_client.MobileCommand.PULL_FOLDER;
 
 import com.google.common.collect.ImmutableMap;
+
 import org.openqa.selenium.remote.Response;
 
 import javax.xml.bind.DatatypeConverter;
@@ -49,7 +50,7 @@ default byte[] pullFile(String remotePath) {
      *                   if path starts with /AppName.app, which will be replaced with
      *                   the application's .app directory
      * @return A byte array of Base64 encoded data, representing a ZIP ARCHIVE
-     * of the contents of the requested folder.
+     *         of the contents of the requested folder.
      */
     default byte[] pullFolder(String remotePath) {
         Response response = execute(PULL_FOLDER, ImmutableMap.of("path", remotePath));

src/main/java/io/appium/java_client/ios/IOSDeviceActionShortcuts.java

@@ -16,12 +16,12 @@
 
 package io.appium.java_client.ios;
 
-import io.appium.java_client.CommandExecutionHelper;
-import io.appium.java_client.DeviceActionShortcuts;
-
 import static io.appium.java_client.ios.IOSMobileCommandHelper.hideKeyboardCommand;
 import static io.appium.java_client.ios.IOSMobileCommandHelper.shakeCommand;
 
+import io.appium.java_client.CommandExecutionHelper;
+import io.appium.java_client.DeviceActionShortcuts;
+
 public interface IOSDeviceActionShortcuts extends DeviceActionShortcuts {
 
     /**

src/test/java/io/appium/java_client/events/EmptyWebDriver.java

@@ -8,7 +8,19 @@
 import io.appium.java_client.FindsByIosUIAutomation;
 
 import org.apache.commons.lang3.StringUtils;
-import org.openqa.selenium.*;
+import org.openqa.selenium.Alert;
+import org.openqa.selenium.By;
+import org.openqa.selenium.Capabilities;
+import org.openqa.selenium.ContextAware;
+import org.openqa.selenium.Cookie;
+import org.openqa.selenium.HasCapabilities;
+import org.openqa.selenium.JavascriptExecutor;
+import org.openqa.selenium.NoSuchElementException;
+import org.openqa.selenium.Rotatable;
+import org.openqa.selenium.ScreenOrientation;
+import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.WebDriverException;
+import org.openqa.selenium.WebElement;
 import org.openqa.selenium.internal.FindsByClassName;
 import org.openqa.selenium.internal.FindsByCssSelector;
 import org.openqa.selenium.internal.FindsById;
@@ -65,12 +77,22 @@ private static List<StubWebElement> createStubList() {
         return null;
     }
 
+    @Override public StubWebElement findElement(By by) {
+        return new StubWebElement();
+    }
+
+    @Override
+    public StubWebElement findElement(String by, String using) throws WebDriverException, NoSuchElementException {
+        return new StubWebElement();
+    }
+
     @Override public List<StubWebElement> findElements(By by) {
         return createStubList();
     }
 
-    @Override public StubWebElement findElement(By by) {
-        return new StubWebElement();
+    @Override
+    public List<StubWebElement> findElements(String by, String using) throws WebDriverException {
+        return createStubList();
     }
 
     @Override public String getPageSource() {
@@ -200,16 +222,6 @@ private static List<StubWebElement> createStubList() {
         return new DesiredCapabilities(map);
     }
 
-    @Override
-    public StubWebElement findElement(String by, String using) throws WebDriverException, NoSuchElementException {
-        return new StubWebElement();
-    }
-
-    @Override
-    public List<StubWebElement> findElements(String by, String using) throws WebDriverException {
-        return createStubList();
-    }
-
     private class StubTargetLocator implements TargetLocator {
 
         private final WebDriver driver;

src/test/java/io/appium/java_client/events/FewInstancesTest.java

@@ -1,8 +1,8 @@
 package io.appium.java_client.events;
 
 import static org.hamcrest.core.Is.is;
-import static org.junit.Assert.assertThat;
 import static org.junit.Assert.assertNotEquals;
+import static org.junit.Assert.assertThat;
 
 import io.appium.java_client.events.listeners.AlertListener;
 import io.appium.java_client.events.listeners.ContextListener;

src/test/java/io/appium/java_client/events/StubWebElement.java

@@ -6,7 +6,14 @@
 import io.appium.java_client.FindsByAndroidUIAutomator;
 import io.appium.java_client.FindsByFluentSelector;
 import io.appium.java_client.FindsByIosUIAutomation;
-import org.openqa.selenium.*;
+import org.openqa.selenium.By;
+import org.openqa.selenium.Dimension;
+import org.openqa.selenium.NoSuchElementException;
+import org.openqa.selenium.OutputType;
+import org.openqa.selenium.Point;
+import org.openqa.selenium.Rectangle;
+import org.openqa.selenium.WebDriverException;
+import org.openqa.selenium.WebElement;
 import org.openqa.selenium.internal.FindsByClassName;
 import org.openqa.selenium.internal.FindsByCssSelector;
 import org.openqa.selenium.internal.FindsById;
@@ -148,10 +155,20 @@ private static List<WebElement> createStubSubElementList() {
         return createStubSubElementList();
     }
 
+    @Override
+    public List<WebElement> findElements(String by, String using) throws WebDriverException {
+        return createStubSubElementList();
+    }
+
     @Override public WebElement findElement(By by) {
         return new StubWebElement();
     }
 
+    @Override
+    public WebElement findElement(String by, String using) throws WebDriverException, NoSuchElementException {
+        return new StubWebElement();
+    }
+
     @Override public boolean isDisplayed() {
         return false;
     }
@@ -179,14 +196,4 @@ private static List<WebElement> createStubSubElementList() {
     @Override public String toString() {
         return this.getClass().getCanonicalName();
     }
-
-    @Override
-    public WebElement findElement(String by, String using) throws WebDriverException, NoSuchElementException {
-        return new StubWebElement();
-    }
-
-    @Override
-    public List<WebElement> findElements(String by, String using) throws WebDriverException {
-        return createStubSubElementList();
-    }
 }