Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup of writeback tests in prep for underlying refactoring #538

Merged
merged 7 commits into from
Dec 12, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Empty file removed add
Empty file.
7 changes: 5 additions & 2 deletions bin/test_itemized
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
#!/bin/bash -e

ROOT_DIR=$(dirname $0)/..
cd $ROOT_DIR

if [[ $# != 1 ]]; then
echo Usage: $0 PROJECT_ID
false
Expand All @@ -10,8 +13,8 @@ shift 1
SITE_PATH=sites/udmi_site_model
DEVICE_ID=AHU-1

PUBBER_OUT=/tmp/pubber.log
SEQUENCER_OUT=/tmp/sequencer.log
PUBBER_OUT=pubber.out
SEQUENCER_OUT=sequencer.out
RESULTS_OUT=/tmp/test_itemized.out
GOLDEN_FILE=etc/test_itemized.out

Expand Down
23 changes: 9 additions & 14 deletions docs/specs/sequences/generated.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,9 @@ Some caveats:
* [system_min_loglevel](#system_min_loglevel): Check that the min log-level config is honored by the device.
* [system_mode_restart](#system_mode_restart): Restart and connect to same endpoint and expect it returns.
* [valid_serial_no](#valid_serial_no)
* [writeback_failure_state](#writeback_failure_state)
* [writeback_invalid_state](#writeback_invalid_state)
* [writeback_success_apply](#writeback_success_apply)
* [writeback_success_state](#writeback_success_state)
* [writeback_failure](#writeback_failure)
* [writeback_invalid](#writeback_invalid)
* [writeback_success](#writeback_success)

## broken_config

Expand Down Expand Up @@ -174,30 +173,26 @@ Restart and connect to same endpoint and expect it returns.

1. Wait for received serial no matches

## writeback_failure_state
## writeback_failure

1. Wait for point filter_alarm_pressure_status to have value_state default (null)
1. Update config before point filter_alarm_pressure_status to have value_state failure:
* Add `pointset.points.filter_alarm_pressure_status.set_value` = `false`
1. Wait for point filter_alarm_pressure_status to have value_state failure

## writeback_invalid_state
## writeback_invalid

1. Wait for point filter_differential_pressure_sensor to have value_state default (null)
1. Update config before point filter_differential_pressure_sensor to have value_state invalid:
* Add `pointset.points.filter_differential_pressure_sensor.set_value` = `15`
1. Wait for point filter_differential_pressure_sensor to have value_state invalid

## writeback_success_apply

1. Update config before point `filter_differential_pressure_setpoint` to have present_value `60`:
* Set `pointset.points.filter_differential_pressure_setpoint.set_value` = `60`
1. Wait for point `filter_differential_pressure_setpoint` to have present_value `60`

## writeback_success_state
## writeback_success

1. Update config before point filter_differential_pressure_setpoint to have value_state default (null):
* Remove `pointset.points.filter_differential_pressure_setpoint.set_value`
1. Wait for point filter_differential_pressure_setpoint to have value_state default (null)
1. Update config before point filter_differential_pressure_setpoint to have value_state applied:
* Set `pointset.points.filter_differential_pressure_setpoint.set_value` = `60`
* Add `pointset.points.filter_differential_pressure_setpoint.set_value` = `60`
1. Wait for point filter_differential_pressure_setpoint to have value_state applied
1. Wait for point `filter_differential_pressure_setpoint` to have present_value `60`
7 changes: 3 additions & 4 deletions etc/sequencer.out
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ RESULT pass valid_serial_no Sequence complete
RESULT pass valid_serial_no Sequence complete
RESULT pass valid_serial_no Sequence complete
RESULT pass valid_serial_no Sequence complete
RESULT pass writeback_failure_state Sequence complete
RESULT pass writeback_invalid_state Sequence complete
RESULT pass writeback_success_apply Sequence complete
RESULT pass writeback_success_state Sequence complete
RESULT pass writeback_failure Sequence complete
RESULT pass writeback_invalid Sequence complete
RESULT pass writeback_success Sequence complete
4 changes: 2 additions & 2 deletions etc/test_itemized.out
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
TEST writeback_success_state noWriteback RESULT fail writeback_success_state timeout nothing
TEST writeback_success_apply noPointState RESULT pass writeback_success_apply Sequence complete
TEST writeback_success noWriteback RESULT fail writeback_success timeout nothing
TEST writeback_success noPointState RESULT fail writeback_success timeout nothing
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@

## writeback_failure_state
## writeback_failure

1. Wait for point filter_alarm_pressure_status to have value_state default (null)
1. Update config before point filter_alarm_pressure_status to have value_state failure:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@

## writeback_invalid_state
## writeback_invalid

1. Wait for point filter_differential_pressure_sensor to have value_state default (null)
1. Update config before point filter_differential_pressure_sensor to have value_state invalid:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@

## writeback_success_state
## writeback_success

1. Update config before point filter_differential_pressure_setpoint to have value_state default (null):
* Remove `pointset.points.filter_differential_pressure_setpoint.set_value`
1. Wait for point filter_differential_pressure_setpoint to have value_state default (null)
1. Update config before point filter_differential_pressure_setpoint to have value_state applied:
* Set `pointset.points.filter_differential_pressure_setpoint.set_value` = `60`
* Add `pointset.points.filter_differential_pressure_setpoint.set_value` = `60`
1. Wait for point filter_differential_pressure_setpoint to have value_state applied
1. Wait for point `filter_differential_pressure_setpoint` to have present_value `60`
15 changes: 0 additions & 15 deletions validator/sequences/writeback_success_apply/config_pointset.json

This file was deleted.

10 changes: 0 additions & 10 deletions validator/sequences/writeback_success_apply/config_system.json

This file was deleted.

25 changes: 0 additions & 25 deletions validator/sequences/writeback_success_apply/config_update.json

This file was deleted.

15 changes: 0 additions & 15 deletions validator/sequences/writeback_success_apply/event_pointset.json

This file was deleted.

11 changes: 0 additions & 11 deletions validator/sequences/writeback_success_apply/event_system.json

This file was deleted.

8 changes: 0 additions & 8 deletions validator/sequences/writeback_success_apply/local_system.json

This file was deleted.

25 changes: 0 additions & 25 deletions validator/sequences/writeback_success_apply/local_update.json

This file was deleted.

6 changes: 0 additions & 6 deletions validator/sequences/writeback_success_apply/sequence.md

This file was deleted.

Loading