feat(server): migrate pascal case http headers rule to context object to support openapi 3

[maxim-tschumak]

Related to #714

[codecov-io]

Codecov Report

Merging #820 into master will increase coverage by 0.24%.
The diff coverage is 75%.

@@             Coverage Diff             @@
##             master    #820      +/-   ##
===========================================
+ Coverage     84.86%   85.1%   +0.24%     
+ Complexity      907     902       -5     
===========================================
  Files           171     171              
  Lines          2557    2558       +1     
  Branches        393     395       +2     
===========================================
+ Hits           2170    2177       +7     
+ Misses          151     149       -2     
+ Partials        236     232       -4

Impacted Files	Coverage Δ	Complexity Δ
...zally/rule/zalando/Use429HeaderForRateLimitRule.kt	62.5% <0%> (-12.5%)	12 <9> (ø)
...do/zally/rule/zalando/PascalCaseHttpHeadersRule.kt	100% <100%> (ø)	5 <5> (ø)	⬇️
...src/main/java/de/zalando/zally/util/OpenApiUtil.kt	76.27% <71.42%> (+15.92%)	0 <0> (ø)	⬇️
...main/java/de/zalando/zally/rule/HttpHeadersRule.kt	82.35% <0%> (-11.77%)	8% <0%> (-5%)

---

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 cae0423...5f4b356. Read the comment docs.

[tkrop]

This one negative check is definitely not enough as long as we do not have tests for de.zalando.zally.util.OpenApiUtil.kt. We need to add a check for each potential header location, to make sure that we do not miss one. Anyhow, I would prefer similar unit tests on de.zalando.zally.util.OpenApiUtil.kt.

[maxim-tschumak]

Agree, we should unit test OpenApiUtil class. I'll implement the tests.

[tkrop]

👍

[maxim-tschumak]

@tkrop please re-review, the improvements have been implemented

[tkrop]

👍

[maxim-tschumak]

👍