sphinxdoc: move tests and docs to module-root directory

docs/BUILD.bazel

@@ -73,7 +73,7 @@ sphinx_docs(
         ":bzl_api_docs",
         ":py_api_srcs",
         ":py_runtime_pair",
-        "@sphinxdocs//sphinxdocs/docs:docs_lib",
+        "@sphinxdocs//docs:docs_lib",
     ],
 )
 

sphinxdocs/.bazelrc.deleted_packages

@@ -1 +1 @@
-common --deleted_packages=sphinxdocs/integration_tests/bcr
+common --deleted_packages=integration_tests/bcr

sphinxdocs/integration_tests/bcr/BUILD.bazel

@@ -0,0 +1,23 @@
+load("@bazel_skylib//rules:build_test.bzl", "build_test")
+load("@sphinxdocs//sphinxdocs:sphinx.bzl", "sphinx_build_binary", "sphinx_docs")
+
+sphinx_docs(
+    name = "docs",
+    srcs = ["index.md"],
+    config = "conf.py",
+    formats = ["html"],
+    sphinx = ":sphinx-build",
+)
+
+sphinx_build_binary(
+    name = "sphinx-build",
+    deps = [
+        "@dev_pip//myst_parser",
+        "@dev_pip//sphinx",
+    ],
+)
+
+build_test(
+    name = "docs_build_test",
+    targets = [":docs"],
+)

sphinxdocs/integration_tests/bcr/MODULE.bazel

@@ -0,0 +1,30 @@
+module(
+    name = "sphinxdocs_example",
+    version = "0.0.0",
+)
+
+bazel_dep(name = "sphinxdocs", version = "0.0.0")
+local_path_override(
+    module_name = "sphinxdocs",
+    path = "../..",
+)
+
+bazel_dep(name = "rules_python", version = "0.0.0")
+local_path_override(
+    module_name = "rules_python",
+    path = "../../..",
+)
+
+dev_pip = use_extension(
+    "@rules_python//python/extensions:pip.bzl",
+    "pip",
+    dev_dependency = True,
+)
+dev_pip.parse(
+    hub_name = "dev_pip",
+    python_version = "3.11",
+    requirements_lock = "@rules_python//docs:requirements.txt",
+)
+use_repo(dev_pip, "dev_pip")
+
+bazel_dep(name = "bazel_skylib", version = "1.8.2")

sphinxdocs/sphinxdocs/BUILD.bazel

@@ -17,7 +17,7 @@ load("@bazel_skylib//rules:common_settings.bzl", "bool_flag")
 load("//sphinxdocs/private:sphinx.bzl", "repeated_string_list_flag")
 
 package(
-    default_visibility = ["//sphinxdocs:__subpackages__"],
+    default_visibility = ["//:__subpackages__"],
 )
 
 # Additional -D values to add to every Sphinx build.
@@ -44,23 +44,27 @@ bool_flag(
 bzl_library(
     name = "sphinx_bzl",
     srcs = ["sphinx.bzl"],
+    visibility = ["//visibility:public"],
     deps = ["//sphinxdocs/private:sphinx_bzl"],
 )
 
 bzl_library(
     name = "sphinx_docs_library_bzl",
     srcs = ["sphinx_docs_library.bzl"],
+    visibility = ["//visibility:public"],
     deps = ["//sphinxdocs/private:sphinx_docs_library_macro_bzl"],
 )
 
 bzl_library(
     name = "sphinx_stardoc_bzl",
     srcs = ["sphinx_stardoc.bzl"],
+    visibility = ["//visibility:public"],
     deps = ["//sphinxdocs/private:sphinx_stardoc_bzl"],
 )
 
 bzl_library(
     name = "readthedocs_bzl",
     srcs = ["readthedocs.bzl"],
+    visibility = ["//visibility:public"],
     deps = ["//sphinxdocs/private:readthedocs_bzl"],
 )

sphinxdocs/sphinxdocs/private/BUILD.bazel

@@ -18,7 +18,7 @@ load("@rules_python//python:py_binary.bzl", "py_binary")
 load("@rules_python//python:py_library.bzl", "py_library")
 
 package(
-    default_visibility = ["//sphinxdocs:__subpackages__"],
+    default_visibility = ["//:__subpackages__"],
 )
 
 # These are only exported because they're passed as files to the @sphinxdocs

sphinxdocs/sphinxdocs/tests/proto_to_markdown/proto_to_markdown_test.py → sphinxdocs/tests/proto_to_markdown/proto_to_markdown_test.py

@@ -17,9 +17,8 @@
 
 from absl.testing import absltest
 from google.protobuf import text_format
-from stardoc.proto import stardoc_output_pb2
-
 from sphinxdocs.private import proto_to_markdown
+from stardoc.proto import stardoc_output_pb2
 
 _EVERYTHING_MODULE = """\
 module_docstring: "MODULE_DOC_STRING"

sphinxdocs/sphinxdocs/tests/sphinx_stardoc/sphinx_output_test.py → sphinxdocs/tests/sphinx_stardoc/sphinx_output_test.py

@@ -1,10 +1,9 @@
 import importlib.resources
 from xml.etree import ElementTree
 
+import tests.sphinx_stardoc as sphinx_stardoc
 from absl.testing import absltest, parameterized
 
-from sphinxdocs.tests import sphinx_stardoc
-
 
 class SphinxOutputTest(parameterized.TestCase):
     def setUp(self):