fix(SchemaGenerator): exclude inherited properties only when base add…

[John-Paul-R]

fix(SchemaGenerator): exclude inherited properties only when base added to AllOf

resolves #3201

---

I've outlined the theory for this change in this comment: #3201 (comment)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.66%. Comparing base (4ae817b) to head (ca8a636).
⚠️ Report is 12 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3692   +/-   ##
=======================================
  Coverage   94.66%   94.66%           
=======================================
  Files         111      111           
  Lines        3858     3859    +1     
  Branches      778      778           
=======================================
+ Hits         3652     3653    +1     
  Misses        206      206           

Flag	Coverage Δ
Linux	94.66% <100.00%> (+<0.01%)	⬆️
Windows	94.66% <100.00%> (+<0.01%)	⬆️
macOS	94.66% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[martincostello]

Could you also add a test somewhere in the existing integration test project that would produce outputs in a snapshot file please? It's much easier to verify the behaviour in those kinds of tests where the rendered OpenAPI document is visible.

[John-Paul-R]

@martincostello Attempted a snapshot test, let me know if that looks as you expect!

[martincostello]

Just one suggestion for another test, but otherwise seems OK to me.

[martincostello]

Thanks!