You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jspwiki.apache.org by ju...@apache.org on 2020/09/30 20:39:01 UTC

[jspwiki] 04/08: apply suggested fixes by intellij

This is an automated email from the ASF dual-hosted git repository.

juanpablo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jspwiki.git

commit 3067429be7de4edbb183405ca76e213466740fc4
Author: juanpablo <ju...@apache.org>
AuthorDate: Wed Sep 30 22:33:48 2020 +0200

    apply suggested fixes by intellij
---
 .../src/main/java/org/apache/wiki/pages/Page.java          | 14 +++++++-------
 .../src/main/java/org/apache/wiki/pages/PageBuilder.java   |  5 ++---
 .../main/java/org/apache/wiki/pages/haddock/LoginPage.java |  5 ++---
 3 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/Page.java b/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/Page.java
index f178b9d..f7b0d1d 100644
--- a/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/Page.java
+++ b/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/Page.java
@@ -18,19 +18,19 @@
  */
 package org.apache.wiki.pages;
 
-import java.io.File;
-import java.io.IOException;
-
-import org.apache.wiki.its.environment.Env;
-
 import com.codeborne.selenide.Selenide;
 import com.codeborne.selenide.WebDriverRunner;
+import org.apache.wiki.its.environment.Env;
+
+import java.io.File;
+import java.io.IOException;
+import java.net.URISyntaxException;
 
 
 /**
  * Common operations for Page Objects.
  * 
- * @see https://selenide.gitbooks.io/user-guide/content/en/pageobjects.html
+ * @see <a href="https://selenide.gitbooks.io/user-guide/content/en/pageobjects.html">Page Objects</a>
  */
 public interface Page {
 
@@ -53,7 +53,7 @@ public interface Page {
         return Env.TESTS_BASE_URL;
     }
     
-    static File download( String url ) throws IOException {
+    static File download( final String url ) throws IOException, URISyntaxException {
         return Selenide.download( url );
     }
 
diff --git a/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/PageBuilder.java b/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/PageBuilder.java
index dbf1e5e..27f6ee5 100644
--- a/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/PageBuilder.java
+++ b/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/PageBuilder.java
@@ -18,10 +18,9 @@
  */
 package org.apache.wiki.pages;
 
-import org.apache.wiki.its.environment.Env;
-
 import com.codeborne.selenide.Configuration;
 import com.codeborne.selenide.Selenide;
+import org.apache.wiki.its.environment.Env;
 
 
 /**
@@ -40,7 +39,7 @@ public class PageBuilder {
     /**
      * Sets up Selenide and opens the requested URL, allowing chaining additional operations over the given {@link Page} instance.
      *
-     * @param cls {@link Page} instance, in order to allow chaining operations.
+     * @param page {@link Page} instance, in order to allow chaining operations.
      * @return {@link Page} instance given on the {@link PageBuilder} constructor.
      */
     public < T extends Page > T openAs( final T page ) {
diff --git a/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/haddock/LoginPage.java b/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/haddock/LoginPage.java
index 4ff7655..c080dfd 100644
--- a/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/haddock/LoginPage.java
+++ b/jspwiki-it-tests/jspwiki-selenide-tests/src/main/java/org/apache/wiki/pages/haddock/LoginPage.java
@@ -18,18 +18,17 @@
  */
 package org.apache.wiki.pages.haddock;
 
+import com.codeborne.selenide.Selenide;
 import org.apache.wiki.its.environment.Env;
 import org.openqa.selenium.By;
 
-import com.codeborne.selenide.Selenide;
-
 public class LoginPage implements HaddockPage {
     
     public ReadWikiPage performLogin() {
         return performLogin( Env.LOGIN_JANNE_USERNAME, Env.LOGIN_JANNE_PASSWORD );
     }
     
-    public ReadWikiPage performLogin( String login, String password ) {
+    public ReadWikiPage performLogin( final String login, final String password ) {
         Selenide.$( By.id( "j_username" ) ).val( login );
         Selenide.$( By.id( "j_password" ) ).val( password );
         Selenide.$( By.name( "submitlogin" ) ).click();