BAEL-65 - moving from sout to logger

This commit is contained in:
Slavisa Baeldung 2016-07-07 15:42:13 +02:00
parent 5733676887
commit 30f00a34af

View File

@ -14,6 +14,8 @@ import org.junit.Assume;
import org.junit.Before; import org.junit.Before;
import org.junit.Test; import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.beans.factory.annotation.Autowired; import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.test.context.ContextConfiguration; import org.springframework.test.context.ContextConfiguration;
import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
@ -23,9 +25,11 @@ import com.baeldung.persistence.model.Foo;
import com.baeldung.spring.PersistenceConfig; import com.baeldung.spring.PersistenceConfig;
@RunWith(SpringJUnit4ClassRunner.class) @RunWith(SpringJUnit4ClassRunner.class)
@ContextConfiguration(classes = { PersistenceConfig.class }, loader = AnnotationConfigContextLoader.class) @ContextConfiguration(classes = {PersistenceConfig.class}, loader = AnnotationConfigContextLoader.class)
public class FooStoredProceduresIntegrationTest { public class FooStoredProceduresIntegrationTest {
private static final Logger LOGGER = LoggerFactory.getLogger(FooStoredProceduresIntegrationTest.class);
@Autowired @Autowired
private SessionFactory sessionFactory; private SessionFactory sessionFactory;
@ -48,8 +52,7 @@ public class FooStoredProceduresIntegrationTest {
sqlQuery.list(); sqlQuery.list();
return true; return true;
} catch (SQLGrammarException e) { } catch (SQLGrammarException e) {
System.out LOGGER.error("WARNING : GetFoosByName() Procedure is may be missing ", e);
.println("WARNING : GetFoosByName() Procedure is may be missing ");
return false; return false;
} }
} }
@ -61,8 +64,7 @@ public class FooStoredProceduresIntegrationTest {
sqlQuery.list(); sqlQuery.list();
return true; return true;
} catch (SQLGrammarException e) { } catch (SQLGrammarException e) {
System.out LOGGER.error("WARNING : GetAllFoos() Procedure is may be missing ", e);
.println("WARNING : GetAllFoos() Procedure is may be missing ");
return false; return false;
} }
} }
@ -83,8 +85,7 @@ public class FooStoredProceduresIntegrationTest {
@SuppressWarnings("unchecked") @SuppressWarnings("unchecked")
List<Foo> allFoos = sqlQuery.list(); List<Foo> allFoos = sqlQuery.list();
for (Foo foo : allFoos) { for (Foo foo : allFoos) {
System.out.println("getAllFoos() SQL Query result : " LOGGER.info("getAllFoos() SQL Query result : {}", foo.getName());
+ foo.getName());
} }
assertEquals(allFoos.size(), fooService.findAll().size()); assertEquals(allFoos.size(), fooService.findAll().size());
@ -93,8 +94,7 @@ public class FooStoredProceduresIntegrationTest {
@SuppressWarnings("unchecked") @SuppressWarnings("unchecked")
List<Foo> allFoos2 = namedQuery.list(); List<Foo> allFoos2 = namedQuery.list();
for (Foo foo : allFoos2) { for (Foo foo : allFoos2) {
System.out.println("getAllFoos() NamedQuery result : " LOGGER.info("getAllFoos() NamedQuery result : {}", foo.getName());
+ foo.getName());
} }
assertEquals(allFoos2.size(), fooService.findAll().size()); assertEquals(allFoos2.size(), fooService.findAll().size());
} }
@ -110,8 +110,7 @@ public class FooStoredProceduresIntegrationTest {
@SuppressWarnings("unchecked") @SuppressWarnings("unchecked")
List<Foo> allFoosByName = sqlQuery.list(); List<Foo> allFoosByName = sqlQuery.list();
for (Foo foo : allFoosByName) { for (Foo foo : allFoosByName) {
System.out.println("getFoosByName() using SQL Query : found => " LOGGER.info("getFoosByName() using SQL Query : found => {}", foo.toString());
+ foo.toString());
} }
// Stored procedure getFoosByName using getNamedQuery() // Stored procedure getFoosByName using getNamedQuery()
@ -120,8 +119,7 @@ public class FooStoredProceduresIntegrationTest {
@SuppressWarnings("unchecked") @SuppressWarnings("unchecked")
List<Foo> allFoosByName2 = namedQuery.list(); List<Foo> allFoosByName2 = namedQuery.list();
for (Foo foo : allFoosByName2) { for (Foo foo : allFoosByName2) {
System.out.println("getFoosByName() using Native Query : found => " LOGGER.info("getFoosByName() using Native Query : found => {}", foo.toString());
+ foo.toString());
} }
} }