zig/mach
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

module: add test for zero-size component bug

Browse source 
Signed-off-by: Stephen Gutekanst <stephen@hexops.com>
This commit is contained in:
Stephen Gutekanst 2024-05-06 13:46:33 -07:00
parent 261f94d3bc
commit 1b65702ae8
1 changed files with 18 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

19
src/module/entities.zig
View file

@ -838,6 +838,7 @@ test "example" {
.name = .{ .type = []const u8 }, .name = .{ .type = []const u8 },
.location = .{ .type = Location }, .location = .{ .type = Location },
.rotation = .{ .type = Rotation }, .rotation = .{ .type = Rotation },
.is_monster = .{ .type = void },
}; };
}; };
@ -876,13 +877,27 @@ test "example" {
try world.removeComponent(player1, .game, .location); try world.removeComponent(player1, .game, .location);
// try world.removeComponent(player1, .game, .location); // doesn't exist? no problem. // try world.removeComponent(player1, .game, .location); // doesn't exist? no problem.
//-------------------------------------------------------------------------
// Zero-size tags, these can be put on entities to indicate they are unique
// in some way. e.g. to indicate which entities are monsters, which you can
// then query
try testing.expectEqual(@as(?void, null), world.getComponent(player1, .game, .is_monster));
try world.setComponent(player1, .game, .is_monster, {});
try testing.expectEqual(@as(?void, {}), world.getComponent(player1, .game, .is_monster));
try testing.expectEqual(@as(?void, {}), world.getComponent(player2, .game, .is_monster)); // BUG: this reports non-null when it should report null!
try world.removeComponent(player1, .game, .is_monster);
try testing.expectEqual(@as(?void, {}), world.getComponent(player1, .game, .is_monster)); // BUG: this reports non-null when it should report null!
try testing.expectEqual(@as(?void, {}), world.getComponent(player2, .game, .is_monster)); // BUG: this reports non-null when it should report null!
//------------------------------------------------------------------------- //-------------------------------------------------------------------------
// Introspect things. // Introspect things.
// //
// Archetype IDs, these are our "table names" - they're just hashes of all the component names // Archetype IDs, these are our "table names" - they're just hashes of all the component names
// within the archetype table. // within the archetype table.
const archetypes = world.archetypes.items; const archetypes = world.archetypes.items;
try testing.expectEqual(@as(usize, 5), archetypes.len); try testing.expectEqual(@as(usize, 6), archetypes.len);
// TODO: better table names, based on columns // TODO: better table names, based on columns
// try testing.expectEqual(@as(u64, 0), archetypes[0].hash); // try testing.expectEqual(@as(u64, 0), archetypes[0].hash);
// try testing.expectEqual(@as(u32, 4), archetypes[1].name); // try testing.expectEqual(@as(u32, 4), archetypes[1].name);
@ -895,6 +910,8 @@ test "example" {
try testing.expectEqual(@as(usize, 0), archetypes[1].len); try testing.expectEqual(@as(usize, 0), archetypes[1].len);
try testing.expectEqual(@as(usize, 0), archetypes[2].len); try testing.expectEqual(@as(usize, 0), archetypes[2].len);
try testing.expectEqual(@as(usize, 1), archetypes[3].len); try testing.expectEqual(@as(usize, 1), archetypes[3].len);
try testing.expectEqual(@as(usize, 0), archetypes[4].len);
try testing.expectEqual(@as(usize, 0), archetypes[5].len);
// Resolve archetype by entity ID and print column names // Resolve archetype by entity ID and print column names
const columns = world.archetypeByID(player2).columns; const columns = world.archetypeByID(player2).columns;