Fixed some spelling errors and test that failed on Bazel

lib/subpackages.bzl

@@ -14,13 +14,13 @@
 
 """Skylib module containing common functions for working with native.subpackages()
 """
-_SUBACKAGES_SUPPORTED = "subpackages" in dir(native)
+_SUBPACKAGES_SUPPORTED = hasattr(native, "subpackages")
 
 def _supported():
-    return _SUBACKAGES_SUPPORTED
+    return _SUBPACKAGES_SUPPORTED
 
 def _check_supported():
-    if not _SUBACKAGES_SUPPORTED:
+    if not _SUBPACKAGES_SUPPORTED:
         fail("native.subpackages not supported.")
 
 def _all(exclude = [], allow_empty = False, fully_qualified = True):
@@ -31,13 +31,13 @@ def _all(exclude = [], allow_empty = False, fully_qualified = True):
     Example:
     Assuming the following BUILD files exist:
 
-    BUILD
-    foo/BUILD
-    foo/sub/BUILD
-    bar/BUILD
-    baz/deep/dir/BUILD
+        BUILD
+        foo/BUILD
+        foo/sub/BUILD
+        bar/BUILD
+        baz/deep/dir/BUILD
 
-    If the current package is 'BUILD' all() will return ['//foo', '//bar',
+    If the current package is '//' all() will return ['//foo', '//bar',
     '//baz/deep/dir'].  //foo/sub is not included because it is a direct
     subpackage of '//foo' not '//'
 

tests/subpackages_tests.bzl

@@ -41,16 +41,19 @@ def _all_test(env):
         "write_file",
     ]
 
+    # subpackages is always in sorted order:
+    all_pkgs = sorted(all_pkgs)
+
     # test defaults
-    loadingtest.asserts(
+    loadingtest.equals(
         env,
         "all",
         all_pkgs,
         subpackages.all(fully_qualified = False),
     )
 
     # test exclusion
-    loadingtest.asserts(
+    loadingtest.equals(
         env,
         "all_w_exclude",
         filtered_pkgs,
@@ -59,8 +62,8 @@ def _all_test(env):
 
 def _exists_test(env):
     """Unit tests for subpackages.exists."""
-    loadingtest.asserts(env, "exists_yes", True, subpackages.exists("copy_file"))
-    loadingtest.asserts(env, "exists_no", False, subpackages.exists("never_existed"))
+    loadingtest.equals(env, "exists_yes", True, subpackages.exists("copy_file"))
+    loadingtest.equals(env, "exists_no", False, subpackages.exists("never_existed"))
 
 def subpackages_test_suite():
     """Creates the test targets and test suite for subpackages.bzl tests."""