more notes on what we don't want to convert

diff --git a/src/tests/keywords/all.mk b/src/tests/keywords/all.mk
index 74cafdb8473dea099f58c21e35270d084d1a2c09..8bb48ca59c346d3f5060b67676d3fb0dba092a98 100644 (file)
--- a/src/tests/keywords/all.mk
+++ b/src/tests/keywords/all.mk
@@ -54,23 +54,25 @@ endif
        @cp $$< $$@
 
 #
-#  All of the "update" tests which should also be run with "-S rewrite_update=yes"
+#  Tests which don't work in the update -> edit conversion
 #
+#  update-to-edit              Conversion doesn't parse &request.[*].
 #  update-remove-index         Remove by index _and_ value.  Not implemented in edit code
-#  update-group-*              subsections are not implemented in compile_update_to_edit()
+#  update-error-*              the errors are different, and we don't really care.
+#  vendor-specific-error       tries to parse RHS string as pair list.  Which is OK in the new code
 #
 
 #  Tests for the "update" keyword
-#KEYWORD_UPDATE_TESTS :=   update-remove-index 
+KEYWORD_UPDATE_TESTS :=   update-remove-index update-to-edit vendor-specific-error
 
 # 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 update-filter
+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 update-filter xlat-unknown
 
 #
 #  Migration support.  Some of the tests don't run under the new
 #  conditions, so we don't run them under the new conditions.
 #
-ifneq "$(findstring ${1}, update-to-edit $(KEYWORD_UPDATE_TESTS) xlat-unknown )" ""
+ifneq "$(findstring ${1}, $(KEYWORD_UPDATE_TESTS) )" ""
 $(OUTPUT)/${1}: NEW_COND=
 
 else ifneq "$(findstring ${1}, $(KEYWORD_UPDATE_REWRITE_TESTS))" ""

diff --git a/src/tests/keywords/update-group b/src/tests/keywords/update-group
index 11923202d7575eaddae4c58894c0d6319be4a524..4299cf70c1fe08391c7a068d0a3abe9b59f8d5f4 100644 (file)
--- a/src/tests/keywords/update-group
+++ b/src/tests/keywords/update-group
@@ -5,6 +5,8 @@ update reply {
        #
        #  Update the contents of a group
        #
+       #  This was parsed in the old edit code, but never did anything!
+       #
        &Tmp-Group-0 := {       # ERROR
                Tmp-String-0 = "foo"
        }