rei-mirrors/Fornjot
1
0
Fork 0
mirror of https://github.com/hannobraun/Fornjot synced 2025-02-06 07:15:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

Set half-edge geometry in test

Browse Source
This commit is contained in:
Hanno Braun 2024-03-20 12:29:20 +01:00
parent f7f52fa687
commit 3f5c2854cd
1 changed files with 36 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
crates/fj-core/src/validate/shell.rs
View File

@ -431,6 +431,34 @@ mod tests {
cycle.update_half_edge( cycle.update_half_edge(
cycle.half_edges().nth_circular(0), cycle.half_edges().nth_circular(0),
|half_edge, core| { |half_edge, core| {
// This is going to be weird:
//
// - That first call to `update_path` is
// going to reverse a path and insert
// a new object with the reversed
// path.
// - The next call to `update_boundary`
// relies on that, because it inserts
// an object too, and that would cause
// a validation failure without the
// first call.
// - But the new object created from
// those two operations doesn't
// actually have its geometry set in
// the geometry layer, because that
// happens in `update_path`, for an
// earlier version of the object.
// - So we need a last `set_path`, which
// sets the path again.
//
// This is very weird, but good new is,
// it's just an artifact of the
// transition from a unified object
// graph to separate topology and
// geometry layers. This should clear up
// again, once the separation is
// finished, and all APIs can adapt to
// the new reality.
[half_edge [half_edge
.update_path( .update_path(
|path| path.reverse(), |path| path.reverse(),
@ -439,6 +467,14 @@ mod tests {
.update_boundary( .update_boundary(
|boundary| boundary.reverse(), |boundary| boundary.reverse(),
core, core,
)
.set_path(
core.layers
.geometry
.of_half_edge(half_edge)
.path
.reverse(),
&mut core.layers.geometry,
)] )]
}, },
core, core,