ENT-13824: Error on rval call that are not builtin-functions inside vars-promise

Signed-off-by: Simon Halvorsen <simon.halvorsen@northern.tech>

Author: SimonThalvorsen

src/cfengine_cli/lint.py

@@ -515,6 +515,26 @@ def _lint_node(node: Node, policy_file: PolicyFile, state: State) -> int:
                 f"Error: Call to unknown function / bundle / body '{name}' {location}"
             )
             return 1
+        if (
+            name not in BUILTIN_FUNCTIONS
+            and state.promise_type == "vars"
+            and state.attribute_name not in ("action", "classes")
+        ):
+            _highlight_range(node, lines)
+            print(
+                f"Error: Call to unknown function '{name}' inside 'vars'-promise {location}"
+            )
+            return 1
+        if (
+            state.promise_type == "vars"
+            and state.attribute_name in ("action", "classes")
+            and qualified_name not in state.bodies
+        ):
+            _highlight_range(node, lines)
+            print(
+                f"Error: '{name}' is not a defined body. Only bodies may be called with '{state.attribute_name}' {location}"
+            )
+            return 1
     return 0
 
 

tests/lint/010_unknown_function_inside_vars.cf

@@ -0,0 +1,22 @@
+body classes bundle_class(a)
+{
+  scope => "bundle";
+  promise_kept => { "$(a):bundle_context" };
+}
+
+bundle agent target(a)
+{
+  reports:
+    "Hello, $(a)";
+}
+
+bundle agent helper
+{
+  vars:
+    "x" string => isvariable("arg");
+    "y"
+      classes => bundle_class("arg"),
+      string => "";
+  methods:
+    "x" usebundle => target("arg");
+}

tests/lint/010_unknown_function_inside_vars.expected.txt

@@ -0,0 +1,12 @@
+
+  vars:
+    "x" string => target("arg");
+                  ^----^
+Error: Call to unknown function 'target' inside 'vars'-promise at tests/lint/010_unknown_function_inside_vars.x.cf:16:19
+
+    "y"
+      classes => isvariable("arg"),
+                 ^--------^
+Error: 'isvariable' is not a defined body. Only bodies may be called with 'classes' at tests/lint/010_unknown_function_inside_vars.x.cf:18:18
+FAIL: tests/lint/010_unknown_function_inside_vars.x.cf (2 errors)
+Failure, 2 errors in total.

tests/lint/010_unknown_function_inside_vars.x.cf

@@ -0,0 +1,22 @@
+body classes bundle_class(a)
+{
+  scope => "bundle";
+  promise_kept => { "$(a):bundle_context" };
+}
+
+bundle agent target(a)
+{
+  reports:
+    "Hello, $(a)";
+}
+
+bundle agent helper
+{
+  vars:
+    "x" string => target("arg");
+    "y"
+      classes => isvariable("arg"),
+      string => "";
+  methods:
+    "x" usebundle => target("arg");
+}