fork/Graphite
1
0
Fork 0
mirror of https://github.com/GraphiteEditor/Graphite.git synced 2026-09-15 22:28:10 +08:00
Code Issues Packages Projects Releases Wiki Activity

Adapt the test surface to our wire kind and no-content graphic

Browse Source
This commit is contained in:
Dennis Kobert
2026-09-08 17:41:29 +00:00
parent 33b20d6329
commit a593430e38
3 changed files with 9 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
editor/src/messages/portfolio/document/document_message_handler.rs
View File

@@ -4354,8 +4354,9 @@ mod document_message_handler_tests {
let phantom_count = news
.iter()
.flat_map(|new| new.iter_element_values())
.filter(|graphic| matches!(graphic, graphene_std::Graphic::None(_)))
// Our no-content graphic is an empty list rather than master's dedicated None variant
.filter(|graphic| graphic.is_empty())
.count();
assert_eq!(phantom_count, 0, "No stacked element should be a phantom None graphic");
assert_eq!(phantom_count, 0, "No stacked element should be a phantom empty graphic");
}
}

3
editor/src/messages/portfolio/document_migration.rs
View File

@@ -2921,7 +2921,8 @@ mod tests {
#[test]
fn removed_definition_swap_targets_resolve() {
assert!(resolve_proto_node_type(graphene_std::ops::passthrough::IDENTIFIER).is_some());
assert!(resolve_proto_node_type(graphene_std::platform_application_io::upload_texture::IDENTIFIER).is_some());
#[cfg(feature = "gpu")]
assert!(resolve_proto_node_type(wgpu_executor::texture_conversion::upload_texture::IDENTIFIER).is_some());
}
// Migrating a Merge network's coercion nodes by alias would leave a reducer in the primary slot, so every alias must reset instead

14
node-graph/graph-craft/src/document/value.rs
View File

@@ -1026,7 +1026,7 @@ mod typedefault_dispatch {
fn typedefault_dispatch_terminates() {
macro_rules! check {
($type_default:ty, $stored:expr) => {{
let ty: Type = $stored;
let ty: TypeDescriptor = $stored;
let expected_type_id = std::any::TypeId::of::<$type_default>();
let dyn_value = TaggedValue::TypeDefault(ty.clone()).to_dynany();
assert_eq!(
@@ -1045,22 +1045,17 @@ mod typedefault_dispatch {
);
}};
}
macro_rules! check_item {
($element:ty) => {
check!(Item<$element>, concrete!($element));
};
}
// One wire kind: a type default names its type, so the item and list lists both check their list form.
macro_rules! check_list {
($element:ty) => {
check!(List<$element>, concrete!(List<$element>));
check!(List<$element>, core_types::descriptor!(List<$element>));
};
}
macro_rules! check_bare {
($type_default:ty) => {
check!($type_default, concrete!($type_default));
check!($type_default, core_types::descriptor!($type_default));
};
}
for_each_item_type_default!(check_item);
for_each_list_type_default!(check_list);
for_each_bare_type_default!(check_bare);
}
@@ -1069,7 +1064,6 @@ mod typedefault_dispatch {
#[cfg(test)]
mod paint_default_parsing {
use super::*;
use core_types::{item, list};
/// A Fill/Stroke paint wire carries `Graphic` elements, so its `Color::BLACK` default must parse through the
/// element recursion into a `Color` for a fresh Fill node's paint to resolve.