Commit 77d7a60ddcb7d77b551f6e2f9dd1c9bab33a10b2
1 parent
25e25c89
Exists in
master
Ajustes nos testes.
Showing
3 changed files
with
15 additions
and
13 deletions
Show diff stats
impl/extension/jsf/src/main/java/br/gov/frameworkdemoiselle/template/AbstractListPageBean.java
| @@ -80,7 +80,6 @@ public abstract class AbstractListPageBean<T, I> extends AbstractPageBean implem | @@ -80,7 +80,6 @@ public abstract class AbstractListPageBean<T, I> extends AbstractPageBean implem | ||
| 80 | 80 | ||
| 81 | @Override | 81 | @Override |
| 82 | public DataModel<T> getDataModel() { | 82 | public DataModel<T> getDataModel() { |
| 83 | - System.out.println("MODIFICADO!!!!!##@#@#@@"); | ||
| 84 | if (this.dataModel == null) { | 83 | if (this.dataModel == null) { |
| 85 | this.dataModel = new ListDataModel<T>(this.getResultList()); | 84 | this.dataModel = new ListDataModel<T>(this.getResultList()); |
| 86 | } | 85 | } |
impl/extension/jsf/src/test/java/br/gov/frameworkdemoiselle/template/AbstractEditPageBeanTest.java
| @@ -62,6 +62,7 @@ import org.powermock.reflect.Whitebox; | @@ -62,6 +62,7 @@ import org.powermock.reflect.Whitebox; | ||
| 62 | import br.gov.frameworkdemoiselle.DemoiselleException; | 62 | import br.gov.frameworkdemoiselle.DemoiselleException; |
| 63 | import br.gov.frameworkdemoiselle.internal.producer.ResourceBundleProducer; | 63 | import br.gov.frameworkdemoiselle.internal.producer.ResourceBundleProducer; |
| 64 | import br.gov.frameworkdemoiselle.util.Beans; | 64 | import br.gov.frameworkdemoiselle.util.Beans; |
| 65 | +import br.gov.frameworkdemoiselle.util.Faces; | ||
| 65 | import br.gov.frameworkdemoiselle.util.Parameter; | 66 | import br.gov.frameworkdemoiselle.util.Parameter; |
| 66 | import br.gov.frameworkdemoiselle.util.Reflections; | 67 | import br.gov.frameworkdemoiselle.util.Reflections; |
| 67 | import br.gov.frameworkdemoiselle.util.ResourceBundle; | 68 | import br.gov.frameworkdemoiselle.util.ResourceBundle; |
| @@ -69,7 +70,7 @@ import br.gov.frameworkdemoiselle.util.ResourceBundle; | @@ -69,7 +70,7 @@ import br.gov.frameworkdemoiselle.util.ResourceBundle; | ||
| 69 | import com.sun.faces.util.Util; | 70 | import com.sun.faces.util.Util; |
| 70 | 71 | ||
| 71 | @RunWith(PowerMockRunner.class) | 72 | @RunWith(PowerMockRunner.class) |
| 72 | -@PrepareForTest({ Parameter.class, Beans.class, Reflections.class, Converter.class, FacesContext.class, Util.class }) | 73 | +@PrepareForTest({ Parameter.class, Beans.class, Reflections.class, Converter.class, FacesContext.class, Util.class, Faces.class }) |
| 73 | public class AbstractEditPageBeanTest { | 74 | public class AbstractEditPageBeanTest { |
| 74 | 75 | ||
| 75 | private AbstractEditPageBean<Contact, Object> pageBean; | 76 | private AbstractEditPageBean<Contact, Object> pageBean; |
| @@ -231,7 +232,6 @@ public class AbstractEditPageBeanTest { | @@ -231,7 +232,6 @@ public class AbstractEditPageBeanTest { | ||
| 231 | 232 | ||
| 232 | String value = "1"; | 233 | String value = "1"; |
| 233 | expect(parameter.getValue()).andReturn(value); | 234 | expect(parameter.getValue()).andReturn(value); |
| 234 | - expect(Util.getConverterForClass(String.class, facesContext)).andReturn(null); | ||
| 235 | 235 | ||
| 236 | replayAll(); | 236 | replayAll(); |
| 237 | assertEquals(value, pageBean.getId()); | 237 | assertEquals(value, pageBean.getId()); |
| @@ -241,23 +241,24 @@ public class AbstractEditPageBeanTest { | @@ -241,23 +241,24 @@ public class AbstractEditPageBeanTest { | ||
| 241 | @Test | 241 | @Test |
| 242 | @SuppressWarnings("unchecked") | 242 | @SuppressWarnings("unchecked") |
| 243 | public void testGetLongId() { | 243 | public void testGetLongId() { |
| 244 | - mockStatic(Util.class); | ||
| 245 | - | 244 | + mockStatic(Faces.class); |
| 245 | + | ||
| 246 | FacesContext facesContext = createMock(FacesContext.class); | 246 | FacesContext facesContext = createMock(FacesContext.class); |
| 247 | - UIViewRoot viewRoot = createMock(UIViewRoot.class); | ||
| 248 | Converter converter = createMock(Converter.class); | 247 | Converter converter = createMock(Converter.class); |
| 248 | + UIViewRoot viewRoot = createMock(UIViewRoot.class); | ||
| 249 | Parameter<String> parameter = createMock(Parameter.class); | 249 | Parameter<String> parameter = createMock(Parameter.class); |
| 250 | - | 250 | + |
| 251 | setInternalState(pageBean, "facesContext", facesContext); | 251 | setInternalState(pageBean, "facesContext", facesContext); |
| 252 | setInternalState(pageBean, "id", parameter); | 252 | setInternalState(pageBean, "id", parameter); |
| 253 | setInternalState(pageBean, "idClass", Long.class, AbstractEditPageBean.class); | 253 | setInternalState(pageBean, "idClass", Long.class, AbstractEditPageBean.class); |
| 254 | - | 254 | + |
| 255 | String value = "1"; | 255 | String value = "1"; |
| 256 | + | ||
| 256 | expect(parameter.getValue()).andReturn(value); | 257 | expect(parameter.getValue()).andReturn(value); |
| 257 | expect(facesContext.getViewRoot()).andReturn(viewRoot); | 258 | expect(facesContext.getViewRoot()).andReturn(viewRoot); |
| 258 | - expect(Util.getConverterForClass(Long.class, facesContext)).andReturn(converter); | 259 | + expect(Faces.getConverter(Long.class)).andReturn(converter); |
| 259 | expect(converter.getAsObject(facesContext, viewRoot, value)).andReturn(Long.valueOf(value)); | 260 | expect(converter.getAsObject(facesContext, viewRoot, value)).andReturn(Long.valueOf(value)); |
| 260 | - | 261 | + |
| 261 | replayAll(); | 262 | replayAll(); |
| 262 | assertEquals(Long.valueOf(value), pageBean.getId()); | 263 | assertEquals(Long.valueOf(value), pageBean.getId()); |
| 263 | verifyAll(); | 264 | verifyAll(); |
| @@ -270,7 +271,6 @@ public class AbstractEditPageBeanTest { | @@ -270,7 +271,6 @@ public class AbstractEditPageBeanTest { | ||
| 270 | setInternalState(pageBean, "facesContext", facesContext); | 271 | setInternalState(pageBean, "facesContext", facesContext); |
| 271 | setInternalState(pageBean, "idClass", Contact.class, AbstractEditPageBean.class); | 272 | setInternalState(pageBean, "idClass", Contact.class, AbstractEditPageBean.class); |
| 272 | setInternalState(pageBean, "bundle", bundle); | 273 | setInternalState(pageBean, "bundle", bundle); |
| 273 | - expect(Util.getConverterForClass(Contact.class, facesContext)).andReturn(null); | ||
| 274 | 274 | ||
| 275 | replayAll(); | 275 | replayAll(); |
| 276 | try { | 276 | try { |
impl/extension/jsf/src/test/java/br/gov/frameworkdemoiselle/util/FacesTest.java
| @@ -46,6 +46,7 @@ import java.util.HashMap; | @@ -46,6 +46,7 @@ import java.util.HashMap; | ||
| 46 | import java.util.List; | 46 | import java.util.List; |
| 47 | import java.util.Map; | 47 | import java.util.Map; |
| 48 | 48 | ||
| 49 | +import javax.faces.application.Application; | ||
| 49 | import javax.faces.application.FacesMessage; | 50 | import javax.faces.application.FacesMessage; |
| 50 | import javax.faces.component.UIViewRoot; | 51 | import javax.faces.component.UIViewRoot; |
| 51 | import javax.faces.context.FacesContext; | 52 | import javax.faces.context.FacesContext; |
| @@ -176,12 +177,14 @@ public class FacesTest { | @@ -176,12 +177,14 @@ public class FacesTest { | ||
| 176 | @SuppressWarnings("static-access") | 177 | @SuppressWarnings("static-access") |
| 177 | @Test | 178 | @Test |
| 178 | public void testGetConverter() { | 179 | public void testGetConverter() { |
| 180 | + Application application = PowerMock.createMock(Application.class); | ||
| 179 | Converter converter = PowerMock.createMock(Converter.class); | 181 | Converter converter = PowerMock.createMock(Converter.class); |
| 180 | PowerMock.mockStatic(Util.class); | 182 | PowerMock.mockStatic(Util.class); |
| 181 | 183 | ||
| 182 | expect(Beans.getReference(FacesContext.class)).andReturn(facesContext); | 184 | expect(Beans.getReference(FacesContext.class)).andReturn(facesContext); |
| 183 | - expect(Util.getConverterForClass(getClass(), facesContext)).andReturn(converter); | ||
| 184 | - | 185 | + expect(facesContext.getApplication()).andReturn(application); |
| 186 | + expect(application.createConverter(getClass())).andReturn(converter); | ||
| 187 | + | ||
| 185 | replayAll(); | 188 | replayAll(); |
| 186 | assertEquals(converter, faces.getConverter(getClass())); | 189 | assertEquals(converter, faces.getConverter(getClass())); |
| 187 | verifyAll(); | 190 | verifyAll(); |