Update sdd, topology, and add int tests

Author: moisesmata

FprimeZephyrReference/Components/Watchdog/docs/sdd.md

@@ -8,14 +8,14 @@ The `Components::Watchdog` component provides the mechanism to "pet" the physica
 
 The requirements for `Components::Watchdog` are as follows:
 
-Requirement | Description | Verification Method
------------ | ----------- | -------------------
-WD-001 | The `Components::Watchdog` component shall activate upon startup. | Unit Test
-WD-002 | The `Components::Watchdog` component shall oscillate the watchdog GPIO pin (24) on/off on each rategroup tick. | Unit Test
-WD-003 | The `Components::Watchdog` component shall provide telemetry for watchdog transition count. | Unit Test
-WD-004 | The `Components::Watchdog` component shall respond to stop signals to halt the watchdog petting. | Unit Test
-WD-005 | The `Components::Watchdog` component shall provide a test command to stop the watchdog petting. | Unit Test
-WD-006 | The `Components::Watchdog` component shall emit an event when the watchdog petting stops. | Unit Test
+Requirement | Description | Verification Method | Verified?
+----------- | ----------- | ------------------- | ---------
+WD-001 | The `Components::Watchdog` component shall activate upon startup. | Inspection | Yes
+WD-002 | The `Components::Watchdog` component shall oscillate the watchdog GPIO pin (24) on/off on each rategroup tick. | Inspection | Yes
+WD-003 | The `Components::Watchdog` component shall provide telemetry for watchdog transition count. | Integration Test | In Progress
+WD-004 | The `Components::Watchdog` component shall respond to stop signals to halt the watchdog petting. | Integration Test | Yes
+WD-005 | The `Components::Watchdog` component shall provide a test command to stop the watchdog petting. | Integration Test | Yes
+WD-006 | The `Components::Watchdog` component shall emit an event when the watchdog petting stops. | Integration Test | Yes
 
 ## 3. Design
 
@@ -52,7 +52,3 @@ WatchdogStop | Emits once the watchdog petting has stopped. .
 Name | Type | Description 
 ---- | ---- | -----
 WatchdogTransitions | U32 | Number of times the GPIO has oscillated from on/off during watchdog petting
-
-## 4. Unit Testing
-
-TODO

FprimeZephyrReference/ReferenceDeployment/Top/CMakeLists.txt

@@ -14,6 +14,7 @@ register_fprime_module(
     AUTOCODER_INPUTS
         "${CMAKE_CURRENT_LIST_DIR}/instances.fpp"
         "${CMAKE_CURRENT_LIST_DIR}/topology.fpp"
+        "${CMAKE_CURRENT_LIST_DIR}/ReferenceDeploymentPackets.fppi"
     SOURCES
         "${CMAKE_CURRENT_LIST_DIR}/ReferenceDeploymentTopology.cpp"
     DEPENDS

FprimeZephyrReference/ReferenceDeployment/Top/topology.fpp

@@ -84,14 +84,14 @@ module ReferenceDeployment {
       # High rate (10Hz) rate group
       rateGroupDriver.CycleOut[Ports_RateGroups.rateGroup10Hz] -> rateGroup10Hz.CycleIn
       rateGroup10Hz.RateGroupMemberOut[0] -> comDriver.schedIn
-      rateGroup10Hz.RateGroupMemberOut[1] -> watchdog.run
 
       # Slow rate (1Hz) rate group
       rateGroupDriver.CycleOut[Ports_RateGroups.rateGroup1Hz] -> rateGroup1Hz.CycleIn
       rateGroup1Hz.RateGroupMemberOut[0] -> ComCcsds.comQueue.run
       rateGroup1Hz.RateGroupMemberOut[1] -> CdhCore.$health.Run
       rateGroup1Hz.RateGroupMemberOut[2] -> ComCcsds.commsBufferManager.schedIn
       rateGroup1Hz.RateGroupMemberOut[3] -> CdhCore.tlmSend.Run
+      rateGroup1Hz.RateGroupMemberOut[4] -> watchdog.run
     }
 
     connections Watchdog {

FprimeZephyrReference/test/int/README.md

@@ -0,0 +1,7 @@
+### To run Integration Tests:
+
+First, start GDS with:
+
+
+Run Integration Test:
+pytest integration_test.py --deployment ../../../build-artifacts/zephyr/fprime-zephyr-deployment

FprimeZephyrReference/test/int/integration_test.py

@@ -0,0 +1,64 @@
+"""
+integration_test.py:
+
+Simple integration tests for the Watchdog component.
+Tests are ordered so that stop tests run last.
+"""
+
+def get_watchdog_transitions(fprime_test_api):
+    """Helper function to request packet and get fresh WatchdogTransitions telemetry"""
+    fprime_test_api.clear_histories()
+    fprime_test_api.send_and_assert_command("CdhCore.tlmSend.SEND_PKT", ["5"], max_delay=2)
+    result = fprime_test_api.assert_telemetry(
+        "ReferenceDeployment.watchdog.WatchdogTransitions",
+        start="NOW", timeout=3
+    )
+    return result.get_val()
+
+
+def test_01_watchdog_telemetry_basic(fprime_test_api):
+    """Test that we can read WatchdogTransitions telemetry"""
+    value = get_watchdog_transitions(fprime_test_api)
+    assert value >= 0, f"WatchdogTransitions should be >= 0, got {value}"
+
+
+def test_02_watchdog_increments(fprime_test_api):
+    """Test that WatchdogTransitions increments over time"""
+    import time
+    
+    initial_value = get_watchdog_transitions(fprime_test_api)
+    time.sleep(2.0)  # Wait for watchdog to run more cycles
+    updated_value = get_watchdog_transitions(fprime_test_api)
+    
+    assert updated_value > initial_value, \
+        f"WatchdogTransitions should increase. Initial: {initial_value}, Updated: {updated_value}"
+
+
+def test_03_stop_watchdog_command(fprime_test_api):
+    """Test TEST_STOP_WATCHDOG command sends and emits WatchdogStop event"""
+    fprime_test_api.clear_histories()
+    
+    fprime_test_api.send_and_assert_command(
+        "ReferenceDeployment.watchdog.TEST_STOP_WATCHDOG", 
+        max_delay=2
+    )
+    
+    fprime_test_api.assert_event(
+        "ReferenceDeployment.watchdog.WatchdogStop",
+        timeout=2
+    )
+
+
+def test_04_watchdog_stops_incrementing(fprime_test_api):
+    """Test that WatchdogTransitions stops incrementing after TEST_STOP_WATCHDOG"""
+    import time
+    
+    # Get initial value (should be from stopped watchdog from previous test)
+    initial_value = get_watchdog_transitions(fprime_test_api)
+    
+    # Wait and check that it's not incrementing (watchdog should already be stopped)
+    time.sleep(2.0)
+    final_value = get_watchdog_transitions(fprime_test_api)
+    
+    assert final_value == initial_value, \
+        f"Watchdog should remain stopped. Initial: {initial_value}, Final: {final_value}"