Skip to content

Commit

Permalink
fix CI assertions
Browse files Browse the repository at this point in the history
  • Loading branch information
ntemmar committed Sep 14, 2022
1 parent abca4d2 commit fb87e1d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion common.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ local add_jdk_version(name) =
local version = find_first_integer(jdk.version);
// santity check that the parsed version is also included in the name
assert std.length(std.findSubstr(std.toString(version), name)) == 1 : "Cannot find version %d in name %s" % [version, name];
{ jdk_version:: version}
{ jdk_version:: version }
;

{
Expand Down
4 changes: 2 additions & 2 deletions vm/ci_common/common.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,12 @@ local common_json = composable(import '../../common.json');
local devkits = common_json.devkits;

{
verify_name(build): build + {
verify_name(build): {
expected_prefix:: std.join('-', [target for target in build.targets if target != "mach5"]) + '-vm',
expected_suffix:: build.os + '-' + build.arch,
assert std.startsWith(build.name, self.expected_prefix) : "'%s' is defined in '%s' with '%s' targets but does not start with '%s'" % [build.name, build.defined_in, build.targets, self.expected_prefix],
assert std.endsWith(build.name, self.expected_suffix) : "'%s' is defined in '%s' with os/arch '%s/%s' but does not end with '%s'" % [build.name, build.defined_in, build.os, build.arch, self.expected_suffix],
},
} + build,

vm_env_mixin(shortverison): {
local jdk = self.downloads.JAVA_HOME,
Expand Down

0 comments on commit fb87e1d

Please sign in to comment.