suumo-search

commit ead3c4304e8b9f7efb94884560e6e0a4c79bb203
parent f4511f9a3a714cf782105bf0ccef334d49d6325d
Author: Kebigon <git@kebigon.xyz>
Date:   Mon, 16 Mar 2020 20:06:54 +0900

Update xpath for Yahoo transit buttons to work with HtmlUnit
Diffstat:
M
src/main/java/xyz/kebigon/housesearch/browser/Browser.java
 | 
3
++-
M
src/main/java/xyz/kebigon/housesearch/browser/yahoo/transit/YahooTransitBrowser.java
 | 
4
++--

2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/main/java/xyz/kebigon/housesearch/browser/Browser.java b/src/main/java/xyz/kebigon/housesearch/browser/Browser.java
@@ -38,7 +38,7 @@ public abstract class Browser implements Closeable
     {
         final String url = driver.getCurrentUrl();
         log.info("Restarting browser, navigate to {}", url);
-        
+
         driver.quit();
         driver = new HtmlUnitDriver();
         driver.navigate().to(url);
@@ -60,6 +60,7 @@ public abstract class Browser implements Closeable
         }
         catch (final NoSuchElementException e)
         {
+            log.warn("Unable to click on '{}'", xpathExpression);
             return false;
         }
     }
diff --git a/src/main/java/xyz/kebigon/housesearch/browser/yahoo/transit/YahooTransitBrowser.java b/src/main/java/xyz/kebigon/housesearch/browser/yahoo/transit/YahooTransitBrowser.java
@@ -51,12 +51,12 @@ public class YahooTransitBrowser extends Browser
             for (final WebElement element : findElements("//ul[@class='routeList']/li/dl/dd/ul"))
                 routes.add(createRoute(from, to, element));
 
-            click("//ul[@id='tabflt']/li/a[@data-rapid_p='2']");
+            click("//ul[@id='tabflt']/li[2]/a");
 
             for (final WebElement element : findElements("//ul[@class='routeList']/li/dl/dd/ul"))
                 routes.add(createRoute(from, to, element));
 
-            click("//ul[@id='tabflt']/li/a[@data-rapid_p='3']");
+            click("//ul[@id='tabflt']/li[3]/a");
 
             for (final WebElement element : findElements("//ul[@class='routeList']/li/dl/dd/ul"))
                 routes.add(createRoute(from, to, element));