Skip to content

test(backend): WebMvcTests for six controllers - #162

Merged
CMaintz merged 4 commits into
mainfrom
test/webmvc-controllers
Oct 1, 2026
Merged

CMaintz merged 4 commits into
mainfrom
test/webmvc-controllers

Conversation

@CMaintz

@CMaintz CMaintz commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

59 tests for the Job, Company, User, Skill, ProfileSection and InterviewPrep controllers, same setup as ApplicationControllerTest. 595 tests total, all passing locally. Stacked on #157.

Two things I noticed but didn't fix here:

  • Outreach "not found" returns 400, while the OpenAPI docs say 404.
  • Framework errors (missing param, bad enum, malformed JSON) hit the catch-all handler and return 500. The tests don't lock that in, so fixing it later won't break anything.

…ofile-section and interview-prep controllers
@CMaintz CMaintz changed the title test(backend): @WebMvcTest coverage for six core controllers test(backend): WebMvcTests for six controllers Sep 30, 2026
Base automatically changed from fix/jackson-cve to main September 30, 2026 14:59
@CMaintz
CMaintz merged commit 8ae83da into main Oct 1, 2026
23 checks passed
@CMaintz
CMaintz deleted the test/webmvc-controllers branch October 1, 2026 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant