Reformat remaining JSON strings in RestControllerTest to avoid long lines #1282

osma · 2022-03-02T11:44:42Z

Reasons for creating this PR

While working on a fix to #1254, I found out that some tests in RestControllerTest contain very long JSON strings on a single line. I need to change some of those and it's very difficult. So I decided to reformat first using this separate PR.

Link to relevant issue(s), if any

n/a

Description of the changes in this PR

Reformat test cases in RestControllerTest which contain long JSON strings on a single line. For some of the tests (the bottom ones) this has already been done in the past, this reformats also the remaining ones.

No functional changes.

Known problems or uncertainties in this PR

n/a

Checklist

phpUnit tests pass locally with my changes
I have added tests that prove my fix is effective or that my feature works (if not, explain why below)
The PR doesn't introduce unintended code changes (e.g. empty lines or useless reindentation)

…ines

codecov · 2022-03-02T11:45:49Z

Codecov Report

Merging #1282 (5bd648d) into master (da1716c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1282   +/-   ##
=========================================
  Coverage     69.36%   69.36%           
  Complexity     1649     1649           
=========================================
  Files            32       32           
  Lines          4047     4047           
=========================================
  Hits           2807     2807           
  Misses         1240     1240

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da1716c...5bd648d. Read the comment docs.