Added some commentary to the new test to explain what and why.

larsbutler · Dec 8, 2011 · 93bdaa4 · 93bdaa4
1 parent 0707009
commit 93bdaa4
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/java_tests/org/opensha/sha/earthquake/rupForecastImpl/FloatingPoissonFaultSourceTest.java b/java_tests/org/opensha/sha/earthquake/rupForecastImpl/FloatingPoissonFaultSourceTest.java
@@ -240,6 +240,19 @@ public void peerTestSet1Case5() throws Exception {
 		return expectedResults;
 	}
 
+	/**
+	 * This test was introduced as a results of this bug report:
+	 * https://bugs.launchpad.net/openquake/+bug/901092
+	 * 
+	 * FloatingPoissonFaultSource overrides the setTectonicRegionType
+	 * method from the base class (ProbEqkSource).
+	 * 
+	 * However, the override fails to do what the base implementation
+	 * does: set the tectonic region type.
+	 * 
+	 * I wonder how long this bug has been here, and why we found
+	 * it just now.
+	 */
 	@Test
 	public void testSetTectonicRegionTypeActuallySetsTectonicRegionType() {
 	    FloatingPoissonFaultSource src =