You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The function addFillet has some unit tests, but there's plenty of logic that goes into the AST mod that is still in the AST fillet action that isn't captured by these tests.
We could put more in a testable function, something like
Added tests for getPathToExtrudeForSegmentSelection:
Implemented tests to check if the function correctly identifies extrude paths from selected segments in different sketches. The tests cover various scenarios, including multiple extrusions and cases where no extrusion exists, ensuring the logic works as expected.
The function
addFillet
has some unit tests, but there's plenty of logic that goes into the AST mod that is still in theAST fillet
action that isn't captured by these tests.We could put more in a testable function, something like
☝️ I might be missing inputs, main thing I'm worried about is the
pathToExtrudeNode
being neglectedThe text was updated successfully, but these errors were encountered: