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

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


The following commit(s) were added to refs/heads/master by this push:
     new a758aeee6 Test modernization, clean up utility classes.
a758aeee6 is described below

commit a758aeee66703fce5253d73047fa1ec9903398b3
Author: James Bognar <[email protected]>
AuthorDate: Mon Aug 25 14:41:02 2025 -0400

    Test modernization, clean up utility classes.
---
 .../org/apache/juneau/bean/swagger/ui/SwaggerUI.java     |  2 +-
 .../org/apache/juneau/config/internal/ConfigMap.java     |  2 +-
 .../src/main/java/org/apache/juneau/BeanFilter.java      | 16 ++++++++--------
 .../java/org/apache/juneau/html/HtmlParserSession.java   |  2 +-
 .../org/apache/juneau/html/HtmlSerializerSession.java    |  2 +-
 .../java/org/apache/juneau/internal/CollectionUtils.java |  6 +++---
 .../java/org/apache/juneau/internal/DelegateBeanMap.java |  4 ++--
 .../java/org/apache/juneau/jsonschema/SchemaUtils.java   |  2 +-
 .../java/org/apache/juneau/jsonschema/TypeCategory.java  |  2 +-
 .../main/java/org/apache/juneau/reflect/ClassInfo.java   |  2 +-
 .../java/org/apache/juneau/serializer/SerializerSet.java |  2 +-
 .../src/main/java/org/apache/juneau/utils/BeanDiff.java  |  4 ++--
 .../main/java/org/apache/juneau/rest/RestOpContext.java  |  2 +-
 .../main/java/org/apache/juneau/rest/RestOperations.java |  2 +-
 .../juneau/rest/swagger/BasicSwaggerProviderSession.java |  2 +-
 15 files changed, 26 insertions(+), 26 deletions(-)

diff --git 
a/juneau-bean/juneau-bean-openapi3/src/main/java/org/apache/juneau/bean/swagger/ui/SwaggerUI.java
 
b/juneau-bean/juneau-bean-openapi3/src/main/java/org/apache/juneau/bean/swagger/ui/SwaggerUI.java
index d07a706b7..b3994b4eb 100644
--- 
a/juneau-bean/juneau-bean-openapi3/src/main/java/org/apache/juneau/bean/swagger/ui/SwaggerUI.java
+++ 
b/juneau-bean/juneau-bean-openapi3/src/main/java/org/apache/juneau/bean/swagger/ui/SwaggerUI.java
@@ -42,7 +42,7 @@ public class SwaggerUI extends ObjectSwap<Swagger,Div> {
                
.caching(Boolean.getBoolean("RestContext.disableClasspathResourceCaching.b") ? 
-1 : 1_000_000)
                .build();
 
-       private static final Set<String> STANDARD_METHODS = set("get", "put", 
"post", "delete", "options");
+       private static final Set<String> STANDARD_METHODS = set2("get", "put", 
"post", "delete", "options");
 
        /**
         * This UI applies to HTML requests only.
diff --git 
a/juneau-core/juneau-config/src/main/java/org/apache/juneau/config/internal/ConfigMap.java
 
b/juneau-core/juneau-config/src/main/java/org/apache/juneau/config/internal/ConfigMap.java
index f664c28ae..68b2a81ec 100644
--- 
a/juneau-core/juneau-config/src/main/java/org/apache/juneau/config/internal/ConfigMap.java
+++ 
b/juneau-core/juneau-config/src/main/java/org/apache/juneau/config/internal/ConfigMap.java
@@ -283,7 +283,7 @@ public class ConfigMap implements ConfigStoreListener {
        public Set<String> getKeys(String section) {
                checkSectionName(section);
                var cs = entries.get(section);
-               var s = imports.isEmpty() && cs != null ? cs.entries.keySet() : 
CollectionUtils.<String>set();
+               var s = imports.isEmpty() && cs != null ? cs.entries.keySet() : 
CollectionUtils.<String>set2();
                if (! imports.isEmpty()) {
                        imports.forEach(x -> 
s.addAll(x.getConfigMap().getKeys(section)));
                        if (cs != null)
diff --git 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/BeanFilter.java 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/BeanFilter.java
index 9f784e65f..41a59cd89 100644
--- 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/BeanFilter.java
+++ 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/BeanFilter.java
@@ -66,10 +66,10 @@ public final class BeanFilter {
                Class<?> beanClass;
                String typeName, example;
                Set<String>
-                       properties = set(),
-                       excludeProperties = set(),
-                       readOnlyProperties = set(),
-                       writeOnlyProperties = set();
+                       properties = set2(),
+                       excludeProperties = set2(),
+                       readOnlyProperties = set2(),
+                       writeOnlyProperties = set2();
                Class<?> implClass, interfaceClass, stopClass;
                boolean sortProperties, fluentSetters;
                BeanCreator<PropertyNamer> propertyNamer = 
BeanCreator.of(PropertyNamer.class);
@@ -450,7 +450,7 @@ public final class BeanFilter {
                 * @return This object.
                 */
                public Builder properties(String...value) {
-                       this.properties = set();
+                       this.properties = set2();
                        for (String v : value)
                                split3(v, x -> properties.add(x));
                        return this;
@@ -494,7 +494,7 @@ public final class BeanFilter {
                 * @return This object.
                 */
                public Builder excludeProperties(String...value) {
-                       this.excludeProperties = set();
+                       this.excludeProperties = set2();
                        for (String v : value)
                                split3(v, x -> excludeProperties.add(x));
                        return this;
@@ -540,7 +540,7 @@ public final class BeanFilter {
                 * @return This object.
                 */
                public Builder readOnlyProperties(String...value) {
-                       this.readOnlyProperties = set();
+                       this.readOnlyProperties = set2();
                        for (String v : value)
                                split3(v, x -> readOnlyProperties.add(x));
                        return this;
@@ -586,7 +586,7 @@ public final class BeanFilter {
                 * @return This object.
                 */
                public Builder writeOnlyProperties(String...value) {
-                       this.writeOnlyProperties = set();
+                       this.writeOnlyProperties = set2();
                        for (String v : value)
                                split3(v, x -> writeOnlyProperties.add(x));
                        return this;
diff --git 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/HtmlParserSession.java
 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/HtmlParserSession.java
index 841e550c5..4080e4e8a 100644
--- 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/HtmlParserSession.java
+++ 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/HtmlParserSession.java
@@ -53,7 +53,7 @@ public final class HtmlParserSession extends XmlParserSession 
{
        // Static
        
//-------------------------------------------------------------------------------------------------------------------
 
-       private static final Set<String> whitespaceElements = 
set("br","bs","sp","ff");
+       private static final Set<String> whitespaceElements = 
set2("br","bs","sp","ff");
 
        /**
         * Creates a new builder for this object.
diff --git 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/HtmlSerializerSession.java
 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/HtmlSerializerSession.java
index 25446fc0c..06e561a73 100644
--- 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/HtmlSerializerSession.java
+++ 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/html/HtmlSerializerSession.java
@@ -912,7 +912,7 @@ public class HtmlSerializerSession extends 
XmlSerializerSession {
                // If it's a non-bean map, only use table if all entries are 
also maps.
                if (cm1.isMap() && ! cm1.isBeanMap()) {
 
-                       Set<Object> set = CollectionUtils.set();
+                       Set<Object> set = CollectionUtils.set2();
                        for (Object o : c) {
                                o = swap(swap, o);
                                if (! canIgnoreValue(cm1, null, o)) {
diff --git 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/internal/CollectionUtils.java
 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/internal/CollectionUtils.java
index 97ea1dc34..803cd3e43 100644
--- 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/internal/CollectionUtils.java
+++ 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/internal/CollectionUtils.java
@@ -225,7 +225,7 @@ public final class CollectionUtils {
         * @return A new modifiable set.
         */
        @SafeVarargs
-       public static <E> LinkedHashSet<E> set(E...values) {
+       public static <E> LinkedHashSet<E> set2(E...values) {
                LinkedHashSet<E> l = new LinkedHashSet<>();
                for (E v : values)
                        l.add(v);
@@ -242,7 +242,7 @@ public final class CollectionUtils {
         */
        @SafeVarargs
        public static <E> LinkedHashSet<E> setOf(Class<E> elementType, 
E...values) {
-               return set(values);
+               return set2(values);
        }
 
        /**
@@ -546,7 +546,7 @@ public final class CollectionUtils {
        public static <E> Set<E> addAll(Set<E> value, E...entries) {
                if (entries != null) {
                        if (value == null)
-                               value = set(entries);
+                               value = set2(entries);
                        else
                                Collections.addAll(value, entries);
                }
diff --git 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/internal/DelegateBeanMap.java
 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/internal/DelegateBeanMap.java
index 756207ffa..2d5a1160c 100644
--- 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/internal/DelegateBeanMap.java
+++ 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/internal/DelegateBeanMap.java
@@ -35,7 +35,7 @@ import org.apache.juneau.collections.*;
  */
 public class DelegateBeanMap<T> extends BeanMap<T> {
 
-       private Set<String> keys = set();
+       private Set<String> keys = set2();
        private JsonMap overrideValues = new JsonMap();
 
        /**
@@ -106,7 +106,7 @@ public class DelegateBeanMap<T> extends BeanMap<T> {
 
        @Override /* Map */
        public synchronized Set<Entry<String,Object>> entrySet() {
-               Set<Entry<String,Object>> s = set();
+               Set<Entry<String,Object>> s = set2();
                keys.forEach(k -> {
                        BeanMapEntry bme;
                        if (overrideValues.containsKey(k))
diff --git 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/jsonschema/SchemaUtils.java
 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/jsonschema/SchemaUtils.java
index b66a9019a..d5bb434bc 100644
--- 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/jsonschema/SchemaUtils.java
+++ 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/jsonschema/SchemaUtils.java
@@ -89,7 +89,7 @@ public class SchemaUtils {
                String s = joinnl(ss);
                if (s.isEmpty())
                        return null;
-               Set<String> set = set();
+               Set<String> set = set2();
                JsonList.ofJsonOrCdl(s).forEach(x -> set.add(x.toString()));
                return set;
        }
diff --git 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/jsonschema/TypeCategory.java
 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/jsonschema/TypeCategory.java
index 9a17a5b2a..7e4e81046 100644
--- 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/jsonschema/TypeCategory.java
+++ 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/jsonschema/TypeCategory.java
@@ -69,7 +69,7 @@ public enum TypeCategory {
        public static Set<TypeCategory> parse(String s) {
                if (s == null || s.isEmpty())
                        return Collections.emptySet();
-               Set<TypeCategory> set = set();
+               Set<TypeCategory> set = set2();
                split3(s, x -> set.add(valueOf(x.toUpperCase())));
                return set;
        }
diff --git 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/reflect/ClassInfo.java
 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/reflect/ClassInfo.java
index 3f4fc1911..e386055dd 100644
--- 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/reflect/ClassInfo.java
+++ 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/reflect/ClassInfo.java
@@ -364,7 +364,7 @@ public final class ClassInfo {
        ClassInfo[] _getInterfaces() {
                if (interfaces == null) {
                        synchronized(this) {
-                               Set<ClassInfo> s = set();
+                               Set<ClassInfo> s = set2();
                                for (ClassInfo ci : _getParents())
                                        for (ClassInfo ci2 : 
ci._getDeclaredInterfaces()) {
                                                s.add(ci2);
diff --git 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/serializer/SerializerSet.java
 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/serializer/SerializerSet.java
index e21dbb2d4..abdd5638d 100644
--- 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/serializer/SerializerSet.java
+++ 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/serializer/SerializerSet.java
@@ -499,7 +499,7 @@ public final class SerializerSet {
                this.entriesList = Utils.ulist2(entries);
 
                List<MediaRange> lmtr = list2();
-               Set<MediaType> lmt = set();
+               Set<MediaType> lmt = set2();
                List<Serializer> l = list2();
                for (Serializer e : entries) {
                        e.getMediaTypeRanges().forEachRange(x -> {
diff --git 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/utils/BeanDiff.java
 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/utils/BeanDiff.java
index f5dd2fd2b..63ee2967e 100644
--- 
a/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/utils/BeanDiff.java
+++ 
b/juneau-core/juneau-marshall/src/main/java/org/apache/juneau/utils/BeanDiff.java
@@ -125,7 +125,7 @@ public class BeanDiff {
                 * @return This object.
                 */
                public Builder<T> include(String...properties) {
-                       include = set(properties);
+                       include = set2(properties);
                        return this;
                }
 
@@ -150,7 +150,7 @@ public class BeanDiff {
                 * @return This object.
                 */
                public Builder<T> exclude(String...properties) {
-                       exclude = set(properties);
+                       exclude = set2(properties);
                        return this;
                }
 
diff --git 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/RestOpContext.java
 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/RestOpContext.java
index 8afae0fbb..4cef4f937 100644
--- 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/RestOpContext.java
+++ 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/RestOpContext.java
@@ -1100,7 +1100,7 @@ public class RestOpContext extends Context implements 
Comparable<RestOpContext>
 
                RestGuardList getGuards() {
                        RestGuardList.Builder b = guards();
-                       Set<String> roleGuard = 
Utils.opt(this.roleGuard).orElseGet(CollectionUtils::set);
+                       Set<String> roleGuard = 
Utils.opt(this.roleGuard).orElseGet(CollectionUtils::set2);
 
                        for (String rg : roleGuard) {
                                try {
diff --git 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/RestOperations.java
 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/RestOperations.java
index 956466402..b6ea34a2b 100644
--- 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/RestOperations.java
+++ 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/RestOperations.java
@@ -77,7 +77,7 @@ public class RestOperations {
                protected Builder(BeanStore beanStore) {
                        super(RestOperations.class, beanStore);
                        map = new TreeMap<>();
-                       set = set();
+                       set = set2();
                }
 
                @Override /* BeanBuilder */
diff --git 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/swagger/BasicSwaggerProviderSession.java
 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/swagger/BasicSwaggerProviderSession.java
index c1891e476..562301868 100644
--- 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/swagger/BasicSwaggerProviderSession.java
+++ 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/swagger/BasicSwaggerProviderSession.java
@@ -1126,7 +1126,7 @@ public class BasicSwaggerProviderSession {
        private Set<String> toSet(String[] ss) {
                if (ss.length == 0)
                        return null;
-               Set<String> set = set();
+               Set<String> set = set2();
                for (String s : ss)
                        split3(s, x -> set.add(x));
                return set.isEmpty() ? null : set;

Reply via email to