commit c6e437a20744746021325c024ad573acf6168b85
author milind-u <milind.upadhyay@gmail.com> Wed Mar 01 23:40:40 2023 -0800
committer Milind Upadhyay <milind.upadhyay@gmail.com> Thu Mar 02 20:45:34 2023 -0700
tree 1b1be58a5b43e93ebcf9a4d0c9d0481230004d35
parent ccf8b4a0f5dab5a5695c181f10e4e854e6b28f51

Add config validator to y2023

Doesn't catch every error, but still worth having here so we keep adding
it in future years.

Signed-off-by: milind-u <milind.upadhyay@gmail.com>
Change-Id: I78703b74aa8f80eed1840d78a1f84478e2f60f14

aos/util/config_validator_macro.bzl

diff --git a/aos/util/config_validator_macro.bzl b/aos/util/config_validator_macro.bzl
index 47ab240..453c5c2 100644
--- a/aos/util/config_validator_macro.bzl
+++ b/aos/util/config_validator_macro.bzl
@@ -1,4 +1,4 @@
-def config_validator_rule(name, config, visibility = None):
+def config_validator_rule(name, config, extension = ".bfbs", visibility = None):
     '''
     Macro to take a config and pass it to the config validator to validate that it will work on a real system.
 
@@ -8,12 +8,12 @@
         name: name that the config validator uses, e.g. "test_config",
         config: config rule that needs to be validated, e.g. "//aos/events:pingpong_config",
     '''
-    config_bfbs = config + ".bfbs"
+    config_file = config + extension
     native.genrule(
         name = name,
         outs = [name + ".txt"],
-        cmd = "$(location //aos/util:config_validator) --config $(location %s) > $@" % config_bfbs,
-        srcs = [config_bfbs],
+        cmd = "$(location //aos/util:config_validator) --config $(location %s) > $@" % config_file,
+        srcs = [config_file],
         tools = ["//aos/util:config_validator"],
         testonly = True,
         visibility = visibility,