nasa / CTF

This is a new repository for a new tool to be added to the cFS ecosystem called cFS Test Framework (CTF).
27 stars 6 forks source link

string variable from telemetry incorrectly set to binary #31

Closed jcktexas closed 2 years ago

jcktexas commented 2 years ago

SetUserVariableFromTlm incorrectly returns a binary array instead of a string for the following. pcFilePath should be '/cf/strm/gpfs/moderate/test' and not start with 'b'.

[15:06:00.982] variable_plugin (107) INFO: Set user variable: 'pcFilePath' from tlm mid: 'STRM_FILE_LOAD_ARM_TLM_MID' variable: 'pcFilePath' [15:06:00.982] cfs_plugin (450) DEBUG: get_tlm_value for target: cfs_lx_es2_stub, mid: STRM_FILE_LOAD_ARM_TLM_MID, tlm_variable: pcFilePath, is_header : False [15:06:00.983] cfs_interface (619) DEBUG: There are 1 packets with mid 0x30854 [15:06:00.983] cfs_interface (624) DEBUG: packet data = STRM_FILE_LOAD_ARM_RESP_T: {TlmHeader: CFE_MSG_TelemetryHeader_t: {Msg: CFE_MSG_Message_t: {CCSDS: CCSDS_SpacePacket_t: {Pri: CCSDS_PrimaryHeader_t: {StreamId: <plugins.ccsds_plugin.readers.ccdd_export_reader.c_ubyte_Array_2 object at 0x7ffa82360b40>, Sequence: <plugins.ccsds_plugin.readers.ccdd_export_reader.c_ubyte_Array_2 object at 0x7ffa82360b40>, Length: <plugins.ccsds_plugin.readers.ccdd_export_reader.c_ubyte_Array_2 object at 0x7ffa82360b40>}, Ext: CCSDS_ExtendedHeader_t: {Subsystem: <plugins.ccsds_plugin.readers.ccdd_export_reader.c_ubyte_Array_2 object at 0x7ffa82360b40>, SystemId: <plugins.ccsds_plugin.readers.ccdd_export_reader.c_ubyte_Array_2 object at 0x7ffa82360b40>}}}, Sec: CFE_MSG_TelemetrySecondaryHeader_t: {Spare: 0, Source: 17, Seconds: 1001020, Subseconds: 12591, Spare1: <plugins.ccsds_plugin.readers.ccdd_export_reader.c_ubyte_Array_2 object at 0x7ffa82483940>}}, TimesOutAt: CFE_TIME_SysTime_t: {Seconds: 1001320, Subseconds: 825167586}, ucTranId: 1, pcFilePath: b'/cf/strm/gpfs/moderate/test', ucRecvRespCode: 1} [15:06:00.983] cfs_plugin (456) DEBUG: get_tlm_value's return tlm_value: b'/cf/strm/gpfs/moderate/test' [15:06:00.983] ctf_utility (81 ) INFO: Set Variable pcFilePath = b'/cf/strm/gpfs/moderate/test' [15:06:00.983] variable_plugin (97 ) INFO: The variable pcFilePath becomes b'/cf/strm/gpfs/moderate/test' [15:06:00.983] test (105) TEST_PASS: Instruction SetUserVariableFromTlm: {'variable_name': 'pcFilePath', 'mid': 'STRM_FILE_LOAD_ARM_TLM_MID', 'tlm_variable': 'pcFilePath', 'target': 'cfs_lx_es2_stub'} [15:06:00.983] test (242) *** INFO: Waiting 1 time-units before executing SendCfsCommand

blueoceanwater commented 2 years ago

It is completed and will be officially released in v1.6.