summaryrefslogtreecommitdiffstats
path: root/test/parser
diff options
context:
space:
mode:
authorAlan Dunn <amdunn@google.com>2016-03-02 08:59:02 -0800
committerSam Mendoza-Jonas <sam@mendozajonas.com>2016-03-15 14:10:22 +1100
commitc19daa69bc15e73b7ebf79d5280ba62c86c20b6f (patch)
tree2cb1d9620a3e8d96fc3d3285b568e0e126591c98 /test/parser
parent939660528bf1568c55b6dcf982cc9020c1dbcdd2 (diff)
downloadtalos-petitboot-c19daa69bc15e73b7ebf79d5280ba62c86c20b6f.tar.gz
talos-petitboot-c19daa69bc15e73b7ebf79d5280ba62c86c20b6f.zip
discover/grub2: Fix handling of positional parameters
Positional parameters are set in the environment with '$' prepended to the name. This causes lookups to fail because parameter lookups don't include the '$'. TESTED: Added a test that covers positional parameters in GRUB2 parser. Build succeeds, tests pass. Bootstrapped-by: Nancy Yuen <yuenn@google.com> Signed-off-by: Alan Dunn <amdunn@google.com> Signed-off-by: Sam Mendoza-Jonas <sam@mendozajonas.com>
Diffstat (limited to 'test/parser')
-rw-r--r--test/parser/Makefile.am1
-rw-r--r--test/parser/test-grub2-pos-param.c35
2 files changed, 36 insertions, 0 deletions
diff --git a/test/parser/Makefile.am b/test/parser/Makefile.am
index e4f9b9c..9a6be6d 100644
--- a/test/parser/Makefile.am
+++ b/test/parser/Makefile.am
@@ -26,6 +26,7 @@ parser_TESTS = \
test/parser/test-grub2-multiple-resolve \
test/parser/test-grub2-multiple-id \
test/parser/test-grub2-single-line-if \
+ test/parser/test-grub2-pos-param \
test/parser/test-grub2-load-env \
test/parser/test-grub2-save-env \
test/parser/test-grub2-save-env-dash-f \
diff --git a/test/parser/test-grub2-pos-param.c b/test/parser/test-grub2-pos-param.c
new file mode 100644
index 0000000..09ac8fc
--- /dev/null
+++ b/test/parser/test-grub2-pos-param.c
@@ -0,0 +1,35 @@
+
+#include "parser-test.h"
+
+#if 0 /* PARSER_EMBEDDED_CONFIG */
+
+function set_params_var {
+ params="$1 $2"
+}
+menuentry 'Linux' {
+ set_params_var abc 123
+ linux test_kernel $params
+}
+
+#endif
+
+void run_test(struct parser_test *test)
+{
+ struct discover_boot_option *opt;
+ struct discover_context *ctx;
+
+ test_read_conf_embedded(test, "/boot/grub/grub.cfg");
+
+ test_run_parser(test, "grub2");
+
+ ctx = test->ctx;
+
+ check_boot_option_count(ctx, 1);
+ opt = get_boot_option(ctx, 0);
+
+ check_name(opt, "Linux");
+ /* The linux command, $params is expected to have been set when
+ * set_params was called in menuentry.
+ */
+ check_args(opt, "abc 123");
+}
OpenPOWER on IntegriCloud