Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions bakkesmod/subtr-actor/rust/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@ use subtr_actor::BoostPickupPadZone;
use subtr_actor::EventLifecycle;
#[cfg(test)]
use subtr_actor::ReplayFrameInputBuilder;
#[cfg(test)]
use subtr_actor::default_stats_module_names;
use subtr_actor::{
BackboardBounceEvent, BoostPickupEvent, BumpEvent, CorePlayerScoreboardEvent, DemolitionEvent,
Event, EventPayload, EventTiming, FiftyFiftyEvent, FrameInput, GoalBuildupKind,
Expand Down
6 changes: 3 additions & 3 deletions bakkesmod/subtr-actor/rust/src/lib_tests/live_graph_events.rs
Original file line number Diff line number Diff line change
Expand Up @@ -101,8 +101,8 @@ fn exposes_stats_collector_module_json_after_processing_frame() {
let module_names = value["module_names"]
.as_array()
.expect("module names should be an array");
assert_eq!(module_names.len(), builtin_stats_module_names().len());
for module_name in builtin_stats_module_names() {
assert_eq!(module_names.len(), default_stats_module_names().len());
for module_name in default_stats_module_names() {
assert!(
module_names.iter().any(|name| name == module_name),
"stats json should expose stats module {module_name}"
Expand All @@ -127,7 +127,7 @@ fn exposes_stats_collector_module_json_after_processing_frame() {
let frame_modules = value["frame"]["modules"]
.as_object()
.expect("frame modules should be an object");
for module_name in builtin_stats_module_names() {
for module_name in default_stats_module_names() {
assert!(
frame_modules.contains_key(*module_name),
"stats frame should include module payload for {module_name}"
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#[test]
fn live_abi_exposes_every_builtin_stats_module_frame_and_config_by_name() {
fn live_abi_exposes_every_default_stats_module_frame_and_config_by_name() {
let engine = subtr_actor_bakkesmod_engine_create();
let touches = [SaTouchEvent {
timing: SaEventTiming::default(),
Expand Down Expand Up @@ -56,7 +56,7 @@ fn live_abi_exposes_every_builtin_stats_module_frame_and_config_by_name() {
let frame_modules = stats["frame"]["modules"]
.as_object()
.expect("stats json should expose frame modules");
for module_name in builtin_stats_module_names() {
for module_name in default_stats_module_names() {
assert_eq!(
live_stats_module_frame_json_value(engine, module_name),
frame_modules
Expand Down
4 changes: 2 additions & 2 deletions bakkesmod/subtr-actor/rust/src/lib_tests/live_graph_json.rs
Original file line number Diff line number Diff line change
Expand Up @@ -807,7 +807,7 @@ fn live_abi_stats_json_matches_direct_full_graph_across_finish() {
}

#[test]
fn live_abi_exposes_every_builtin_stats_module_by_name() {
fn live_abi_exposes_every_default_stats_module_by_name() {
let engine = subtr_actor_bakkesmod_engine_create();
let touches = [SaTouchEvent {
timing: SaEventTiming::default(),
Expand Down Expand Up @@ -861,7 +861,7 @@ fn live_abi_exposes_every_builtin_stats_module_by_name() {
let modules = stats["modules"]
.as_object()
.expect("stats json should expose a modules object");
for module_name in builtin_stats_module_names() {
for module_name in default_stats_module_names() {
assert_eq!(
live_stats_module_json_value(engine, module_name),
modules
Expand Down
4 changes: 2 additions & 2 deletions bakkesmod/subtr-actor/verify-plugin-source.py
Original file line number Diff line number Diff line change
Expand Up @@ -2078,8 +2078,8 @@ def main() -> int:
)
require_contains(
web_player_main_source,
'renderModuleSummaryGroup("Timeline ranges", rangeToggles)',
"stats evaluation player launcher module summary timeline range group",
'renderModuleSummaryGroup("Timeline views", rangeToggles)',
"stats evaluation player launcher module summary timeline view group",
errors,
)
require_contains(
Expand Down
Loading
Loading