github-advanced-security[bot] commented on code in PR #702:
URL: https://github.com/apache/syncope/pull/702#discussion_r1587329153


##########
wa/starter/src/test/java/org/apache/syncope/wa/starter/AbstractTest.java:
##########
@@ -58,6 +80,136 @@
         public SyncopeCoreTestingServer syncopeCoreTestingServer() {
             return new SyncopeCoreTestingServer();
         }
+
+        // This bean definition allows MacOS builds to complete successfully
+        @Bean(name = AttributeDefinitionStore.BEAN_NAME)
+        @RefreshScope(proxyMode = ScopedProxyMode.DEFAULT)
+        public AttributeDefinitionStore attributeDefinitionStore() {
+            return new AttributeDefinitionStore() {
+
+                @Override
+                public AttributeDefinitionStore 
registerAttributeDefinition(final AttributeDefinition defn) {
+                    return this;
+                }
+
+                @Override
+                public AttributeDefinitionStore registerAttributeDefinition(
+                        final String key, final AttributeDefinition defn) {
+
+                    return this;
+                }
+
+                @Override
+                public Optional<AttributeDefinition> 
locateAttributeDefinitionByName(final String name) {
+                    return Optional.empty();
+                }
+
+                @Override
+                public AttributeDefinitionStore 
removeAttributeDefinition(final String key) {
+                    return this;
+                }
+
+                @Override
+                public Optional<AttributeDefinition> 
locateAttributeDefinition(final String key) {
+                    return Optional.empty();
+                }
+
+                @Override
+                public <T extends AttributeDefinition> Optional<T> 
locateAttributeDefinition(
+                        final String key, final Class<T> clazz) {
+
+                    return Optional.empty();
+                }
+
+                @Override
+                public <T extends AttributeDefinition> Optional<T> 
locateAttributeDefinition(
+                        final Predicate<AttributeDefinition> predicate) {
+
+                    return Optional.empty();
+                }
+
+                @Override
+                public Collection<AttributeDefinition> 
getAttributeDefinitions() {
+                    return Set.of();
+                }
+
+                @Override
+                public <T extends AttributeDefinition> Stream<T> 
getAttributeDefinitionsBy(final Class<T> type) {
+                    return Stream.empty();
+                }
+
+                @Override
+                public Optional<Pair<AttributeDefinition, List<Object>>> 
resolveAttributeValues(
+                        final String key, final 
AttributeDefinitionResolutionContext context) {
+
+                    return Optional.empty();
+                }
+
+                @Override
+                public Map<String, List<Object>> resolveAttributeValues(
+                        final Collection<String> attributeDefinitions,
+                        final Map<String, List<Object>> availableAttributes,
+                        final Principal principal,
+                        final RegisteredService registeredService,
+                        final Service service) {
+
+                    return Map.of();
+                }
+
+                @Override
+                public boolean isEmpty() {
+                    return true;
+                }
+
+                @Override
+                public AttributeDefinitionStore store(final Resource resource) 
{
+                    return this;
+                }
+
+                @Override
+                public AttributeDefinitionStore importStore(final 
AttributeDefinitionStore definitionStore) {
+                    return this;
+                }
+            };
+        }
+
+        @Bean(name = "aggregatingAttributeRepository")
+        @RefreshScope(proxyMode = ScopedProxyMode.DEFAULT)
+        public IPersonAttributeDao aggregatingAttributeRepository(
+                @Qualifier("attributeRepositoryAttributeMerger")
+                final IAttributeMerger attributeRepositoryAttributeMerger) {

Review Comment:
   ## Useless parameter
   
   The parameter 'attributeRepositoryAttributeMerger' is never used.
   
   [Show more 
details](https://github.com/apache/syncope/security/code-scanning/1583)



##########
wa/starter/src/test/java/org/apache/syncope/wa/starter/AbstractTest.java:
##########
@@ -58,6 +80,136 @@
         public SyncopeCoreTestingServer syncopeCoreTestingServer() {
             return new SyncopeCoreTestingServer();
         }
+
+        // This bean definition allows MacOS builds to complete successfully
+        @Bean(name = AttributeDefinitionStore.BEAN_NAME)
+        @RefreshScope(proxyMode = ScopedProxyMode.DEFAULT)
+        public AttributeDefinitionStore attributeDefinitionStore() {
+            return new AttributeDefinitionStore() {
+
+                @Override
+                public AttributeDefinitionStore 
registerAttributeDefinition(final AttributeDefinition defn) {
+                    return this;
+                }
+
+                @Override
+                public AttributeDefinitionStore registerAttributeDefinition(
+                        final String key, final AttributeDefinition defn) {
+
+                    return this;
+                }
+
+                @Override
+                public Optional<AttributeDefinition> 
locateAttributeDefinitionByName(final String name) {
+                    return Optional.empty();
+                }
+
+                @Override
+                public AttributeDefinitionStore 
removeAttributeDefinition(final String key) {
+                    return this;
+                }
+
+                @Override
+                public Optional<AttributeDefinition> 
locateAttributeDefinition(final String key) {
+                    return Optional.empty();
+                }
+
+                @Override
+                public <T extends AttributeDefinition> Optional<T> 
locateAttributeDefinition(
+                        final String key, final Class<T> clazz) {
+
+                    return Optional.empty();
+                }
+
+                @Override
+                public <T extends AttributeDefinition> Optional<T> 
locateAttributeDefinition(
+                        final Predicate<AttributeDefinition> predicate) {
+
+                    return Optional.empty();
+                }
+
+                @Override
+                public Collection<AttributeDefinition> 
getAttributeDefinitions() {
+                    return Set.of();
+                }
+
+                @Override
+                public <T extends AttributeDefinition> Stream<T> 
getAttributeDefinitionsBy(final Class<T> type) {
+                    return Stream.empty();
+                }
+
+                @Override
+                public Optional<Pair<AttributeDefinition, List<Object>>> 
resolveAttributeValues(
+                        final String key, final 
AttributeDefinitionResolutionContext context) {
+
+                    return Optional.empty();
+                }
+
+                @Override
+                public Map<String, List<Object>> resolveAttributeValues(
+                        final Collection<String> attributeDefinitions,
+                        final Map<String, List<Object>> availableAttributes,
+                        final Principal principal,
+                        final RegisteredService registeredService,
+                        final Service service) {
+
+                    return Map.of();
+                }
+
+                @Override
+                public boolean isEmpty() {
+                    return true;
+                }
+
+                @Override
+                public AttributeDefinitionStore store(final Resource resource) 
{
+                    return this;
+                }
+
+                @Override
+                public AttributeDefinitionStore importStore(final 
AttributeDefinitionStore definitionStore) {
+                    return this;
+                }
+            };
+        }
+
+        @Bean(name = "aggregatingAttributeRepository")
+        @RefreshScope(proxyMode = ScopedProxyMode.DEFAULT)
+        public IPersonAttributeDao aggregatingAttributeRepository(
+                @Qualifier("attributeRepositoryAttributeMerger")
+                final IAttributeMerger attributeRepositoryAttributeMerger) {
+
+            return new IPersonAttributeDao() {

Review Comment:
   ## Inconsistent compareTo
   
   This class declares [compareTo](1) but inherits equals; the two could be 
inconsistent.
   
   [Show more 
details](https://github.com/apache/syncope/security/code-scanning/1584)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@syncope.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to