Remove empty globs (#124)

* Remove empty globs
* Enable bazel disallow empty globs

Enable the option to support an early detection of invalid globs in bazel functions.

Author: simon-d-bmw

.bazelrc

@@ -10,3 +10,5 @@ build --incompatible_default_to_explicit_init_py
 
 common --registry=https://raw.githubusercontent.com/eclipse-score/bazel_registry/main/
 common --registry=https://bcr.bazel.build
+
+common --incompatible_disallow_empty_glob

docs.bzl

@@ -75,7 +75,7 @@ def docs(source_files_to_scan_for_needs_links = None, source_dir = "docs", conf_
         sphinx_build_binary(
             name = "sphinx_build" + suffix,
             visibility = ["//visibility:public"],
-            data = ["@score_docs_as_code//src:docs_assets", "@score_docs_as_code//src:score_extension_files"] + external_needs_deps,
+            data = ["@score_docs_as_code//src:docs_assets", "@score_docs_as_code//src:docs_as_code_py_modules"] + external_needs_deps,
             deps = sphinx_requirements + deps,
         )
         _incremental(
@@ -183,7 +183,7 @@ def _docs(name = "docs", suffix = "", format = "html", external_needs_deps = lis
             "**/*.json",
             "**/*.csv",
             "**/*.inc",
-        ], exclude = ["**/tests/*"]),
+        ], exclude = ["**/tests/*"], allow_empty = True),
         config = ":conf.py",
         extra_opts = [
             "-W",
@@ -205,7 +205,7 @@ def _docs(name = "docs", suffix = "", format = "html", external_needs_deps = lis
 
     native.filegroup(
         name = "assets" + target_suffix,
-        srcs = native.glob(["_assets/**"]),
+        srcs = native.glob(["_assets/**"], allow_empty = True),
         visibility = ["//visibility:public"],
     )
 

src/BUILD

@@ -118,21 +118,6 @@ py_library(
     visibility = ["//visibility:public"],
 )
 
-filegroup(
-    name = "score_extension_files",
-    srcs = glob(
-        [
-            "src/**",
-        ],
-        exclude = [
-            "**/test/**",
-            "**/tests/**",
-            "**/__pycache__/**",
-        ],
-    ) + [":docs_as_code_py_modules"],
-    visibility = ["//visibility:public"],
-)
-
 filegroup(
     name = "docs_assets",
     srcs = glob([
@@ -153,17 +138,6 @@ filegroup(
     visibility = ["//visibility:public"],
 )
 
-# Needed for 'test_rules_file_based'
-filegroup(
-    name = "test_rst_files",
-    srcs = glob([
-        "extensions/**/*.rst",
-        "extensions/**/*.py",
-        "conf.py",
-    ]),
-    visibility = ["//visibility:public"],
-)
-
 dash_license_checker(
     src = ":requirements_lock",
     file_type = "requirements",

src/extensions/score_metamodel/BUILD

@@ -33,7 +33,7 @@ score_py_pytest(
     size = "small",
     srcs = glob(["tests/*.py"]),
     # All requirements already in the library so no need to have it double
-    data = ["//src:test_rst_files"] + glob(
+    data = glob(
         ["tests/**/*.rst"],
     ),
     deps = [":score_metamodel"],

src/extensions/score_source_code_linker/BUILD

@@ -30,8 +30,7 @@ score_py_pytest(
     name = "score_source_code_linker_test",
     size = "small",
     srcs = glob([
-        "tests/**/*.py",
-        "test/**/*.json",
+        "tests/*.py",
     ]),
     args = [
         "-s",