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

ahuber pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/isis.git


The following commit(s) were added to refs/heads/master by this push:
     new eb33e0b3cb ISIS-3049: prepare new reg. test
eb33e0b3cb is described below

commit eb33e0b3cb081a2e00a300ab4f0a50d94df78446
Author: Andi Huber <[email protected]>
AuthorDate: Mon Jul 11 11:11:37 2022 +0200

    ISIS-3049: prepare new reg. test
    
    - testing annotation provided 'choicesFrom' fallback, if no explicit
    choices member-support is given
---
 .../domainmodel/MetaModelRegressionTest.java       |   2 +-
 .../src/test/resources/metamodel.xml.zip           | Bin 89706 -> 90721 bytes
 .../testdomain/model/good/ProperMemberSupport.java |  43 ++++++++++++++++++++-
 .../model/good/ProperMemberSupport_action1.java    |   3 ++
 .../model/good/ProperMemberSupport_action2.java    |   3 ++
 .../model/good/ProperMemberSupport_action3.java    |   3 ++
 .../model/good/ProperMemberSupport_action4.java    |  23 ++++++-----
 .../model/good/ProperMemberSupport_action5.java    |   3 ++
 ...tion5.java => ProperMemberSupport_action6.java} |  24 +++++-------
 .../good/ProperMemberSupport_collection1.java      |   3 ++
 .../good/ProperMemberSupport_collection2.java      |   3 ++
 .../good/ProperMemberSupport_collection3.java      |   3 ++
 .../model/good/ProperMemberSupport_property1.java  |   5 ++-
 .../model/good/ProperMemberSupport_property2.java  |   5 ++-
 .../model/good/ProperMemberSupport_property3.java  |   3 ++
 15 files changed, 97 insertions(+), 29 deletions(-)

diff --git 
a/regressiontests/stable-domainmodel/src/test/java/org/apache/isis/testdomain/domainmodel/MetaModelRegressionTest.java
 
b/regressiontests/stable-domainmodel/src/test/java/org/apache/isis/testdomain/domainmodel/MetaModelRegressionTest.java
index 2478a7b0b5..0421f4d6b3 100644
--- 
a/regressiontests/stable-domainmodel/src/test/java/org/apache/isis/testdomain/domainmodel/MetaModelRegressionTest.java
+++ 
b/regressiontests/stable-domainmodel/src/test/java/org/apache/isis/testdomain/domainmodel/MetaModelRegressionTest.java
@@ -83,7 +83,7 @@ class MetaModelRegressionTest {
         }
 
         //for maintenance
-        //createReferenceMetaModelFile(new 
File("d:/tmp/_scratch/metamodel.xml.zip"));
+        createReferenceMetaModelFile(new 
File("d:/tmp/_scratch/metamodel.xml.zip"));
     }
 
     @Test
diff --git 
a/regressiontests/stable-domainmodel/src/test/resources/metamodel.xml.zip 
b/regressiontests/stable-domainmodel/src/test/resources/metamodel.xml.zip
index 59b75137e3..5044160038 100644
Binary files 
a/regressiontests/stable-domainmodel/src/test/resources/metamodel.xml.zip and 
b/regressiontests/stable-domainmodel/src/test/resources/metamodel.xml.zip differ
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport.java
index b04cc76064..4a021a3304 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport.java
@@ -49,7 +49,48 @@ import lombok.Setter;
  * named           x    x    x
  * validate        x         x   x
  * </pre>
- *
+ * <p>
+ * Mixins
+ * <table><tr>
+ * <td><b>action1</b></td><td>
+ * testing whether member-support methods are picked up when prefixed by 
param-index,
+ * also testing proper mixed-in member-id, also friendly name and description 
from annotation
+ * </td></tr><tr>
+ * <td><b>action2</b></td><td>
+ * testing whether has PublishedActionFacet
+ * </td></tr><tr>
+ * <td><b>action3</b></td><td>
+ * testing whether member-support methods are picked up when prefixed by 
param-index,
+ * (variant of action1, with non-scalar action parameters)
+ * </td></tr><tr>
+ * <td><b>action4</b></td><td>
+ * testing whether member-support methods are picked up when prefixed by 
param-name
+ * </td></tr><tr>
+ * <td><b>action5</b></td><td>
+ * testing imperative naming and describing
+ * </td></tr><tr>
+ * <td><b>action6</b></td><td>
+ * testing annotation provided 'choicesFrom' fallback,
+ * if no explicit choices member-support is given
+ * </td></tr><tr>
+ * <td><b>property1</b></td><td>
+ * analog to action1
+ * </td></tr><tr>
+ * <td><b>property2</b></td><td>
+ * analog to action2
+ * </td></tr><tr>
+ * <td><b>property3</b></td><td>
+ * testing imperative naming and describing
+ * </td></tr><tr>
+ * <td><b>collection1</b></td><td>
+ * analog to action1
+ * </td></tr><tr>
+ * <td><b>collection2</b></td><td>
+ * analog to action2
+ * </td></tr><tr>
+ * <td><b>collection3</b></td><td>
+ * testing imperative naming and describing
+ * </td></tr></table>
  */
 @DomainObject(nature = Nature.VIEW_MODEL)
 public class ProperMemberSupport {
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action1.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action1.java
index 49e225399a..4ea7a371a9 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action1.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action1.java
@@ -28,6 +28,9 @@ import org.apache.isis.applib.annotation.Publishing;
 
 import lombok.RequiredArgsConstructor;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Action(executionPublishing = Publishing.ENABLED)
 @ActionLayout(named = "foo", describedAs = "bar")
 @RequiredArgsConstructor
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action2.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action2.java
index 7f56665d5b..97c85cda3b 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action2.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action2.java
@@ -27,6 +27,9 @@ import org.apache.isis.applib.annotation.Publishing;
 
 import lombok.RequiredArgsConstructor;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Action(executionPublishing = Publishing.ENABLED)
 @RequiredArgsConstructor
 public class ProperMemberSupport_action2 {
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action3.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action3.java
index 6e5994e5e1..615fe419d3 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action3.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action3.java
@@ -28,6 +28,9 @@ import org.apache.isis.applib.annotation.Publishing;
 
 import lombok.RequiredArgsConstructor;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Action(executionPublishing = Publishing.ENABLED)
 @RequiredArgsConstructor
 public class ProperMemberSupport_action3 {
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action4.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action4.java
index 497569c795..9529e3c1fa 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action4.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action4.java
@@ -30,6 +30,9 @@ import lombok.RequiredArgsConstructor;
 import lombok.Value;
 import lombok.experimental.Accessors;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Action(executionPublishing = Publishing.ENABLED)
 @RequiredArgsConstructor
 public class ProperMemberSupport_action4 {
@@ -44,7 +47,7 @@ public class ProperMemberSupport_action4 {
 
     // proper mix-in action
 
-    public ProperMemberSupport act(List<String> a, List<String> b) {
+    public ProperMemberSupport act(final List<String> a, final List<String> b) 
{
         return holder;
     }
 
@@ -56,39 +59,39 @@ public class ProperMemberSupport_action4 {
         return false;
     }
 
-    @MemberSupport public String validateAct(Parameter params) {
+    @MemberSupport public String validateAct(final Parameter params) {
         return null;
     }
 
-    @MemberSupport public Set<String> autoCompleteA(Parameter params, 
@MinLength(3) String search) {
+    @MemberSupport public Set<String> autoCompleteA(final Parameter params, 
@MinLength(3) final String search) {
         return null;
     }
 
-    @MemberSupport public Set<String> autoCompleteB(Parameter params, 
@MinLength(3) String search) {
+    @MemberSupport public Set<String> autoCompleteB(final Parameter params, 
@MinLength(3) final String search) {
         return null;
     }
 
-    @MemberSupport public Set<String> choicesA(Parameter params) {
+    @MemberSupport public Set<String> choicesA(final Parameter params) {
         return null;
     }
 
-    @MemberSupport public Set<String> choicesB(Parameter params) {
+    @MemberSupport public Set<String> choicesB(final Parameter params) {
         return null;
     }
 
-    @MemberSupport public List<String> defaultA(Parameter params) {
+    @MemberSupport public List<String> defaultA(final Parameter params) {
         return null;
     }
 
-    @MemberSupport public List<String> defaultB(Parameter params) {
+    @MemberSupport public List<String> defaultB(final Parameter params) {
         return null;
     }
 
-    @MemberSupport public String validateA(Parameter params) {
+    @MemberSupport public String validateA(final Parameter params) {
         return null;
     }
 
-    @MemberSupport public String validateB(Parameter params) {
+    @MemberSupport public String validateB(final Parameter params) {
         return null;
     }
 
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action5.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action5.java
index 925c15291e..4f0a9da3c0 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action5.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action5.java
@@ -24,6 +24,9 @@ import org.apache.isis.applib.annotation.Publishing;
 
 import lombok.RequiredArgsConstructor;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Action(executionPublishing = Publishing.ENABLED)
 @RequiredArgsConstructor
 public class ProperMemberSupport_action5 {
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action5.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action6.java
similarity index 71%
copy from 
regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action5.java
copy to 
regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action6.java
index 925c15291e..bea2d14aec 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action5.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_action6.java
@@ -20,29 +20,23 @@ package org.apache.isis.testdomain.model.good;
 
 import org.apache.isis.applib.annotation.Action;
 import org.apache.isis.applib.annotation.MemberSupport;
-import org.apache.isis.applib.annotation.Publishing;
 
 import lombok.RequiredArgsConstructor;
 
-@Action(executionPublishing = Publishing.ENABLED)
+/**
+ * Testing annotation provided 'choicesFrom' fallback,
+ * if no explicit choices member-support is given.
+ * <p>
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
+@Action(choicesFrom = "myColl")
 @RequiredArgsConstructor
-public class ProperMemberSupport_action5 {
+public class ProperMemberSupport_action6 {
 
     private final ProperMemberSupport mixee;
 
-    @MemberSupport public ProperMemberSupport act() {
+    @MemberSupport public ProperMemberSupport act(final String p0) {
         return mixee;
     }
 
-    // -- IMPERATIVE NAMING AND DESCRIBING
-
-    @MemberSupport public String namedAct() {
-        return "named-imperative[action5]";
-    }
-
-    @MemberSupport public String describedAct() {
-        return "described-imperative[action5]";
-    }
-
-
 }
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection1.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection1.java
index 083bbd3371..d59a3ca26f 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection1.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection1.java
@@ -27,6 +27,9 @@ import org.apache.isis.applib.annotation.MemberSupport;
 
 import lombok.RequiredArgsConstructor;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Collection
 @CollectionLayout(named = "foo", describedAs = "bar")
 @RequiredArgsConstructor
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection2.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection2.java
index 44e7080df6..2e59c1af44 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection2.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection2.java
@@ -27,6 +27,9 @@ import org.apache.isis.applib.annotation.MemberSupport;
 
 import lombok.RequiredArgsConstructor;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Collection
 @CollectionLayout(named = "foo", describedAs = "bar")
 @RequiredArgsConstructor
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection3.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection3.java
index 09f3991391..87ceb0beaf 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection3.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_collection3.java
@@ -27,6 +27,9 @@ import org.apache.isis.applib.annotation.MemberSupport;
 
 import lombok.RequiredArgsConstructor;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Collection
 @CollectionLayout(named = "foo", describedAs = "bar") // <-- overruled by 
imperative naming
 @RequiredArgsConstructor
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property1.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property1.java
index 0a5c315365..da4b46479c 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property1.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property1.java
@@ -27,6 +27,9 @@ import org.apache.isis.applib.annotation.PropertyLayout;
 
 import lombok.RequiredArgsConstructor;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Property
 @PropertyLayout(named = "foo", describedAs = "bar")
 @RequiredArgsConstructor
@@ -42,7 +45,7 @@ public class ProperMemberSupport_property1 {
 
     // -- PROPERLY DECLARED SUPPORTING METHODS
 
-    @MemberSupport public Set<String> autoCompleteProp(@MinLength(3) String 
search) {
+    @MemberSupport public Set<String> autoCompleteProp(@MinLength(3) final 
String search) {
         return null;
     }
 
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property2.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property2.java
index 2e3c180ac1..68a2adf813 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property2.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property2.java
@@ -27,6 +27,9 @@ import org.apache.isis.applib.annotation.PropertyLayout;
 
 import lombok.RequiredArgsConstructor;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Property
 @PropertyLayout(named= "foo", describedAs = "bar")
 @RequiredArgsConstructor
@@ -43,7 +46,7 @@ public class ProperMemberSupport_property2 {
 
     // -- PROPERLY DECLARED SUPPORTING METHODS
 
-    @MemberSupport public Set<String> autoCompleteProp(@MinLength(3) String 
search) {
+    @MemberSupport public Set<String> autoCompleteProp(@MinLength(3) final 
String search) {
         return null;
     }
 
diff --git 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property3.java
 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property3.java
index ca0cd82fb2..d0ada7c6ed 100644
--- 
a/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property3.java
+++ 
b/regressiontests/stable/src/main/java/org/apache/isis/testdomain/model/good/ProperMemberSupport_property3.java
@@ -24,6 +24,9 @@ import org.apache.isis.applib.annotation.PropertyLayout;
 
 import lombok.RequiredArgsConstructor;
 
+/**
+ * For (test) mixin descriptions see {@link ProperMemberSupport}.
+ */
 @Property
 @PropertyLayout(named= "foo", describedAs = "bar") // <-- overruled by 
imperative naming, but used for column naming and describing
 @RequiredArgsConstructor

Reply via email to