Fix buildifier lint warnings

Author: fmeum

CHANGELOG.md

@@ -11,6 +11,8 @@
     ([#52](https://github.com/bazelbuild/rules_testing/issues/52))
   * StructSubject for asserting arbitrary structs.
     ([#53](https://github.com/bazelbuild/rules_testing/issues/53))
+  * CollectionSubject now supports tuples.
+    ([#69](https://github.com/bazelbuild/rules_testing/pull/69))
   * (docs) Created human-friendly changelog
 
 ## [0.3.0] - 2023-07-06

docgen/docgen.bzl

@@ -14,8 +14,8 @@
 
 """Rules to help generate rules_testing docs."""
 
-load("@io_bazel_stardoc//stardoc:stardoc.bzl", "stardoc")
 load("@bazel_skylib//rules:build_test.bzl", "build_test")
+load("@io_bazel_stardoc//stardoc:stardoc.bzl", "stardoc")
 
 def sphinx_stardocs(name, bzl_libraries, **kwargs):
     """Generate Sphinx-friendly markdown docs using Stardoc for bzl libraries.

lib/private/default_info_subject.bzl

@@ -14,9 +14,9 @@
 
 """# DefaultInfoSubject"""
 
-load(":runfiles_subject.bzl", "RunfilesSubject")
 load(":depset_file_subject.bzl", "DepsetFileSubject")
 load(":file_subject.bzl", "FileSubject")
+load(":runfiles_subject.bzl", "RunfilesSubject")
 
 def _default_info_subject_new(info, *, meta):
     """Creates a `DefaultInfoSubject`

lib/test_suite.bzl

@@ -3,8 +3,8 @@
 Aggregates multiple Starlark tests in a single test_suite.
 """
 
-load("//lib/private:util.bzl", "get_test_name_from_function")
 load("//lib:unit_test.bzl", "unit_test")
+load("//lib/private:util.bzl", "get_test_name_from_function")
 
 def test_suite(name, *, tests = [], basic_tests = [], test_kwargs = {}):
     """Instantiates given test macros/implementations and gathers their main targets into a `test_suite`.

lib/truth.bzl

@@ -51,11 +51,11 @@ load("//lib/private:expect.bzl", "Expect")
 load("//lib/private:file_subject.bzl", "FileSubject")
 load("//lib/private:int_subject.bzl", "IntSubject")
 load("//lib/private:label_subject.bzl", "LabelSubject")
+load("//lib/private:matching.bzl", _matching = "matching")
 load("//lib/private:runfiles_subject.bzl", "RunfilesSubject")
 load("//lib/private:str_subject.bzl", "StrSubject")
-load("//lib/private:target_subject.bzl", "TargetSubject")
-load("//lib/private:matching.bzl", _matching = "matching")
 load("//lib/private:struct_subject.bzl", "StructSubject")
+load("//lib/private:target_subject.bzl", "TargetSubject")
 
 # Rather than load many symbols, just load this symbol, and then all the
 # asserts will be available.

tests/struct_subject/struct_subject_tests.bzl

@@ -14,8 +14,8 @@
 
 """Tests for StructSubject"""
 
-load("//lib:truth.bzl", "subjects")
 load("//lib:test_suite.bzl", "test_suite")
+load("//lib:truth.bzl", "subjects")
 load("//tests:test_util.bzl", "test_util")
 
 _tests = []

tests/test_util.bzl

@@ -14,9 +14,10 @@
 
 """Utilities for testing rules_testing code."""
 
+load("//lib:truth.bzl", "matching")
+
 # buildifier: disable=bzl-visibility
 load("//lib/private:expect_meta.bzl", "ExpectMeta")
-load("//lib:truth.bzl", "matching")
 
 def _fake_meta(real_env):
     """Create a fake ExpectMeta object for testing.

tests/truth_tests.bzl

@@ -15,8 +15,8 @@
 """Tests for truth.bzl."""
 
 load("@bazel_skylib//lib:unittest.bzl", ut_asserts = "asserts")
-load("//lib:truth.bzl", "matching", "subjects", "truth")
 load("//lib:analysis_test.bzl", "analysis_test", "test_suite")
+load("//lib:truth.bzl", "matching", "subjects", "truth")
 
 # Bazel 5 has a bug where every access of testing.ExecutionInfo is a new
 # object that isn't equal to itself. This is fixed in Bazel 6.

tests/unit_test_tests.bzl

@@ -1,7 +1,7 @@
 """Tests for unit_test."""
 
-load("//lib:unit_test.bzl", "unit_test")
 load("//lib:test_suite.bzl", "test_suite")
+load("//lib:unit_test.bzl", "unit_test")
 
 def _test_basic(env):
     _ = env  # @unused