From 083ff26ceda3676ba8c32259b99f0cb3cc6c3bd4 Mon Sep 17 00:00:00 2001 From: Alex Eagle Date: Fri, 20 Jan 2023 14:20:10 -0800 Subject: [PATCH] feat: add assert helper for zip contents Helps to make sure that Python wheels are correctly constructed, as one example. --- docs/testing.md | 22 ++++++++++++++++++++++ lib/testing.bzl | 50 +++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 72 insertions(+) diff --git a/docs/testing.md b/docs/testing.md index ff4ff04..a6ffca7 100644 --- a/docs/testing.md +++ b/docs/testing.md @@ -2,6 +2,28 @@ Helpers for making test assertions + + +## assert_archive_contains + +
+assert_archive_contains(name, archive, expected, type, kwargs)
+
+ +Assert that an archive file contains at least the given file entries. + +**PARAMETERS** + + +| Name | Description | Default Value | +| :------------- | :------------- | :------------- | +| name | name of the resulting sh_test target | none | +| archive | Label of the the .tar or .zip file | none | +| expected | Label of a file containing a (partial) file listing | none | +| type | "tar" or "zip" | "zip" | +| kwargs | additional named arguments for the resulting sh_test | none | + + ## assert_contains diff --git a/lib/testing.bzl b/lib/testing.bzl index 7acf67b..a1de050 100644 --- a/lib/testing.bzl +++ b/lib/testing.bzl @@ -113,3 +113,53 @@ def assert_json_matches(name, file1, file2, filter1 = ".", filter2 = "."): file2, ), ) + +def assert_archive_contains(name, archive, expected, type = "zip", **kwargs): + """Assert that an archive file contains at least the given file entries. + + Args: + name: name of the resulting sh_test target + archive: Label of the the .tar or .zip file + expected: Label of a file containing a (partial) file listing + type: "tar" or "zip" + **kwargs: additional named arguments for the resulting sh_test + """ + + if not type in ["tar", "zip"]: + fail("type must be 'tar' or 'zip'") + + # Command to list the files in the archive + command = "zipinfo -1" if type == "zip" else "tar -tf" + + # -f $actual: use this file to contain one pattern per line + # -F: treat each pattern as a plain string, not a regex + # -x: match whole lines only + # -v: only print lines which don't match + grep = "grep -F -x -v -f $actual" + + script_name = "_gen_assert_" + name + + write_file( + name = script_name, + out = "assert_{}.sh".format(name), + content = [ + "#!/usr/bin/env bash", + "actual=$(mktemp)", + "{} $1 > $actual".format(command), + "# Grep exits 1 if no matches, which is success for this test.", + "if {} $2; then".format(grep), + " echo", + " echo 'ERROR: above line(s) appeared in {} but are not present in the archive' $1".format(expected), + " exit 1", + "fi", + ], + ) + + native.sh_test( + name = name, + srcs = [script_name], + args = ["$(rootpath %s)" % archive, "$(rootpath %s)" % expected], + data = [archive, expected], + timeout = "short", + **kwargs + )