package me.alb_i986.selenium.tinafw.pages;

import java.util.List;

import me.alb_i986.selenium.tinafw.utils.PropertyLoader;

import org.apache.log4j.Logger;
import org.openqa.selenium.*;
import org.openqa.selenium.support.PageFactory;
import org.openqa.selenium.support.ui.ExpectedConditions;

/**
 * Abstract class representing a generic page.
 * A Page has a {@link WebDriver}. 
 * The constant {@link #BASE_URL} stores the base URL of the SUT,
 * and it is loaded from the config file.
 * 
 * <p>
 * Any subclass should override {@link #waitUntilIsLoaded()}
 * to implement a more meaningful wait, specific to that page.
 *
 */
public abstract class Page {

	/**
	 * Configurable via the property "tinafw.base_url".
	 */
	public final static String BASE_URL = PropertyLoader.getTinaFwConfig("base_url");
	
	protected static final Logger logger = Logger.getLogger(Page.class);

	protected WebDriver driver;
	protected Page previousPage;
	
	/**
	 * Calls {@link PageFactory#initElements(WebDriver, Object)}
	 * and {@link #waitUntilIsLoaded()}.
	 * 
	 * @param driver
	 * @param previous
	 */
	public Page(WebDriver driver, Page previous) {
		this.driver = driver;
		this.previousPage = previous;
		PageFactory.initElements(driver, this);
		waitUntilIsLoaded();
	}
	

	/**
	 * @see WebDriver.Navigation#back()
	 */
	public Page gotoBack() {
		if(previousPage == null)
			throw new IllegalStateException("can't go back: no previous page exists");
		driver.navigate().back();
		return previousPage;
	}
	
	/**
	 * @see WebDriver#getTitle()
	 */
	public String getTitle() {
		return driver.getTitle();
	}

	/**
	 * @see WebDriver#getPageSource()
	 */
	public String getPageSource() {
		return driver.getPageSource();
	}

	/**
	 * @see WebDriver#getCurrentUrl()
	 */
	public String getCurrentUrl() {
		return driver.getCurrentUrl();
	}

	/**
	 * @param locator
	 * @return {@link WebElement}
	 * 
	 * @see WebDriver#findElement(By)
	 */
	protected WebElement getElement(By locator) {
		return driver.findElement(locator);
	}

	/**
	 * @param locator
	 * @return list of {@link WebElement}
	 * 
	 * @see WebDriver#findElements(By)
	 */
	protected List<WebElement> getElements(By locator) {
		return driver.findElements(locator);
	}

	/**
	 * Dumb wait until the HTML body element is visible.
	 * Actually this method should be abstract,
	 * but it isn't just for convenience.
	 * It is highly recommended to override it.
	 */
	protected void waitUntilIsLoaded() {
		PageHelper.waitUntil(
			ExpectedConditions.presenceOfElementLocated(By.cssSelector("body")),
			driver
		);
	}
	
}

@@ -1234,16 +1234,77 @@
  this);%0A
+%09%09logger.info(%22Loading page %22 + getClass().getSimpleName());%0A
 %09%09waitUn

improve logging: "loading page $PAGE_CLASS" every time a `new Page` is called