small action tweak

Reviewed-on: #33

Cargo.lock

@@ -634,7 +634,7 @@ checksum = "3fdb12b2476b595f9358c5161aa467c2438859caa136dec86c26fdd2efe17b92"
 
 [[package]]
 name = "ops-sat-rs"
-version = "0.1.1"
+version = "0.2.0"
 dependencies = [
  "chrono",
  "delegate 0.12.0",

src/main.rs

@@ -217,6 +217,9 @@ fn main() {
     .expect("creating TCP SPP client failed");
 
     let timestamp_helper = TimeStampHelper::default();
+    // TODO: If the host feature is active, we should use an image executor
+    //  which only displays the execution parameters and does not try
+    //  to call the batch application which does not exist.
     let mut camera_handler: Ims100BatchHandler = Ims100BatchHandler::new_with_default_img_executor(
         CAMERA_HANDLER,
         HOME_PATH.get().unwrap(),

src/pus/action.rs

@@ -32,7 +32,9 @@ use super::{
     TargetedPusService,
 };
 
-pub const DATA_REPLY: u8 = 130;
+pub enum Subservice {
+    DataReply = 130,
+}
 
 pub struct ActionReplyHandler {
     fail_data_buf: [u8; 128],
@@ -275,7 +277,8 @@ pub fn send_data_reply<TmSender: EcssTmSender>(
     tm_sender: &TmSender,
 ) -> Result<(), EcssTmtcError> {
     let sp_header = SpHeader::new_from_apid(apid_target.apid);
-    let sec_header = PusTmSecondaryHeader::new(8, DATA_REPLY, 0, 0, stamp_helper.stamp());
+    let sec_header =
+        PusTmSecondaryHeader::new(8, Subservice::DataReply as u8, 0, 0, stamp_helper.stamp());
     let mut data = Vec::new();
     data.extend(apid_target.apid.to_be_bytes());
     data.extend(apid_target.unique_id.to_be_bytes());