Minor corrections

RIPAGlobal · Jun 24, 2024 · 69aff9b · 69aff9b
1 parent 96673f0
commit 69aff9b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/app/controllers/scimitar/schemas_controller.rb b/app/controllers/scimitar/schemas_controller.rb
@@ -20,7 +20,7 @@ def index
 
       # Now we either have a simple render method, or a complex one.
       #
-      schema_to_render = if Scimitar.engine_configuration.schema_list_from_attribute_mappings.empty?
+      schemas_to_render = if Scimitar.engine_configuration.schema_list_from_attribute_mappings.empty?
         list
       else
         self.redraw_schema_list_using_mappings(list)
@@ -30,10 +30,10 @@ def index
         schemas: [
             'urn:ietf:params:scim:api:messages:2.0:ListResponse'
         ],
-        totalResults: schema_to_render.size,
+        totalResults: schemas_to_render.size,
         startIndex:   1,
-        itemsPerPage: schema_to_render.size,
-        Resources:    schema_to_render
+        itemsPerPage: schemas_to_render.size,
+        Resources:    schemas_to_render
       })
     end
 

diff --git a/spec/apps/dummy/app/models/mock_user.rb b/spec/apps/dummy/app/models/mock_user.rb
@@ -82,8 +82,11 @@ def self.scim_attributes_map
           }
         },
       ],
-      groups: [ # NB read-only, so no :find_with key
+      groups: [
         {
+          # Read-only, so no :find_with key. There's no 'class' specified here
+          # either, to help test the "/Schemas" endpoint's reflection code.
+          #
           list:  :mock_groups,
           using: {
             value:   :id,