diff --git a/test/bats/test-device-connection/test-device-authorization.bats b/test/bats/test-device-connection/test-device-authorization.bats index beca1465..b7b92683 100644 --- a/test/bats/test-device-connection/test-device-authorization.bats +++ b/test/bats/test-device-connection/test-device-authorization.bats @@ -23,10 +23,9 @@ DEVICE_ID="testdevice" DEVICE_ID2="testdevice2" DEVICE_TOKEN_SCOPE="device_id gateway mqtt-broker offline_access subdevice_ids" DEVICE_TOKEN_AUDIENCE_FROM_DIRECT='mqtt-broker' -SUBDEVICE_ID=testsubdevice1 SUBDEVICE_IDS='"[\"testsubdevice1\"]"' +# shellcheck disable=SC2089 SUBDEVICE_IDS2='"[\"testsubdevice1\",\"testsubdevice2\",\"testsubdevice3\"]"' -MQTT_URL=emqx-listeners:1883 MQTT_TOPIC_NAME="spBv1.0/${NAMESPACE}/DDATA/${GATEWAY_ID}/${DEVICE_ID}" MQTT_SUBDEVICE_TOPIC_NAME="spBv1.0/${NAMESPACE}/DDATA/${GATEWAY_ID}/testsubdevice1" MQTT_SUBDEVICE_TOPIC_NAME2="spBv1.0/${NAMESPACE}/DDATA/${GATEWAY_ID}/testsubdevice2" @@ -170,7 +169,7 @@ check_refreshed_device_token_with_subcomponents() { check_json_field "${jwt}" "azp" "device" || return 1 check_json_field "${jwt}" "device_id" "${DEVICE_ID}" || return 1 check_json_field "${jwt}" "gateway" "${GATEWAY_ID}" || return 1 - check_json_field "${jwt}" "subdevice_ids" $(echo "${SUBDEVICE_IDS}"| tr -d '"') || return 1 + check_json_field "${jwt}" "subdevice_ids" "$(echo "${SUBDEVICE_IDS}"| tr -d '\"')" || return 1 check_device_token_scope "${jwt}" || return 1 check_vanilla_device_token_audience "${jwt}" || return 1 } @@ -180,7 +179,7 @@ check_refreshed_device_token_with_subcomponents2() { check_json_field "${jwt}" "azp" "device" || return 1 check_json_field "${jwt}" "device_id" "${DEVICE_ID}" || return 1 check_json_field "${jwt}" "gateway" "${GATEWAY_ID}" || return 1 - check_json_field "${jwt}" "subdevice_ids" $(echo "${SUBDEVICE_IDS2}"| tr -d '"') || return 1 + check_json_field "${jwt}" "subdevice_ids" "$(echo "${SUBDEVICE_IDS2}"| tr -d '\"')" || return 1 check_device_token_scope "${jwt}" || return 1 check_vanilla_device_token_audience "${jwt}" || return 1 }