This is an automated email from the ASF dual-hosted git repository. danhaywood pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/isis.git
commit 77330c42bada206c1f62e0ca83f2941883fe7508 Author: Dan Haywood <[email protected]> AuthorDate: Tue Aug 2 16:19:20 2022 +0100 ISIS-3108: fixing unit tests --- .../DomainObjectAnnotationFacetFactoryTest.java | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/core/metamodel/src/test/java/org/apache/isis/core/metamodel/facets/object/domainobject/DomainObjectAnnotationFacetFactoryTest.java b/core/metamodel/src/test/java/org/apache/isis/core/metamodel/facets/object/domainobject/DomainObjectAnnotationFacetFactoryTest.java index f6c649657a..aa2fd69242 100644 --- a/core/metamodel/src/test/java/org/apache/isis/core/metamodel/facets/object/domainobject/DomainObjectAnnotationFacetFactoryTest.java +++ b/core/metamodel/src/test/java/org/apache/isis/core/metamodel/facets/object/domainobject/DomainObjectAnnotationFacetFactoryTest.java @@ -22,6 +22,7 @@ import java.util.UUID; import javax.inject.Named; +import org.assertj.core.api.Assertions; import org.jmock.Expectations; import org.junit.After; import org.junit.Assert; @@ -148,9 +149,10 @@ extends AbstractFacetFactoryJUnit4TestCase { .forTesting(DomainObjectAnnotationFacetFactoryTest.Customer.class, mockMethodRemover, facetHolder); facetFactory.processEntityChangePublishing(context.synthesizeOnType(DomainObject.class), context); - final Facet facet = facetHolder.getFacet(EntityChangePublishingFacet.class); + final EntityChangePublishingFacet facet = facetHolder.getFacet(EntityChangePublishingFacet.class); assertThat(facet, is(notNullValue())); Assert.assertTrue(facet instanceof EntityChangePublishingFacetFromConfiguration); + Assertions.assertThat(facet.isEnabled()).isTrue(); expectNoMethodsRemoved(); } @@ -163,7 +165,8 @@ extends AbstractFacetFactoryJUnit4TestCase { .forTesting(DomainObjectAnnotationFacetFactoryTest.Customer.class, mockMethodRemover, facetHolder)); final EntityChangePublishingFacet facet = facetHolder.getFacet(EntityChangePublishingFacet.class); - Assert.assertNull(facet); + Assert.assertNotNull(facet); + Assertions.assertThat(facet.isEnabled()).isFalse(); expectNoMethodsRemoved(); } @@ -179,9 +182,10 @@ extends AbstractFacetFactoryJUnit4TestCase { facetFactory.process(ProcessClassContext .forTesting(CustomerWithDomainObjectAndAuditingSetToAsConfigured.class, mockMethodRemover, facetHolder)); - final Facet facet = facetHolder.getFacet(EntityChangePublishingFacet.class); + final EntityChangePublishingFacet facet = facetHolder.getFacet(EntityChangePublishingFacet.class); Assert.assertNotNull(facet); Assert.assertTrue(facet instanceof EntityChangePublishingFacetForDomainObjectAnnotationAsConfigured); + Assertions.assertThat(facet.isEnabled()).isTrue(); expectNoMethodsRemoved(); } @@ -194,7 +198,8 @@ extends AbstractFacetFactoryJUnit4TestCase { .forTesting(CustomerWithDomainObjectAndAuditingSetToAsConfigured.class, mockMethodRemover, facetHolder)); final EntityChangePublishingFacet facet = facetHolder.getFacet(EntityChangePublishingFacet.class); - Assert.assertNull(facet); + Assert.assertNotNull(facet); + Assertions.assertThat(facet.isEnabled()).isFalse(); expectNoMethodsRemoved(); }
