Reduce RSpec/ExampleLength in CSP request spec

[mjankowski]

Trying to nudge some of the various RSpec/* cops overriden max values down a number or two if we can.

In this one:

• Move the random stub to a before
• Move the expected value to helper method
• Slightly simplify how we are massaging the strings on both sides

[ClearlyClaire]

This works, but I don't think the order of clauses is something we should be testing for, I'm afraid this might be too specific and might need changes if something in Rails' internals changes.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.01%. Comparing base (86500e3) to head (73584dc).
Report is 241 commits behind head on main.

❗ Current head 73584dc differs from pull request most recent head 91f63c3. Consider uploading reports for the commit 91f63c3 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #29104   +/-   ##
=======================================
  Coverage   85.01%   85.01%           
=======================================
  Files        1059     1059           
  Lines       28277    28293   +16     
  Branches     4538     4535    -3     
=======================================
+ Hits        24040    24054   +14     
- Misses       3074     3075    +1     
- Partials     1163     1164    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mjankowski]

Good point, we care about the elements but not the order. Updated to keep the method extraction, but revert back to a looser element comparison instead of the exact string comparison the first version here had.