person/notifications: respect show_maximum param

- Implement  for /my/notifications.
- Add specs for default behavior, limited results and capping at Notification.max_per_page.
- Defensive filter handling and improved comments.

Fixes #18910.

Author: geetxnshgoyal

.circleci/conditional_config.yml

@@ -71,6 +71,7 @@ aliases:
       bundle config build.nokogiri --use-system-libraries
       bundle config build.sassc --disable-march-tune-native
       bundle config build.nio4r --with-cflags='-Wno-return-type'
+      bundle config build.xmlhash --with-cflags='-Wno-error=shorten-64-to-32'
       bundle config set --local path 'vendor/bundle'
       bundle install --jobs=4 --retry=3 --local
 

.ruby-version

@@ -1 +1 @@
-3.4.2
+3.2.2

.vscode/c_cpp_properties.json

@@ -0,0 +1,17 @@
+{
+    "configurations": [
+        {
+            "name": "Mac",
+            "includePath": [
+                "${workspaceFolder}/**",
+                "/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks/Ruby.framework/Versions/2.6/Headers"
+            ],
+            "defines": [],
+            "compilerPath": "/usr/bin/clang",
+            "cStandard": "c17",
+            "cppStandard": "c++14",
+            "intelliSenseMode": "macos-clang-arm64"
+        }
+    ],
+    "version": 4
+}

src/api/.bundle/config

@@ -0,0 +1,2 @@
+---
+BUNDLE_BUILD__XMLHASH: "--with-cflags='-Wno-error=shorten-64-to-32'"

src/api/Gemfile.lock

@@ -414,7 +414,7 @@ GEM
       zeitwerk (~> 2.6)
     rainbow (3.1.1)
     rake (13.2.1)
-    rantly (3.0.0)
+    rantly (2.0.0)
     rbtree (0.4.6)
     rbtree3 (0.7.1)
     rdoc (6.16.0)

src/api/spec/controllers/person/notifications_controller_spec.rb

@@ -24,20 +24,33 @@
 
   describe 'index' do
     context 'called by authorized user' do
-      let!(:notifications) { create_list(:notification_for_request, 2, :web_notification, :request_state_change, subscriber: user) }
+      let!(:notifications) { create_list(:notification_for_request, 5, :web_notification, :request_state_change, subscriber: user) }
 
       before do
         login user
-        get :index, format: :xml
-
         notifications.each do |notification|
           notification.projects << user.home_project
           notification.save
         end
       end
 
-      it { expect(response).to have_http_status(:success) }
-      it { expect(response.body).to include('<notifications count="2">') }
+      it 'returns all notifications by default' do
+        get :index, format: :xml
+        expect(response).to have_http_status(:success)
+        expect(response.body).to include('<notifications count="5">')
+      end
+
+      it 'returns limited notifications when show_maximum is set' do
+        get :index, params: { format: :xml, show_maximum: 3 }
+        expect(response).to have_http_status(:success)
+        expect(response.body).to include('<notifications count="3">')
+      end
+
+      it 'caps show_maximum at Notification.max_per_page' do
+        get :index, params: { format: :xml, show_maximum: 999 }
+        expect(response).to have_http_status(:success)
+        expect(response.body).to include("<notifications count=\"#{Notification.max_per_page}\">")
+      end
 
       context 'filter by kind' do
         let!(:notifications) { create_list(:notification_for_request, 2, :web_notification, :request_state_change, subscriber: user, delivered: true) }