Dev/mibi/arc inheritance circle

[sonomirco]

What type of PR is this? (check all applicable)

• ♻️ Refactor
• ✨ Feature
• 🐛 Bug Fix
• 👷 Optimization
• 📝 Documentation Update
• 🔖 Release
• 🚩 Other

Description

This PR makes Arc inheriting from Circle.

Related Tickets & Documents

This PR closes #262

Added tests?

• 👍 yes
• 🙅 no, because they aren't needed
• 🙋 no, because I need help

Added to documentation?

• 📓 docs
• 🙅 no documentation needed

[sonomirco]

@cesarecaoduro and @d3ssy probably some tests between Arc and Circle are repetitive, what do you think?
This Pr only makes the inheritance of Circle into Arc, next step will be to integrate the new methods for ICurve and remove the properties for the NURBS and expose the ToNurbs method.

[d3ssy]

@cesarecaoduro and @d3ssy probably some tests between Arc and Circle are repetitive, what do you think?
This Pr only makes the inheritance of Circle into Arc, next step will be to integrate the new methods for ICurve and remove the properties for the NURBS and expose the ToNurbs method.

Yes, since arc inherits from circle, any tests written for circle don't need to be repeated for arc. ArcTests should be testing any additional behavior if any.

[cesarecaoduro]

@sonomirco and @d3ssy I would probably keep both for readability