updated due to edits for chap 1,2,3,4,5

eviltester · eviltester · commit 8c674344428c · 2014-06-13T12:50:07.000+01:00
diff --git a/source/pom.xml b/source/pom.xml
@@ -15,21 +15,22 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     </properties>
 
-
     <dependencies>
 
         <dependency>
             <groupId>junit</groupId>
             <artifactId>junit</artifactId>
             <version>4.11</version>
         </dependency>
+
     </dependencies>
 
     <build>
         <plugins>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-compiler-plugin</artifactId>
+                <version>3.1</version>
                 <configuration>
                     <source>1.7</source>
                     <target>1.7</target>
diff --git a/source/reportingpom.xml b/source/reportingpom.xml
@@ -16,7 +16,6 @@
         http://maven.apache.org/surefire/maven-surefire-report-plugin/report-mojo.html
 
         mvn clean test -Dmaven.source=reportingpom.xml
-        mvn surefire-report:report -Dmaven.source=reportingpom.xml
         -->
 
     <properties>
diff --git a/source/src/main/java/com/javafortesters/domainobject/TestAppEnv.java b/source/src/main/java/com/javafortesters/domainobject/TestAppEnv.java
@@ -1,6 +1,5 @@
 package com.javafortesters.domainobject;
 
-
 public class TestAppEnv {
 
     public static final String DOMAIN = "192.123.0.3";
@@ -9,4 +8,4 @@ public class TestAppEnv {
     public static String getUrl() {
         return "http://" + DOMAIN + ":" + PORT;
     }
-}
+}
diff --git a/source/src/main/java/com/javafortesters/domainobject/interim/TestAppEnv.java b/source/src/main/java/com/javafortesters/domainobject/interim/TestAppEnv.java
@@ -10,7 +10,7 @@ public static String getUrl() {
 */
 
 
-package com.javafortesters.domainobject.empty;
+package com.javafortesters.domainobject.interim;
 
 public class TestAppEnv {
 
diff --git a/source/src/test/java/com/javafortesters/myfirsttest/empty/MyFirstTest.java b/source/src/test/java/com/javafortesters/myfirsttest/empty/MyFirstTest.java
diff --git a/source/src/test/java/com/javafortesters/myfirsttest/examples/naming/NameClass.java b/source/src/test/java/com/javafortesters/myfirsttest/examples/naming/NameClass.java
@@ -0,0 +1,16 @@
+package com.javafortesters.myfirsttest.examples.naming;
+
+import org.junit.Test;
+
+import static org.junit.Assert.assertTrue;
+
+public class NameClass {
+
+    @Test
+    public void whenClassNameHasNoTestInItThenItIsNotRun(){
+        // this test will not run from maven so i can make
+        // a failing test... it fails in the IDE
+        assertTrue("whenClassNameHasNoTestInItThenItIsNotRun",
+                   false);
+    }
+}
diff --git a/source/src/test/java/com/javafortesters/myfirsttest/examples/naming/NameClassTest.java b/source/src/test/java/com/javafortesters/myfirsttest/examples/naming/NameClassTest.java
@@ -0,0 +1,15 @@
+package com.javafortesters.myfirsttest.examples.naming;
+
+import org.junit.Test;
+
+import static org.junit.Assert.assertTrue;
+
+public class NameClassTest {
+
+    @Test
+    public void whenClassHasTestAtEndThenTestIsRun(){
+        // this test will run from maven so it needs to pass
+        assertTrue("whenClassHasTestAtEndThenTestIsRun",
+                    true);
+    }
+}
diff --git a/source/src/test/java/com/javafortesters/myfirsttest/examples/naming/NameTestClass.java b/source/src/test/java/com/javafortesters/myfirsttest/examples/naming/NameTestClass.java
@@ -0,0 +1,16 @@
+package com.javafortesters.myfirsttest.examples.naming;
+
+import org.junit.Test;
+
+import static org.junit.Assert.assertTrue;
+
+public class NameTestClass {
+
+    @Test
+    public void whenClassHasTestInMiddleThenTestIsNotRun(){
+        // this test will not run from maven so i can make
+        // a failing test... it fails in the IDE
+        assertTrue("whenClassHasTestInMiddleThenTestIsNotRun",
+                    false);
+    }
+}
diff --git a/source/src/test/java/com/javafortesters/myfirsttest/examples/naming/TestNameClass.java b/source/src/test/java/com/javafortesters/myfirsttest/examples/naming/TestNameClass.java
@@ -0,0 +1,15 @@
+package com.javafortesters.myfirsttest.examples.naming;
+
+import org.junit.Test;
+
+import static org.junit.Assert.assertTrue;
+
+public class TestNameClass {
+
+    @Test
+    public void whenClassHasTestAtFrontThenTestIsRun(){
+        // this test will run from maven so it needs to pass
+        assertTrue("whenClassHasTestAtFrontThenTestIsRun",
+                   true);
+    }
+}
diff --git a/source/src/test/java/com/javafortesters/myfirsttest/exercises/MyFirstTestExercisesTest.java b/source/src/test/java/com/javafortesters/myfirsttest/exercises/MyFirstTestExercisesTest.java
@@ -7,7 +7,7 @@
 import static org.junit.Assert.assertEquals;
 
 
-public class MyFirstTestExercises {
+public class MyFirstTestExercisesTest {
 
     @Test
     public void canAddTwoPlusTwo(){
diff --git a/source/src/test/java/com/javafortesters/myfirsttest/interim/emptyMethod/MyFirstTest.java b/source/src/test/java/com/javafortesters/myfirsttest/interim/emptyMethod/MyFirstTest.java
@@ -1,4 +1,4 @@
-package com.javafortesters.myfirsttest.empty.emptyMethod;
+package com.javafortesters.myfirsttest.interim.emptyMethod;
 
 public class MyFirstTest {
 
diff --git a/source/src/test/java/com/javafortesters/myfirsttest/interim/emptyMethodTestAnnotated/MyFirstTest.java b/source/src/test/java/com/javafortesters/myfirsttest/interim/emptyMethodTestAnnotated/MyFirstTest.java
@@ -1,4 +1,4 @@
-package com.javafortesters.myfirsttest.empty.emptyMethodTestAnnotated;
+package com.javafortesters.myfirsttest.interim.emptyMethodTestAnnotated;
 
 
 import org.junit.Test;
diff --git a/source/src/test/java/com/javafortesters/myfirsttest/interim/emptyWithTemplate/MyFirstTest.java b/source/src/test/java/com/javafortesters/myfirsttest/interim/emptyWithTemplate/MyFirstTest.java
@@ -1,4 +1,4 @@
-package com.javafortesters.myfirsttest.empty.emptyWithTemplate;
+package com.javafortesters.myfirsttest.interim.emptyWithTemplate;
 
 /**
  * Created with IntelliJ IDEA.
diff --git a/source/src/test/java/com/javafortesters/testswithotherclasses/IntegerExamplesTest.java b/source/src/test/java/com/javafortesters/testswithotherclasses/IntegerExamplesTest.java
@@ -4,7 +4,7 @@
 
 import static org.junit.Assert.assertEquals;
 
-public class IntegerTestExamples {
+public class IntegerExamplesTest {
 
     @Test
     public void integerExploration(){
diff --git a/source/src/test/java/com/javafortesters/testswithotherclasses/IntegerExercisesTest.java b/source/src/test/java/com/javafortesters/testswithotherclasses/IntegerExercisesTest.java
@@ -4,7 +4,7 @@
 
 import static org.junit.Assert.assertEquals;
 
-public class IntegerExercises {
+public class IntegerExercisesTest {
 
     @Test
     public void canConfirmIntMinAndMaxLimits(){
diff --git a/source/src/test/java/com/javafortesters/testswithotherclasses/empty/IntegerTestExamples.java b/source/src/test/java/com/javafortesters/testswithotherclasses/empty/IntegerTestExamples.java
diff --git a/source/src/test/java/com/javafortesters/testswithotherclasses/interim/IntegerExamplesTest.java b/source/src/test/java/com/javafortesters/testswithotherclasses/interim/IntegerExamplesTest.java
@@ -0,0 +1,9 @@
+package com.javafortesters.testswithotherclasses.interim;
+import org.junit.Test;
+
+public class IntegerExamplesTest {
+
+    @Test
+    public void integerExploration(){
+    }
+}
diff --git a/source/src/test/java/com/javafortesters/testswithotherclasses/warning/WarningsForIntegerTest.java b/source/src/test/java/com/javafortesters/testswithotherclasses/warning/WarningsForIntegerTest.java
@@ -0,0 +1,33 @@
+package com.javafortesters.testswithotherclasses.warning;
+
+import org.junit.Test;
+
+import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.assertEquals;
+
+public class WarningsForIntegerTest {
+
+    @Test(expected = AssertionError.class)
+    public void intEqualsIntVsIntegerEqualsInteger(){
+
+        assertEquals(4,4);
+        assertTrue(4==4);
+
+        Integer firstFour = new Integer(4);
+        Integer secondFour = new Integer(4);
+
+        assertEquals(firstFour, secondFour);
+        assertTrue(firstFour==secondFour);
+    }
+
+    @Test
+    public void integerEqualsIntegerAssertTrue(){
+
+        Integer firstFour = new Integer(4);
+        Integer secondFour = new Integer(4);
+
+        assertEquals(firstFour, secondFour);
+        assertTrue(firstFour.equals(secondFour));
+    }
+
+}
diff --git a/source/src/test/java/com/javafortesters/testwithourownclasses/domainobject/TestAppEnvironmentTest.java b/source/src/test/java/com/javafortesters/testwithourownclasses/domainobject/TestAppEnvironmentTest.java
@@ -2,7 +2,6 @@
 
 import com.javafortesters.domainobject.TestAppEnv;
 import org.junit.Test;
-
 import static org.junit.Assert.assertEquals;
 
 public class TestAppEnvironmentTest {
@@ -25,4 +24,4 @@ public void canGetDomainAndPortStatically(){
                 "67",
                 TestAppEnv.PORT);
     }
-}
+}
diff --git a/source/src/test/java/com/javafortesters/testwithourownclasses/domainobject/exercises/TestAppEnvironmentNoStaticImportTest.java b/source/src/test/java/com/javafortesters/testwithourownclasses/domainobject/exercises/TestAppEnvironmentNoStaticImportTest.java
@@ -0,0 +1,38 @@
+package com.javafortesters.testwithourownclasses.domainobject.exercises;
+
+import com.javafortesters.domainobject.TestAppEnv;
+import org.junit.Assert;
+import org.junit.Test;
+
+// I could import everything on TestAppEnv statically, and then
+// I don't need to prefix getUrl with TestAppEnv
+/*
+import static com.javafortesters.domainobject.TestAppEnv.*;
+*/
+// If I just import the DOMAIN and PORT then I still need to
+// prefix getUrl with TestAppEnv
+import static com.javafortesters.domainobject.TestAppEnv.DOMAIN;
+import static com.javafortesters.domainobject.TestAppEnv.PORT;
+
+
+public class TestAppEnvironmentNoStaticImportTest {
+
+    @Test
+    public void canGetUrlStatically(){
+        Assert.assertEquals("Returns Hard Coded URL",
+                "http://192.123.0.3:67",
+                TestAppEnv.getUrl());
+    }
+
+    @Test
+    public void canGetDomainAndPortStatically(){
+
+        Assert.assertEquals("Just the Domain",
+                "192.123.0.3",
+                DOMAIN);
+
+        Assert.assertEquals("Just the port",
+                "67",
+                PORT);
+    }
+}
diff --git a/source/src/test/java/com/javafortesters/testwithourownclasses/domainobject/interim/TestAppEnvironmentTest.java b/source/src/test/java/com/javafortesters/testwithourownclasses/domainobject/interim/TestAppEnvironmentTest.java
@@ -6,7 +6,7 @@ public class TestAppEnvironmentTest {
 */
 
 
-package com.javafortesters.testwithourownclasses.domainobject.empty;
+package com.javafortesters.testwithourownclasses.domainobject.interim;
 
 import org.junit.Test;
 
