use BRep_Builder for single face shell

[snoyer]

Tried making a shell from the single curved face of a cone, got segfault from BRepBuilderAPI_MakeShell. BRep_Builder seems to work fine in that case.
I'm not sure this is a 100% reliable solution but according to the tests it's not worse.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.61%. Comparing base (d78ca93) to head (d0d1a44).
Report is 4 commits behind head on dev.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #875      +/-   ##
==========================================
+ Coverage   96.58%   96.61%   +0.03%     
==========================================
  Files          31       31              
  Lines        9124     9138      +14     
==========================================
+ Hits         8812     8829      +17     
+ Misses        312      309       -3     

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

[gumyr]

Oh, sorry I split the test_direct_api.py file into 32 separate files which breaks this PR. It should be easy to fix though as the content of the tests hasn't changed so test_shells.py should be the new home of this test.

Otherwise, thanks for this change. This lower level implementation looks better in handling the case of a non-planar face which doesn't seem like it should be too difficult.

[snoyer]

I can't fix it right now but you should have write access to the PR branch if you want to resolve