[isis] branch ISIS-3108 updated: ISIS-3108: fixing unit tests

[danhaywood]

This is an automated email from the ASF dual-hosted git repository.

danhaywood pushed a commit to branch ISIS-3108
in repository https://gitbox.apache.org/repos/asf/isis.git


The following commit(s) were added to refs/heads/ISIS-3108 by this push:
     new 7d2d74efbd ISIS-3108: fixing unit tests
7d2d74efbd is described below

commit 7d2d74efbdb3b68076fdda34adccf6236bde19ff
Author: Dan Haywood <d...@haywood-associates.co.uk>
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();
             }