Minor formatting changes plus doc updates
diff --git a/docs/analysis_test_doc.md b/docs/analysis_test_doc.md index 762a82a..817be66 100755 --- a/docs/analysis_test_doc.md +++ b/docs/analysis_test_doc.md
@@ -31,7 +31,6 @@ Args: name: The name of the test rule. targets: A list of targets to ensure build. - ### Attributes
diff --git a/docs/diff_test_doc.md b/docs/diff_test_doc.md index ec29077..07ce8b9 100755 --- a/docs/diff_test_doc.md +++ b/docs/diff_test_doc.md
@@ -1,14 +1,12 @@ ## diff_test <pre> -diff_test(<a href="#diff_test-name">name</a>, <a href="#diff_test-file1">file1</a>, <a href="#diff_test-file2">file2</a>, <a href="#diff_test-expect_same">expect_same</a>, <a href="#diff_test-kwargs">kwargs</a>) +diff_test(<a href="#diff_test-name">name</a>, <a href="#diff_test-file1">file1</a>, <a href="#diff_test-file2">file2</a>, <a href="#diff_test-kwargs">kwargs</a>) </pre> -A test that compares the contents of two files. +A test that compares two files. -The test succeeds when the files are expected to be the same (with regard to -file contents) and are in fact the same, or when the files are expected to -be different and are in fact so. +The test succeeds if the files' contents match. ### Parameters @@ -46,17 +44,6 @@ </p> </td> </tr> - <tr id="diff_test-expect_same"> - <td><code>expect_same</code></td> - <td> - optional. default is <code>True</code> - <p> - Whether the files are expected to be the same or not. The - test passes if this is True and the files are the same, or if this is - False and the files are not the same. - </p> - </td> - </tr> <tr id="diff_test-kwargs"> <td><code>kwargs</code></td> <td>
diff --git a/docs/unittest_doc.md b/docs/unittest_doc.md index 2256d12..4fbd96f 100755 --- a/docs/unittest_doc.md +++ b/docs/unittest_doc.md
@@ -55,7 +55,7 @@ ## analysistest.make <pre> -analysistest.make(<a href="#analysistest.make-impl">impl</a>, <a href="#analysistest.make-expect_failure">expect_failure</a>, <a href="#analysistest.make-config_settings">config_settings</a>) +analysistest.make(<a href="#analysistest.make-impl">impl</a>, <a href="#analysistest.make-expect_failure">expect_failure</a>, <a href="#analysistest.make-attrs">attrs</a>, <a href="#analysistest.make-config_settings">config_settings</a>) </pre> Creates an analysis test rule from its implementation function. @@ -111,6 +111,16 @@ </p> </td> </tr> + <tr id="analysistest.make-attrs"> + <td><code>attrs</code></td> + <td> + optional. default is <code>{}</code> + <p> + An optional dictionary to supplement the attrs passed to the + unit test's `rule()` constructor. + </p> + </td> + </tr> <tr id="analysistest.make-config_settings"> <td><code>config_settings</code></td> <td> @@ -663,7 +673,7 @@ <tr id="unittest.make-attrs"> <td><code>attrs</code></td> <td> - optional. default is <code>None</code> + optional. default is <code>{}</code> <p> An optional dictionary to supplement the attrs passed to the unit test's `rule()` constructor.
diff --git a/rules/analysis_test.bzl b/rules/analysis_test.bzl index a43ef90..46d32dd 100644 --- a/rules/analysis_test.bzl +++ b/rules/analysis_test.bzl
@@ -52,6 +52,5 @@ Args: name: The name of the test rule. - targets: A list of targets to ensure build. - """, + targets: A list of targets to ensure build.""", )
diff --git a/tests/analysis_test_test.sh b/tests/analysis_test_test.sh index 5dc52b8..db9bbac 100755 --- a/tests/analysis_test_test.sh +++ b/tests/analysis_test_test.sh
@@ -15,7 +15,7 @@ # limitations under the License. # # End to end tests for analysis_test.bzl. -# +# # End to end tests of analysis_test.bzl cover verification that # analysis_test tests fail when their underlying test targets fail analysis.
diff --git a/tests/unittest_test.sh b/tests/unittest_test.sh index 5b9d43f..22239b5 100755 --- a/tests/unittest_test.sh +++ b/tests/unittest_test.sh
@@ -15,7 +15,7 @@ # limitations under the License. # # End to end tests for unittest.bzl. -# +# # Specifically, end to end tests of unittest.bzl cover verification that # analysis-phase tests written with unittest.bzl appropriately # cause test failures in cases where violated assertions are made.