0721db9542
tests / build-and-test (push) Failing after 42s
Implements the largest functional gap from the compliance audit. The
equipment now queues events the host can't immediately receive (either
because there's no SELECTED session or because the demo's force-spool
flag is on) and transmits the queue on host request.
What's new
include/secsgem/gem/store/spool.hpp
SpoolStore: a deque queue with a configurable per-stream whitelist
(so only streams 5+6 spool by default), a max_size cap with FIFO
eviction on overflow, and a `force_spool` test flag. Enqueue
returns one of Queued / Dropped_NotSpoolable / Dropped_Full so the
caller can fall back to live delivery when appropriate. Drain
pops the entire queue in FIFO order. Two new ack enums:
ResetSpoolAck (S2F44 RSPACK) and SpoolRequestAck (S6F24 RSDA), plus
SpoolRequestCode (S6F23 RSDC, Transmit/Purge).
data/messages.yaml + auto-regenerated messages.hpp
S2F43 W <L,n <B stream>> Reset Spooling
S2F44 <L,2 <B RSPACK> <L,a ...>> Reset Spooling Ack
S6F23 W <B RSDC> Request Spooled Data
S6F24 <B RSDA> Request Spooled Data Ack
data/equipment.yaml
`spool:` section: max_size + spoolable_streams list. Two new host
commands SPOOL_ON / SPOOL_OFF that flip the force-spool flag (these
stand in for "host link down" in the demo without dropping TCP).
include/secsgem/gem/store/host_commands.hpp
Spec/Result gain an optional<bool> force_spool field. S2F41
dispatch returns the result, the server applies it after S2F42 is
queued.
src/config/loader.cpp
Reads `spool:` from equipment.yaml; reads `force_spool` from each
host_commands entry; populates SpoolStore + CommandSpec.
apps/secs_server.cpp
New `deliver_or_spool(msg, what)` helper. emit_event and
emit_alarm_set funnel through it: if force_spool is on (or there's
no active session), msg.stream is checked against the spoolable
list and the message is enqueued; otherwise it's sent live.
Two new handlers:
S2F43 parses the stream list, updates SpoolStore, replies S2F44
S6F23 RSDC=Transmit drains and re-sends each as a fresh primary
(posted on the executor so the S6F24 ack flushes first);
RSDC=Purge clears the queue and acks.
The S2F41 handler now also propagates result.force_spool into the
SpoolStore.
apps/secs_client.cpp
Demo extended with 4 new steps after the FAULT branch:
SPOOL_ON -> S2F42 Accept
START -> S2F42 Accept; CEID 300 emission spooled (no live S6F11)
SPOOL_OFF -> S2F42 Accept; queue still has the message
S6F23(Transmit) -> S6F24 Accept; spooled S6F11 arrives next
Then the existing S7F19/S7F5/S10F1/S1F15/Separate flow continues.
tests/test_data_model.cpp
Four new TEST_CASEs for SpoolStore (whitelist, FIFO eviction at
max_size, drain ordering, force flag).
tests/test_loader.cpp
Confirms equipment.yaml's `spool:` section populates the store and
`force_spool: true/false` flows through to dispatch results.
COMPLIANCE.md
Spooling moves from ⬜ to 🟡. Adds S2F43/F44 + S6F23/F24 as ✅ in
the message coverage matrix; calls out what's still missing
(S6F25/F26 notification, automatic activation on HSMS NOT-SELECTED,
persistent on-disk spool).
Verified
- Tests: 73 cases / 383 assertions pass (+4 spool cases).
- Demo (docker compose up server client) walks the full happy path
and the spool path, observed in the server log as:
spool: force_spool=true (depth=0)
spool: S6F11 CEID=300 queued (depth=1)
spool: force_spool=false (depth=1)
S6F23 transmit: draining 1 messages
and on the host side as the queued S6F11 arriving in the correct
order after S6F24.
Known limitations (logged in COMPLIANCE.md)
- Spool activation is manual via SPOOL_ON/OFF rather than
automatically triggered by HSMS NOT-SELECTED.
- No S6F25/F26 spooled-data-ready notification on re-SELECT.
- In-memory only; an equipment restart loses queued events.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
245 lines
8.6 KiB
C++
245 lines
8.6 KiB
C++
#include <doctest/doctest.h>
|
|
|
|
#include "secsgem/gem/data_model.hpp"
|
|
|
|
using namespace secsgem::gem;
|
|
|
|
// ---- Status variables ----------------------------------------------------
|
|
|
|
TEST_CASE("SVID add / get / list / set value") {
|
|
StatusVariableStore svids;
|
|
svids.add({1, "Clock", "", s2::Item::ascii("19700101000000")});
|
|
svids.add({2, "EventsEnabled", "", s2::Item::boolean(true)});
|
|
|
|
CHECK(svids.get(1).has_value());
|
|
CHECK_FALSE(svids.get(99).has_value());
|
|
CHECK(svids.all().size() == 2);
|
|
|
|
svids.set_value(2, s2::Item::boolean(false));
|
|
CHECK(svids.get(2)->value == s2::Item::boolean(false));
|
|
}
|
|
|
|
// ---- Equipment constants -------------------------------------------------
|
|
|
|
TEST_CASE("ECID set rejects unknown id and out-of-range values") {
|
|
EquipmentConstantStore ecids;
|
|
ecids.add({10, "MaxSpool", "msgs", s2::Item::u4(uint32_t{100}),
|
|
s2::Item::u4(uint32_t{100}), "0", "1000"});
|
|
CHECK(ecids.set_value(10, s2::Item::u4(uint32_t{50})) == EquipmentAck::Accept);
|
|
CHECK(ecids.get(10)->value == s2::Item::u4(uint32_t{50}));
|
|
CHECK(ecids.set_value(999, s2::Item::u4(uint32_t{1})) == EquipmentAck::Denied_UnknownEcid);
|
|
|
|
// Out-of-range rejected (closes COMPLIANCE.md gap).
|
|
CHECK(ecids.set_value(10, s2::Item::u4(uint32_t{5000})) == EquipmentAck::Denied_OutOfRange);
|
|
CHECK(ecids.get(10)->value == s2::Item::u4(uint32_t{50})); // unchanged
|
|
}
|
|
|
|
TEST_CASE("ECID with no min/max accepts any value") {
|
|
EquipmentConstantStore ecids;
|
|
ecids.add({1, "n", "", s2::Item::u4(uint32_t{0}), s2::Item::u4(uint32_t{0}), "", ""});
|
|
CHECK(ecids.set_value(1, s2::Item::u4(uint32_t{99999999})) == EquipmentAck::Accept);
|
|
}
|
|
|
|
TEST_CASE("ECID range validation skipped for non-numeric formats") {
|
|
EquipmentConstantStore ecids;
|
|
ecids.add({1, "label", "", s2::Item::ascii("a"), s2::Item::ascii("a"), "0", "10"});
|
|
CHECK(ecids.set_value(1, s2::Item::ascii("foo")) == EquipmentAck::Accept);
|
|
}
|
|
|
|
// ---- Clock ---------------------------------------------------------------
|
|
|
|
TEST_CASE("clock string has 16 characters") {
|
|
Clock c;
|
|
auto s = c.current_time_string();
|
|
REQUIRE(s.size() == 16);
|
|
}
|
|
|
|
TEST_CASE("set_time_string accepts well-formed and rejects malformed") {
|
|
Clock c;
|
|
CHECK(c.set_time_string("20260101000000") == TimeAck::Accept);
|
|
CHECK(c.set_time_string("2026010100000000") == TimeAck::Accept);
|
|
CHECK(c.set_time_string("not-a-date-12345") == TimeAck::Error);
|
|
CHECK(c.set_time_string("short") == TimeAck::Error);
|
|
}
|
|
|
|
// ---- Host command registry ----------------------------------------------
|
|
|
|
TEST_CASE("host command registry returns spec + result") {
|
|
HostCommandRegistry r;
|
|
r.register_command("START", {HostCmdAck::Accept, 300, std::nullopt});
|
|
r.register_command("STOP", {HostCmdAck::CannotDoNow, std::nullopt, std::nullopt});
|
|
r.register_command("FAULT", {HostCmdAck::Accept, std::nullopt, 1});
|
|
|
|
CHECK(r.has("START"));
|
|
CHECK_FALSE(r.has("PAUSE"));
|
|
|
|
auto start = r.dispatch("START", {});
|
|
CHECK(start.ack == HostCmdAck::Accept);
|
|
CHECK(start.emit_ceid.value_or(0) == 300);
|
|
|
|
auto fault = r.dispatch("FAULT", {});
|
|
CHECK(fault.set_alarm.value_or(0) == 1);
|
|
|
|
CHECK(r.dispatch("UNKNOWN", {}).ack == HostCmdAck::InvalidCommand);
|
|
}
|
|
|
|
// ---- Event reports -------------------------------------------------------
|
|
|
|
TEST_CASE("define reports rejects unknown VID") {
|
|
EventReportSubscriptions ev;
|
|
ev.register_event({100, "x"});
|
|
auto exists = [](uint32_t id) { return id == 1 || id == 2; };
|
|
|
|
CHECK(ev.define_reports({{1000, {1, 2}}}, exists) == DefineReportAck::Accept);
|
|
CHECK(ev.define_reports({{1001, {1, 999}}}, exists) == DefineReportAck::InvalidVid);
|
|
}
|
|
|
|
TEST_CASE("full event-report pipeline") {
|
|
StatusVariableStore svids;
|
|
svids.add({1, "ControlState", "", s2::Item::ascii("OnlineRemote")});
|
|
svids.add({2, "Clock", "", s2::Item::ascii("19700101000000")});
|
|
|
|
EventReportSubscriptions ev;
|
|
ev.register_event({100, "ControlStateChanged"});
|
|
ev.register_event({200, "AlarmSetEvent"});
|
|
|
|
auto exists = [&](uint32_t id) { return svids.has(id); };
|
|
auto value = [&](uint32_t id) { return svids.value(id); };
|
|
|
|
REQUIRE(ev.define_reports({{1000, {1}}, {1001, {1, 2}}}, exists) == DefineReportAck::Accept);
|
|
REQUIRE(ev.link_event_reports({{100, {1000, 1001}}, {200, {1001}}}) == LinkEventAck::Accept);
|
|
CHECK_FALSE(ev.is_enabled(100));
|
|
|
|
REQUIRE(ev.enable_events(true, {100}) == EnableEventAck::Accept);
|
|
CHECK(ev.is_enabled(100));
|
|
CHECK_FALSE(ev.is_enabled(200));
|
|
|
|
auto reports = ev.compose_for(100, value);
|
|
REQUIRE(reports.size() == 2);
|
|
CHECK(reports[0].rptid == 1000);
|
|
REQUIRE(reports[0].values.size() == 1);
|
|
CHECK(reports[0].values[0] == s2::Item::ascii("OnlineRemote"));
|
|
CHECK(reports[1].rptid == 1001);
|
|
REQUIRE(reports[1].values.size() == 2);
|
|
}
|
|
|
|
TEST_CASE("enable_events with empty CEID list enables all registered events") {
|
|
EventReportSubscriptions ev;
|
|
ev.register_event({1, "a"});
|
|
ev.register_event({2, "b"});
|
|
CHECK(ev.enable_events(true, {}) == EnableEventAck::Accept);
|
|
CHECK(ev.is_enabled(1));
|
|
CHECK(ev.is_enabled(2));
|
|
CHECK(ev.enable_events(false, {}) == EnableEventAck::Accept);
|
|
CHECK_FALSE(ev.is_enabled(1));
|
|
}
|
|
|
|
TEST_CASE("link_event_reports rejects unknown CEID or RPTID") {
|
|
EventReportSubscriptions ev;
|
|
ev.register_event({100, "x"});
|
|
ev.define_reports({{500, {1}}}, [](uint32_t) { return true; });
|
|
|
|
CHECK(ev.link_event_reports({{999, {500}}}) == LinkEventAck::UnknownCeid);
|
|
CHECK(ev.link_event_reports({{100, {999}}}) == LinkEventAck::UnknownRptid);
|
|
CHECK(ev.link_event_reports({{100, {500}}}) == LinkEventAck::Accept);
|
|
}
|
|
|
|
// ---- Alarms --------------------------------------------------------------
|
|
|
|
TEST_CASE("alarm enable / set / clear / list") {
|
|
AlarmRegistry r;
|
|
r.add({1, "Chiller Temp High", 4});
|
|
r.add({2, "Door Open", 1});
|
|
|
|
CHECK(r.set_enabled(1, true) == AlarmAck::Accept);
|
|
CHECK(r.enabled(1));
|
|
CHECK(r.set_enabled(999, true) == AlarmAck::Error);
|
|
|
|
auto alcd_set = r.set_active(1);
|
|
REQUIRE(alcd_set.has_value());
|
|
CHECK((*alcd_set & 0x80) != 0);
|
|
CHECK((*alcd_set & 0x7F) == 4);
|
|
CHECK(r.active(1));
|
|
|
|
auto alcd_clr = r.clear_active(1);
|
|
REQUIRE(alcd_clr.has_value());
|
|
CHECK((*alcd_clr & 0x80) == 0);
|
|
CHECK_FALSE(r.active(1));
|
|
|
|
CHECK(r.all().size() == 2);
|
|
}
|
|
|
|
// ---- Process programs ----------------------------------------------------
|
|
|
|
TEST_CASE("recipe CRUD") {
|
|
RecipeStore r;
|
|
r.add("RECIPE-A", "step1\nstep2\n");
|
|
r.add("RECIPE-B", "alt body");
|
|
CHECK(r.list().size() == 2);
|
|
CHECK(r.get("RECIPE-A").value() == "step1\nstep2\n");
|
|
CHECK_FALSE(r.get("UNKNOWN").has_value());
|
|
|
|
CHECK(r.remove("RECIPE-A") == ProcessProgramAck::Accept);
|
|
CHECK_FALSE(r.get("RECIPE-A").has_value());
|
|
CHECK(r.remove("RECIPE-A") == ProcessProgramAck::PpidNotFound);
|
|
}
|
|
|
|
// ---- Composite EquipmentDataModel ---------------------------------------
|
|
|
|
TEST_CASE("EquipmentDataModel composes the eight stores") {
|
|
EquipmentDataModel m;
|
|
m.svids.add({1, "Clock", "", s2::Item::ascii("")});
|
|
m.dvids.add({2, "Temp", "C", s2::Item::u4(uint32_t{25})});
|
|
CHECK(m.vid_exists(1));
|
|
CHECK(m.vid_exists(2));
|
|
CHECK_FALSE(m.vid_exists(3));
|
|
CHECK(m.vid_value(2) == s2::Item::u4(uint32_t{25}));
|
|
}
|
|
|
|
// ---- Spool ---------------------------------------------------------------
|
|
|
|
TEST_CASE("spool enqueue respects spoolable_streams whitelist") {
|
|
SpoolStore s;
|
|
s.set_spoolable_streams({5, 6});
|
|
CHECK(s.enqueue(s2::Message(6, 11, false)) == SpoolStore::EnqueueResult::Queued);
|
|
CHECK(s.enqueue(s2::Message(5, 1, false)) == SpoolStore::EnqueueResult::Queued);
|
|
CHECK(s.enqueue(s2::Message(1, 2, false)) == SpoolStore::EnqueueResult::Dropped_NotSpoolable);
|
|
CHECK(s.size() == 2);
|
|
}
|
|
|
|
TEST_CASE("spool FIFO eviction when max_size reached") {
|
|
SpoolStore s;
|
|
s.set_spoolable_streams({6});
|
|
s.set_max_size(3);
|
|
for (int i = 0; i < 5; ++i)
|
|
s.enqueue(s2::Message(6, 11, false, s2::Item::u4(static_cast<uint32_t>(i))));
|
|
CHECK(s.size() == 3);
|
|
// Oldest two (0, 1) were evicted; remaining are 2, 3, 4.
|
|
auto drained = s.drain();
|
|
REQUIRE(drained.size() == 3);
|
|
auto first_u4 = [](const s2::Message& m) -> uint32_t {
|
|
return std::get<std::vector<uint32_t>>(m.body->storage()).front();
|
|
};
|
|
CHECK(first_u4(drained[0]) == 2);
|
|
CHECK(first_u4(drained[2]) == 4);
|
|
}
|
|
|
|
TEST_CASE("spool drain returns FIFO order and empties the queue") {
|
|
SpoolStore s;
|
|
s.set_spoolable_streams({6});
|
|
s.enqueue(s2::Message(6, 11, false, s2::Item::u4(uint32_t{1})));
|
|
s.enqueue(s2::Message(6, 11, false, s2::Item::u4(uint32_t{2})));
|
|
auto out = s.drain();
|
|
CHECK(out.size() == 2);
|
|
CHECK(s.empty());
|
|
CHECK(s.drain().empty());
|
|
}
|
|
|
|
TEST_CASE("spool force flag controls whether enqueue is taken") {
|
|
SpoolStore s;
|
|
s.set_spoolable_streams({6});
|
|
CHECK_FALSE(s.force_spool());
|
|
s.set_force_spool(true);
|
|
CHECK(s.force_spool());
|
|
}
|