From 7b8945ff15972f18cf62f0967679dc98bc5617c1 Mon Sep 17 00:00:00 2001 From: Daniel Date: Fri, 14 Feb 2025 15:47:13 +0100 Subject: [PATCH] Don't write input stats to both input and output (#734) --- plugins/statistics_exporter/instance.ts | 2 +- plugins/statistics_exporter/test/plugin.js | 10 ++++++++++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/plugins/statistics_exporter/instance.ts b/plugins/statistics_exporter/instance.ts index 91e68284..2557089c 100644 --- a/plugins/statistics_exporter/instance.ts +++ b/plugins/statistics_exporter/instance.ts @@ -98,7 +98,7 @@ function setForceFlowStatistic( statisticName, reverseDirection, item - ).set(value); + ); } } diff --git a/plugins/statistics_exporter/test/plugin.js b/plugins/statistics_exporter/test/plugin.js index 4d315fbc..6c0393e1 100644 --- a/plugins/statistics_exporter/test/plugin.js +++ b/plugins/statistics_exporter/test/plugin.js @@ -66,9 +66,19 @@ describe("statistics_exporter plugin", function() { assert.equal(instance._instanceForceFlowStatistics.labels( "7357", "nauvis", "player", "item_production_statistics", "input", "inserter").get(), 10); + assert.equal(instance._instanceForceFlowStatistics.labels( + "7357", "nauvis", "player", "item_production_statistics", "output", "iron-plate").get(), + 24); + // Assert that it didn't record inserter consumption as production + assert.equal(instance._instanceForceFlowStatistics.labels( + "7357", "nauvis", "player", "item_production_statistics", "output", "inserter").get(), + 0); assert.equal(instance._instanceGameFlowStatistics.labels( "7357", "nauvis", "pollution_statistics", "input", "boiler").get(), 2000); + assert.equal(instance._instanceGameFlowStatistics.labels( + "7357", "nauvis", "pollution_statistics", "output", "tree-proxy").get(), + 560); assert.equal(instance._instancePlatformMapping.labels( "7357", "platform-1", "player", "nauvis").get(), 1);