Code verbesserung

[Mark110]

Guten Morgen,

auch heute würde mich Eure Meinung zu meinem Code interessieren.
Er funktioniert so wie er ist. Die Frage die ich mir Stelle ist, ob man Dinge kürzer schreiben kann (und dann auch sollte.. ohne verlust der Übersicht) und ob der Ansatz mit 3 ähnlichen Methoden so vernünftig ist.
Es handelt sich um Testautomation.

Bin gespant auf Eure Beitrage

import org.apache.log4j.Logger;


import tchelperclass.util.Log;

import com.rational.test.ft.ObjectNotFoundException;
import com.rational.test.ft.RationalTestException;
import com.rational.test.ft.object.interfaces.TestObject;
import com.rational.test.ft.script.RationalTestScript;
import com.rational.test.ft.value.RegularExpression;

public abstract class FindTestObject extends RationalTestScript {

	
	// **********************************************
	// The class defines the methodes to find objects 
	// **********************************************
	
	protected static final Logger log = Logger.getLogger(FindTestObject.class);
	protected static final double timeOut = 15;
	protected static final double timeBetweenChecks = 1;
	protected TestObject rootTestObject = null;
	protected TestObject[] to = null;
	protected long start = System.currentTimeMillis();
	protected long end;
	private int timer = 0;
			
	public abstract TestObject getRootObject();

	
	// ************************************** 
	// Methods that try to get the Testobject
	// **************************************
		

	public TestObject getTestObject(String property, String value, String property2, String value2) {
		try {
			rootTestObject = getRootObject();
			while(timer<timeOut) {
				// Try to find TestObject until timout
				to = rootTestObject.find(atDescendant(property, value, property2,value2));
				if (to.length > 2) {
					log.error("More than one Object found on website! Methode getTestObject(" + property + "," + value + "," + property2 + "," + value2 + ")");
					throw new ObjectNotFoundException("TestObject not found on Website");
				}
				if (to.length != 0) {
					end = System.currentTimeMillis();
					log.debug("Testobject found within: " + getElapsedTime(start, end) + " seconds");
					start = System.currentTimeMillis();
					// Reset timer
					timer = 0;
					return ((TestObject) (to[0]));
				}
				unregister(to);
				sleep(timeBetweenChecks);
				timer++;
				System.out.println(timer);
			}
			log.error("TestObject not found in Website. Methode getTestObject(" + property + "," + value + "," + property2 + "," + value2 + ")");
			throw new ObjectNotFoundException("TestObject not found in Website");
		} catch (RationalTestException rte) {
			log.error("Rational Exception: Methode getTestObject(" + property + "," + value + "," + property2 + "," + value2 + ")");
			throw new ObjectNotFoundException("TestObject not found in Website");
		}
	}

	
	public TestObject getTestObject(String property, String value, String property2, RegularExpression reg) {
		try {
			rootTestObject = getRootObject();
			while (timer < timeOut) {
				// Try to find TestObject until timout
				to = rootTestObject.find(atDescendant(property, value, property2, reg));
				if (to.length > 2) {
					log.error("More than one Object found on website. Methode getTestObject(" + property + "," + value + "," + property2 + "," + reg + ")");
					throw new ObjectNotFoundException("TestObject not found on Website");
				}
				if (to.length != 0) {
					end = System.currentTimeMillis();
					log.debug("Testobject found within: " + getElapsedTime(start, end) + " seconds");
					start = System.currentTimeMillis();
					// Reset timer
					timer = 0;
					return ((TestObject) (to[0]));
				}
				unregister(to);
				sleep(timeBetweenChecks);
				timer++;
			}
			log.error("TestObject not found in Website.Methode getTestObject(" + property + "," + value + "," + property2 + "," + reg + ")");
			throw new ObjectNotFoundException("TestObject not found in Website");
		} catch (RationalTestException rte) {
			log.error("Rational Exception: Methode getTestObject(" + property + "," + value + "," + property2 + "," + reg + ")");
			throw new ObjectNotFoundException("TestObject not found in Website");
		}
	}
	
	public TestObject getTestObject(TestObject testObject, String property,	String value) {
		TestObject childNode = null;
		
		try {
			while (timer < timeOut) {
				// Try to find TestObject until timeout
				childNode = testObject;
				to = childNode.find(atDescendant(property, value));

				if (to.length > 2) {
					log.error("More than one Object found on website. Methode getTestObject(" + testObject + "," + property + "," + value + ")");
					throw new ObjectNotFoundException(
							"TestObject not found on Website");
				}
				if (to.length != 0) {
					end = System.currentTimeMillis();
					log.debug("Testobject found within: " + getElapsedTime(start, end) + " seconds");
					start = System.currentTimeMillis();
					// Reset timer
					timer = 0;
					return ((TestObject) (to[0]));
				}
				unregister(to);
				sleep(timeBetweenChecks);
				timer++;
			}
			log.error("TestObject not found in Website. Methode getTestObject(" + testObject + "," + property + "," + value + ")");
			throw new ObjectNotFoundException("TestObject not found in Website");
		} catch (RationalTestException rte) {
			log.error("Rational Exception: Methode getTestObject(" + testObject + "," + property + "," + value + ")");
			throw new ObjectNotFoundException(
					"getTestObject() - Object not found");
		}
	}
	
	
	// Calculate runtime
	
	public String getElapsedTime(long start, long end) {
		long calcElpasedTime = (end/1000)-(start/1000);
		//long calcElpasedTimeMS = (end)-(start);
		String elapsedSec = Long.toString(calcElpasedTime);
		//String elapsedMS = Long.toString(calcElpasedTimeMS);
		return elapsedSec;
	}
}

Die Methoden erhalten von einer Webseite Attribute die Sie finden sollen.
Die abtrakte Methode getRootTestObject kümmert sich dabei um die höchste Ebene eines Dialoges oder eines HTML Documents.

 

[maki]

Vielleciht hilft dir ja das hier: http://www.youtube.com/watch?v=Pq6LHFM4JvE

Die Stunde lohnt sich!

 

[Mark110]

oh klasse link! Den Film werde ich mir am Wochenende anschauen.
Gibt es so grundsätzlich nichts zu meinem Quelltext zu sagen?

 

[Mark110]

Hmmm keiner eine Mainung dazu? Wäre auch zufrieden, wenn jemand sagen würde: "Ja das ist so ok"

 

[maki]

IMHO: Tests sollten selbsterklärend sein, dass ist dieser sicherlich nicht, Was soll denn dann aus den Unterklassen werden? Noch unübersichtlicher?
Exceptionhandling in Tests finde ich voll daneben, ein test geht durch, oder eben nicht, bietet Rational denn keine Asserts?
Was ist mit der Möglichkeit die Dauer von tests zu ermitteln? Musst du so etwas wirklich selbst machen?

IMHO sollten testklassen so einfach wie möglich gehalten werden, weniger ist da eigentlich immer mehr.

 

[Mark110]

Danke für deinen Beitrag maki.
Also es ist eigentlich ein Test den Du da siehst, es ist vielmehr die Logik der Testautomatisierung, ein Objekt aus einer Webseite zu finden. Rational bietet dafür die find() methode an.
Wenn ich z.b. folgende websiete habe:

<html>
<body>
<input type="text" name="test">
</body>
</html>

dann kann ich mit meinen methoden z.b. prop1=name value1=test auf dieses objekt zugreifen und z.b. eine setText() methode aufrufen die mir das feld mit einem text füllt.

Es kann manchmal sein, dass man mehr als ein Attribut benötigt deshalb habe ich mehrere Methoden die ähnlich aussehen.

Die richtigen Tests sind in einem eigenen package, in dem dann beispielsweise eine testklasse FuelleGui.java heißt.
Für jede Webseite habe ich eine Klasse erstellt mit Methoden, in denen ich die Webseiten Objekte zurück erhalte. Also jeden Button, jedes Eingabefeld etc.

Meine Testklasse initialisiert dann diese Klasse und greift über die MEthoden auf die Objekte zu.
Z.b.
Login login = new Login(); // Die Login Page

login.getFieldKundennummer().setText("51");
login.getFieldBenutzer().setText("test5");
login.getFieldPasswort().setText("test");
login.getButtonAnmelden().click();


Rational bietet alles was java auch bietet also auch asserts.
Was sollte ich aber an der Stelle mit Asserts?

Das mein Exceptionhandling mies ist möchte ich nciht abstreiten.
Ich lerne java seit Januar und testautom. ist neuland für mich.

 

[maki]

Wie gesagt, ein test sollte kein Exceptionhandling haben, am besten mit die Methode "throws Exception" deklarieren.
Bietet Rational denn typische Test Methoden wie assertEqual, assertNotNull, assert... und vor allem fail?

Ansonsten würde ich dir raten ein Buch über Tests zu lesen und erstmal mit UnitTests anzufangen bevor du dich an Integrationstests wagst, es gibt ein paar Dinge zu beachten

 

[Ariol]

StringBuilder sollte/könnte evtl. Geschwindigkeitsvorteile beim Zusammensetzen der Fehlermeldungen bringen.

 

[Guest]

public abstract class FindTestObject extends RationalTestScript {

	
	// **********************************************
	// The class defines the methodes to find objects 
	// **********************************************
	
...
	// ************************************** 
	// Methods that try to get the Testobject
	// **************************************
		

	public TestObject getTestObject(String property, String value, String property2, String value2) 	
	
	// Calculate runtime
	
	public String getElapsedTime(long start, long end)

Könnte doch auch so aussehen. Der Übersichtlichkeit halber. IDE's wie Eclipse zeigen dann auch was eine bestimmte Methode macht, wenn Sie jemand aufruft. Man kann eine "Doku" generieren lassen. Sinnvolle Javadocs werden gelesen über In-Linekomentare wird eher geschmunzelt.

/**
  *
  * The class defines the methodes to find objects
  */
public abstract class FindTestObject extends RationalTestScript {



	/**
	 *
	 * Methods that try to get the Testobject
	 * param property  blabalaalb
	 * param value       eine ganz wichtiger Value
	 * param property2
	 * param value2
	 */
	public TestObject getTestObject(String property, String value, String property2, String value2) 	
	
	/**
	 *
	 * Calculate runtime
	 * param start der Start 
	 * param end  das Ende
	 */	
	public String getElapsedTime(long start, long end)

 

[Mark110]

wenn ich bei solchen javadoc kommentaren die auto format funktion von eclipse benutze zerschiesst eclipse den kommentar.

 

[Guest]

wenn ich bei solchen javadoc kommentaren die auto format funktion von eclipse benutze zerschiesst eclipse den kommentar.

Sorry, aber das hört sich doch ziemlich "hahnebüchen" an ???:L

"Der Autoformater (Hund) hat meine Javadocs zerschossen (Hausaufgaben gefressen)"

Javadocs und autoformat machen bei mir das folgende. Und ich benutze auch Eclipse.

import org.apache.log4j.Logger;

import tchelperclass.util.Log;

import com.rational.test.ft.ObjectNotFoundException;
import com.rational.test.ft.RationalTestException;
import com.rational.test.ft.object.interfaces.TestObject;
import com.rational.test.ft.script.RationalTestScript;
import com.rational.test.ft.value.RegularExpression;

/**
 * The class defines the methodes to find objects
 * 
 * @author Wer denn?
 *
 */
public abstract class FindTestObject extends RationalTestScript {

	protected static final Logger log = Logger.getLogger(FindTestObject.class);

	protected static final double timeOut = 15;

	protected static final double timeBetweenChecks = 1;

	protected TestObject rootTestObject = null;

	protected TestObject[] to = null;

	protected long start = System.currentTimeMillis();

	protected long end;

	private int timer = 0;

	public abstract TestObject getRootObject();

	/**
	 * Methods that try to get the Testobject
	 * 
	 * @param property bla bla blup
	 * @param value schwup die wup
	 * @param property2 la le lu
	 * @param value2 und das im nu
	 * @return vielleicht ein TestObject
	 */
	public TestObject getTestObject(String property, String value,
			String property2, String value2) {
		try {
			rootTestObject = getRootObject();
			while (timer < timeOut) {
				// Try to find TestObject until timout
				to = rootTestObject.find(atDescendant(property, value,
						property2, value2));
				if (to.length > 2) {
					log
							.error("More than one Object found on website! Methode getTestObject("
									+ property
									+ ","
									+ value
									+ ","
									+ property2
									+ "," + value2 + ")");
					throw new ObjectNotFoundException(
							"TestObject not found on Website");
				}
				if (to.length != 0) {
					end = System.currentTimeMillis();
					log.debug("Testobject found within: "
							+ getElapsedTime(start, end) + " seconds");
					start = System.currentTimeMillis();
					// Reset timer
					timer = 0;
					return ((TestObject) (to[0]));
				}
				unregister(to);
				sleep(timeBetweenChecks);
				timer++;
				System.out.println(timer);
			}
			log.error("TestObject not found in Website. Methode getTestObject("
					+ property + "," + value + "," + property2 + "," + value2
					+ ")");
			throw new ObjectNotFoundException("TestObject not found in Website");
		} catch (RationalTestException rte) {
			log.error("Rational Exception: Methode getTestObject(" + property
					+ "," + value + "," + property2 + "," + value2 + ")");
			throw new ObjectNotFoundException("TestObject not found in Website");
		}
	}

	/**
	 * Und das ganze von vorn
	 * 
	 * @param property
	 * @param value
	 * @param property2
	 * @param reg
	 * @return
	 */
	public TestObject getTestObject(String property, String value,
			String property2, RegularExpression reg) {
		try {
			rootTestObject = getRootObject();
			while (timer < timeOut) {
				// Try to find TestObject until timout
				to = rootTestObject.find(atDescendant(property, value,
						property2, reg));
				if (to.length > 2) {
					log
							.error("More than one Object found on website. Methode getTestObject("
									+ property
									+ ","
									+ value
									+ ","
									+ property2
									+ "," + reg + ")");
					throw new ObjectNotFoundException(
							"TestObject not found on Website");
				}
				if (to.length != 0) {
					end = System.currentTimeMillis();
					log.debug("Testobject found within: "
							+ getElapsedTime(start, end) + " seconds");
					start = System.currentTimeMillis();
					// Reset timer
					timer = 0;
					return ((TestObject) (to[0]));
				}
				unregister(to);
				sleep(timeBetweenChecks);
				timer++;
			}
			log.error("TestObject not found in Website.Methode getTestObject("
					+ property + "," + value + "," + property2 + "," + reg
					+ ")");
			throw new ObjectNotFoundException("TestObject not found in Website");
		} catch (RationalTestException rte) {
			log.error("Rational Exception: Methode getTestObject(" + property
					+ "," + value + "," + property2 + "," + reg + ")");
			throw new ObjectNotFoundException("TestObject not found in Website");
		}
	}

	/**
	 * jetzt wirds aber langweilig
	 * 
	 * @param testObject
	 * @param property
	 * @param value
	 * @return oh ohoh
	 */
	public TestObject getTestObject(TestObject testObject, String property,
			String value) {
		TestObject childNode = null;

		try {
			while (timer < timeOut) {
				// Try to find TestObject until timeout
				childNode = testObject;
				to = childNode.find(atDescendant(property, value));

				if (to.length > 2) {
					log
							.error("More than one Object found on website. Methode getTestObject("
									+ testObject
									+ ","
									+ property
									+ ","
									+ value
									+ ")");
					throw new ObjectNotFoundException(
							"TestObject not found on Website");
				}
				if (to.length != 0) {
					end = System.currentTimeMillis();
					log.debug("Testobject found within: "
							+ getElapsedTime(start, end) + " seconds");
					start = System.currentTimeMillis();
					// Reset timer
					timer = 0;
					return ((TestObject) (to[0]));
				}
				unregister(to);
				sleep(timeBetweenChecks);
				timer++;
			}
			log.error("TestObject not found in Website. Methode getTestObject("
					+ testObject + "," + property + "," + value + ")");
			throw new ObjectNotFoundException("TestObject not found in Website");
		} catch (RationalTestException rte) {
			log.error("Rational Exception: Methode getTestObject(" + testObject
					+ "," + property + "," + value + ")");
			throw new ObjectNotFoundException(
					"getTestObject() - Object not found");
		}
	}

	/**
	 *  elapsed Time
	 *  @param start ein long
	 *  @param end und nochmal ein long
	 */
	public String getElapsedTime(long start, long end) {
		long calcElpasedTime = (end / 1000) - (start / 1000);
		//long calcElpasedTimeMS = (end)-(start);
		String elapsedSec = Long.toString(calcElpasedTime);
		//String elapsedMS = Long.toString(calcElpasedTimeMS);
		return elapsedSec;
	}
}

 

[Mark110]

ok ich habe normale mehrzeilige java kommentare gemacht.