Removed redundant controller and fail check from the test

javaee/tests/src/test/java/org/jboss/forge/addon/javaee/jpa/NewFieldWizardTest.java

@@ -37,6 +37,8 @@
 import org.junit.Test;
 import org.junit.runner.RunWith;
 
+import static org.hamcrest.core.IsEqual.equalTo;
+import static org.junit.Assert.assertThat;
 import static org.junit.Assert.fail;
 
 @RunWith(Arquillian.class)
@@ -219,20 +221,14 @@ public void testNewOneToManyEagerFetchField() throws Exception
          controller.setValueFor("type", otherEntity.getJavaSource().getCanonicalName());
          controller.setValueFor("relationshipType", RelationshipType.ONE_TO_MANY);
          Assert.assertTrue(controller.canMoveToNextStep());
-         WizardCommandController next = controller.next();
-         next.setValueFor("fetchType", FetchType.EAGER);
+         controller.next();
+         controller.setValueFor("fetchType", FetchType.EAGER);
          Assert.assertTrue(controller.canExecute());
          Result result = controller.execute();
          Assert.assertFalse(result instanceof Failed);
-         if (result instanceof CompositeResult)
-         {
-            CompositeResult compositeResult = (CompositeResult) result;
-            Assert.assertEquals("Relationship One-to-Many created", compositeResult.getResults().get(1).getMessage());
-         }
-         else
-         {
-            fail("Result is of type " + CompositeResult.class.getSimpleName() + " is expected.");
-         }
+         assertThat ("Result should be of type CompositeResult", result instanceof CompositeResult, equalTo(true));
+         CompositeResult compositeResult = (CompositeResult) result;
+         Assert.assertEquals("Relationship One-to-Many created", compositeResult.getResults().get(1).getMessage());
       }
       JavaClass javaClass = (JavaClass) entity.getJavaSource();
       Assert.assertTrue(javaClass.hasField("accounts"));