Skip indexing py_binary rules

Author: yushan8

gazelle/python/resolve.go

@@ -55,6 +55,9 @@ func (*Resolver) Name() string { return languageName }
 // If nil is returned, the rule will not be indexed. If any non-nil slice is
 // returned, including an empty slice, the rule will be indexed.
 func (py *Resolver) Imports(c *config.Config, r *rule.Rule, f *rule.File) []resolve.ImportSpec {
+	if r.Kind() == "py_binary" {
+		return nil
+	}
 	cfgs := c.Exts[languageName].(pythonconfig.Configs)
 	cfg := cfgs[f.Pkg]
 	srcs := r.AttrStrings("srcs")

gazelle/python/testdata/binary_without_entrypoint_per_file_generation/BUILD.out

@@ -19,10 +19,7 @@ py_library(
     name = "lib2",
     srcs = ["lib2.py"],
     visibility = ["//:__subpackages__"],
-    deps = [
-        ":lib",
-        ":lib_and_main",
-    ],
+    deps = [":lib"],
 )
 
 py_binary(

gazelle/python/testdata/binary_without_entrypoint_per_file_generation/lib2.py

@@ -1,2 +1 @@
 import lib
-import lib_and_main

gazelle/python/testdata/import_with_same_srcs_library_and_binary/BUILD.in

@@ -0,0 +1,3 @@
+# gazelle:python_extension enabled
+# gazelle:python_library_naming_convention py_default_library
+# gazelle:python_generation_mode package

gazelle/python/testdata/import_with_same_srcs_library_and_binary/BUILD.out

@@ -0,0 +1,3 @@
+# gazelle:python_extension enabled
+# gazelle:python_library_naming_convention py_default_library
+# gazelle:python_generation_mode package

gazelle/python/testdata/import_with_same_srcs_library_and_binary/README.md

@@ -0,0 +1,2 @@
+# Import with same srcs for library and binary
+This test case asserts a file with py_library and py_binary rules that include the same .py file in srcs will resolve to the py_library rule correctly.

gazelle/python/testdata/import_with_same_srcs_library_and_binary/WORKSPACE

@@ -0,0 +1 @@
+# This is a Bazel workspace for the Gazelle test data.

gazelle/python/testdata/import_with_same_srcs_library_and_binary/bar/BUILD.in

@@ -0,0 +1,8 @@
+load("@rules_python//python:defs.bzl", "py_library")
+
+py_library(
+    name = "bar",
+    srcs = ["bar.py"],
+    visibility = ["//:__subpackages__"],
+    deps = ["//foo"],
+)

gazelle/python/testdata/import_with_same_srcs_library_and_binary/bar/BUILD.out

@@ -0,0 +1,8 @@
+load("@rules_python//python:defs.bzl", "py_library")
+
+py_library(
+    name = "bar",
+    srcs = ["bar.py"],
+    visibility = ["//:__subpackages__"],
+    deps = ["//foo:py_default_library"],
+)

gazelle/python/testdata/import_with_same_srcs_library_and_binary/bar/bar.py

@@ -0,0 +1 @@
+import foo.script

gazelle/python/testdata/import_with_same_srcs_library_and_binary/foo/BUILD.in

@@ -0,0 +1,13 @@
+load("@rules_python//python:defs.bzl", "py_binary", "py_library")
+
+py_binary(
+    name = "script",
+    srcs = ["script.py"],
+    visibility = ["//:__subpackages__"],
+)
+
+py_library(
+    name = "py_default_library",
+    srcs = ["script.py"],
+    visibility = ["//:__subpackages__"],
+)

gazelle/python/testdata/import_with_same_srcs_library_and_binary/foo/BUILD.out

@@ -0,0 +1,3 @@
+
+if __name__ == "__main__":
+    print("Hello, world!")

gazelle/python/testdata/import_with_same_srcs_library_and_binary/foo/script.py

@@ -0,0 +1,17 @@
+# Copyright 2023 The Bazel Authors. All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+---
+expect:
+  exit_code: 0