OutOfRangeException on integration tests

[nickvergessen]

How to use GitHub

• Please use the 👍 reaction to show that you are affected by the same issue.
• Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
• Subscribe to receive notifications on status change and new comments.

---

Combination of

• fix(API): Add more specific int-types #10857 and
• Enable AppFramework dispatcher to enforce integer ranges server#41578 results in
• https://github.com/nextcloud/spreed/actions/runs/7007500662/job/19063273322?pr=11014#step:14:1394

Potential solution

• Middleware that catches OutOfRangeException and converts it to a 400 status code, so the API doesn't break

{
  "Exception": "OutOfRangeException",
  "Message": "Parameter amount must be between 1 and 20",
  "Code": 0,
  "Trace": [
    {
      "file": "/home/runner/actions-runner/_work/spreed/spreed/lib/private/AppFramework/Http/Dispatcher.php",
      "line": 223,
      "function": "ensureParameterValueSatisfiesRange",
      "class": "OC\\AppFramework\\Http\\Dispatcher",
      "type": "->",
      "args": [
        "amount",
        21
      ]
    },
    {
      "file": "/home/runner/actions-runner/_work/spreed/spreed/lib/private/AppFramework/Http/Dispatcher.php",
      "line": 138,
      "function": "executeController",
      "class": "OC\\AppFramework\\Http\\Dispatcher",
      "type": "->",
      "args": [
        [
          "OCA\\Talk\\Controller\\BreakoutRoomController"
        ],
        "configureBreakoutRooms"
      ]
    },
    {
      "file": "/home/runner/actions-runner/_work/spreed/spreed/lib/private/AppFramework/App.php",
      "line": 184,
      "function": "dispatch",
      "class": "OC\\AppFramework\\Http\\Dispatcher",
      "type": "->",
      "args": [
        [
          "OCA\\Talk\\Controller\\BreakoutRoomController"
        ],
        "configureBreakoutRooms"
      ]
    },
    {
      "file": "/home/runner/actions-runner/_work/spreed/spreed/lib/private/Route/Router.php",
      "line": 315,
      "function": "main",
      "class": "OC\\AppFramework\\App",
      "type": "::",
      "args": [
        "OCA\\Talk\\Controller\\BreakoutRoomController",
        "configureBreakoutRooms",
        [
          "OC\\AppFramework\\DependencyInjection\\DIContainer"
        ],
        [
          "v1",
          "snac2av7",
          "ocs.spreed.BreakoutRoom.configureBreakoutRooms"
        ]
      ]
    },
    {
      "file": "/home/runner/actions-runner/_work/spreed/spreed/ocs/v1.php",
      "line": 64,
      "function": "match",
      "class": "OC\\Route\\Router",
      "type": "->",
      "args": [
        "/ocsapp/apps/spreed/api/v1/breakout-rooms/snac2av7"
      ]
    },
    {
      "file": "/home/runner/actions-runner/_work/spreed/spreed/ocs/v2.php",
      "line": 23,
      "args": [
        "/home/runner/actions-runner/_work/spreed/spreed/ocs/v1.php"
      ],
      "function": "require_once"
    }
  ],
  "File": "/home/runner/actions-runner/_work/spreed/spreed/lib/private/AppFramework/Http/Dispatcher.php",
  "Line": 264,
  "CustomMessage": "Exception thrown: OutOfRangeException"
}