we don't support groups in update -> edit conversions

This could probably be added, but perhaps not today.

diff --git a/src/tests/keywords/all.mk b/src/tests/keywords/all.mk
index eeb616c2ba951925b748bae9e3ebb3ddce405a9a..daf442dd70d0ad7e879979fc49122aae8e02d5b2 100644 (file)
--- a/src/tests/keywords/all.mk
+++ b/src/tests/keywords/all.mk
@@ -57,14 +57,13 @@ endif
 #  All of the "update" tests which should also be run with "-S rewrite_update=yes"
 #
 #  update-remove-index         Remove by index _and_ value.  Not implemented in edit code
-#  update-filter               lots of errors
 #
 
 #  Tests for the "update" keyword
-KEYWORD_UPDATE_TESTS :=   update-remove-index update-filter 
+KEYWORD_UPDATE_TESTS :=   update-remove-index 
 
 # Tests for rewriting "update"
-KEYWORD_UPDATE_REWRITE_TESTS := update-all update-array update-delete update-remove-any update-group update-hex update-remove-value update-index update-list-error update-remove-list update-prepend unknown-update update-error update-error-2 update-exec-error update-list-null-rhs update-exec update-error-3 update-group-error update-null-value-assign vendor-specific-error
+KEYWORD_UPDATE_REWRITE_TESTS := update-all update-array update-delete update-remove-any update-group update-hex update-remove-value update-index update-list-error update-remove-list update-prepend unknown-update update-error update-error-2 update-exec-error update-list-null-rhs update-exec update-error-3 update-group-error update-null-value-assign vendor-specific-error update-filter
 
 #
 #  Migration support.  Some of the tests don't run under the new

diff --git a/src/tests/keywords/update-group b/src/tests/keywords/update-group
index 03d891700419ed23fc81686aa6c463a6d29d5ed7..11923202d7575eaddae4c58894c0d6319be4a524 100644 (file)
--- a/src/tests/keywords/update-group
+++ b/src/tests/keywords/update-group
@@ -5,9 +5,13 @@ update reply {
        #
        #  Update the contents of a group
        #
-       &Tmp-Group-0 := {
+       &Tmp-Group-0 := {       # ERROR
                Tmp-String-0 = "foo"
        }
 }
 
+if !&reply.Tmp-Group-0.Tmp-String-0 {
+       test_fail
+}
+
 success