Use Error property when throwing exception

[marcominerva]

What kind of change does this PR introduce?
Bug fix

What is the current behavior?
See #1376

Please check if the PR fulfills these requirements

• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

Other information:
Fixes #1376

[304NotModified]

✅️ Tests for the changes have been added (for bug fixes / features)

I dont see any tests in this PR?

[marcominerva]

Is adding a new test class to Refit.Tests enough? Thank you @304NotModified

[304NotModified]

That's a bit hard to decide without any test code.

But I think only an unit tests is needed, not a integration test / service test or something like that. (See https://martinfowler.com/articles/practical-test-pyramid.html)

martinfowler.com

The Practical Test Pyramid

Find out what kinds of automated tests you should implement for your application and learn by examples what these tests could look like.

[marcominerva]

Thank you for the clarification @304NotModified, I have added a Unit test for this change.

[304NotModified]

Thanks for the tests! Added some comments to make them a bit better :)

[marcominerva]

Done :) Let me know if now is OK!

[304NotModified]

LGTM

[nathanpovo]

This will also fix #1465

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.