Addressing spec inconsistencies with Open API repo spec (v2.3.0)

[sergiishamrai-okta]

Java SDK uses a local copy of the Open API spec, by the original design. This PR contains fixes in swagger code generator classes and templates to address all inconsistencies with OpenAPI spec.

Issue(s)

OKTA-349906

Description

Category

• Bugfix
• Enhancement
• Unit Test(s)

[codecov-io]

Codecov Report

Merging #558 (abc19b3) into master (077279d) will decrease coverage by 0.13%.
The diff coverage is 88.23%.

@@             Coverage Diff              @@
##             master     #558      +/-   ##
============================================
- Coverage     81.40%   81.26%   -0.14%     
+ Complexity      998      997       -1     
============================================
  Files           122      122              
  Lines          3318     3320       +2     
  Branches        418      419       +1     
============================================
- Hits           2701     2698       -3     
- Misses          428      434       +6     
+ Partials        189      188       -1     

Impacted Files	Coverage Δ	Complexity Δ
...com/okta/sdk/impl/resource/DefaultUserBuilder.java	77.14% <86.66%> (ø)	32.00 <10.00> (ø)
...a/sdk/impl/resource/AbstractPropertyRetriever.java	75.00% <100.00%> (+0.40%)	52.00 <0.00> (+1.00)
...in/java/com/okta/sdk/impl/ds/DefaultDataStore.java	78.18% <0.00%> (-2.28%)	66.00% <0.00%> (-2.00%)

---

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 077279d...abc19b3. Read the comment docs.

[arvindkrishnakumar-okta]

Can you attach a diff of the API changes reported by japicmp plugin?

[arvindkrishnakumar-okta]

Please add a note to the PR description (or title) on which specific Open API spec version we're syncing the Java SDK spec to (v2.3?).

[sergiishamrai-okta]

Japicmp report:

japicmp.txt

[arvindkrishnakumar-okta]

Japicmp report:

japicmp.txt

html file please? it would be more readable.

[arvindkrishnakumar-okta]

We should also write ITs for the new APIs that are brought in with this spec update (wherever possible).

[arvindkrishnakumar-okta]

LGTM, please double check if we've added ITs (wherever possible) for all the new models introduced with this spec update.