Remove binary artifacts due to security policy violation

test/BUILD

@@ -1,5 +1,5 @@
 load("@rules_python//python:defs.bzl", "py_binary")
-load("@rules_java//java:defs.bzl", "java_binary", "java_import", "java_library")
+load("@rules_java//java:defs.bzl", "java_binary", "java_library")
 load(
     "//scala:scala.bzl",
     "scala_binary",
@@ -616,9 +616,13 @@ scala_library(
 )
 
 # make sure making a fat jar strips signatures
-java_import(
+java_library(
     name = "fakejar",
-    jars = ["fake_sig.jar"],
+    resources = [
+        "//test/src/main/resources/META-INF:FAKE.DSA",
+        "//test/src/main/resources/META-INF:FAKE.RSA",
+    ],
+    visibility = ["//visibility:public"],
 )
 
 scala_binary(

test/example_jars/BUILD

@@ -1,17 +1,17 @@
-load("@rules_java//java:defs.bzl", "java_import")
+load("@rules_java//java:defs.bzl", "java_library")
 
-java_import(
+java_library(
     name = "example_jar1",
-    jars = [
-        "example_jar1.jar",
+    resources = [
+        "//test/src/main/resources/META-INF/services:org.apache.beam.sdk.io.FileSystemRegistrar",
     ],
     visibility = ["//visibility:public"],
 )
 
-java_import(
+java_library(
     name = "example_jar2",
-    jars = [
-        "example_jar2.jar",
+    resources = [
+        "//test/src/main/resources/META-INF/services:org.apache.beam.sdk.io.FileSystemRegistrar",
     ],
     visibility = ["//visibility:public"],
 )

test/example_jars/expected_service_manifest.txt

@@ -1,2 +1,2 @@
 org.apache.beam.sdk.extensions.gcp.storage.GcsFileSystemRegistrar
-org.apache.beam.sdk.io.LocalFileSystemRegistrar
+org.apache.beam.sdk.extensions.gcp.storage.GcsFileSystemRegistrar

test/src/main/resources/META-INF/BUILD

@@ -0,0 +1,4 @@
+exports_files([
+    "FAKE.DSA",
+    "FAKE.RSA",
+])

test/src/main/resources/META-INF/services/BUILD

@@ -0,0 +1,3 @@
+exports_files([
+    "org.apache.beam.sdk.io.FileSystemRegistrar",
+])

test/src/main/resources/META-INF/services/org.apache.beam.sdk.io.FileSystemRegistrar

@@ -0,0 +1 @@
+org.apache.beam.sdk.extensions.gcp.storage.GcsFileSystemRegistrar

test/src/main/resources/scalarules/test/BUILD

@@ -1,9 +1,11 @@
 exports_files([
     "byes",
     "hellos",
-    "hellos-and-byes.jar",
     "more-byes",
     "more-hellos",
+    "Foo.thrift",
+    "Bar.thrift",
+    "Baz.thrift",
 ])
 
 genrule(

test/src/main/resources/scalarules/test/Bar.thrift

@@ -0,0 +1,5 @@
+namespace java com.foo.bar.baz
+
+struct Bar {
+  1: optional string Bar
+}

test/src/main/resources/scalarules/test/Baz.thrift

@@ -0,0 +1,5 @@
+namespace java com.foo.bar.baz
+
+struct Baz {
+  1: optional string Baz
+}

test/src/main/resources/scalarules/test/Foo.thrift

@@ -0,0 +1,5 @@
+namespace java com.foo.bar.baz
+
+struct Foo {
+  1: optional string foo
+}

test/src/main/scala/scalarules/test/resource_jars/BUILD

@@ -1,7 +1,19 @@
+load("@rules_java//java:defs.bzl", "java_library")
 load("//scala:scala.bzl", "scala_test")
 
+java_library(
+    name = "jarWithResources",
+    resources = [
+        "//test/src/main/resources/scalarules/test:byes",
+        "//test/src/main/resources/scalarules/test:hellos",
+        "//test/src/main/resources/scalarules/test:more-byes",
+        "//test/src/main/resources/scalarules/test:more-hellos",
+    ],
+    visibility = ["//visibility:public"],
+)
+
 scala_test(
     name = "resource_jars",
     srcs = ["TestResourceJars.scala"],
-    resource_jars = ["//test/src/main/resources/scalarules/test:hellos-and-byes.jar"],
+    resource_jars = [":jarWithResources"],
 )

test/src/main/scala/scalarules/test/resource_jars/TestResourceJars.scala

@@ -15,7 +15,7 @@ class TestResourceJars extends AnyFlatSpec {
     )
     expectedSubstrings.foreach {
       case (resource_name, substring) => {
-        val stream = getClass.getResourceAsStream("/" + resource_name)
+        val stream = getClass.getResourceAsStream("/scalarules/test/" + resource_name)
         assert(stream != null, s"failed to find resource $resource_name")
         val content = Source.fromInputStream(stream).getLines().mkString("\n")
         assert(content.contains(substring), s"resource $resource_name did not contain substring $substring")

test/src/main/scala/scalarules/test/scala_import/BUILD

@@ -20,11 +20,17 @@ scala_import(
     ],
 )
 
-# Jars as files
+scala_library(
+    name = "generic_scalalib",
+    testonly = True,
+    srcs = ["generic/Generic.scala"],
+    visibility = ["//visibility:private"],
+)
+
 scala_import(
-    name = "relate",
+    name = "imported_genericlib_jar",
     jars = [
-        "relate_2.11-2.1.1.jar",
+        ":generic_scalalib.jar",
     ],
 )
 
@@ -41,7 +47,7 @@ scala_specs2_junit_test(
     size = "small",
     srcs = ["ScalaImportExposesFileJarsTest.scala"],
     suffixes = ["Test"],
-    deps = [":relate"],
+    deps = [":imported_genericlib_jar"],
 )
 
 scala_library(
@@ -123,15 +129,15 @@ scala_specs2_junit_test(
 # https://github.com/bazelbuild/rules_scala/issues/1188
 scala_import(
     name = "stamping_first_time",
-    jars = ["relate_2.11-2.1.1.jar"],
+    jars = ["//test/src/main/scala/scalarules/test/scala_import/nl:ScalaImportNeverLink"],
 )
 
 scala_import(
     name = "stamping_second_time",
-    jars = ["relate_2.11-2.1.1.jar"],
+    jars = ["//test/src/main/scala/scalarules/test/scala_import/nl:ScalaImportNeverLink"],
 )
 
 scala_import_stamping_test_suite(
     name = "stamping_tests",
-    jar = "relate_2.11-2.1.1.jar",
+    jar = "//test/src/main/scala/scalarules/test/scala_import/nl:ScalaImportNeverLink",
 )

test/src/main/scala/scalarules/test/scala_import/ScalaImportExposesFileJarsTest.scala

@@ -1,13 +1,14 @@
 package scalarules.test.scala_import
 
+import  scalarules.test.scala_import.generic.Generic
+
 import org.specs2.mutable.SpecificationWithJUnit
-import com.lucidchart.relate.SqlRow
 
 class ScalaImportExposesJarsTest extends SpecificationWithJUnit {
 
   "scala_import" >> {
     "enable importing jars from files" in {
-      println(classOf[SqlRow])
+      assert(Generic.foo == "bar")
       success
     }
   }

test/src/main/scala/scalarules/test/scala_import/generic/Generic.scala

@@ -0,0 +1,5 @@
+package scalarules.test.scala_import.generic
+
+object Generic {
+  val foo = "bar"
+}

test/src/main/scala/scalarules/test/scala_import/nl/BUILD.bazel

@@ -1,10 +1,17 @@
 load("//scala:scala.bzl", "scala_specs2_junit_test")
 load("//scala:scala_import.bzl", "scala_import")
+load("@rules_java//java:defs.bzl", "java_library")
+
+java_library(
+    name = "ScalaImportNeverLink",
+    srcs = ["ScalaImportNeverLink.java"],
+    visibility = ["//visibility:public"],
+)
 
 scala_import(
     name = "scala_import_never_link",
     jars = [
-        "scala_import_never_link.jar",
+        ":ScalaImportNeverLink",
     ],
     neverlink = 1,
 )

test/src/main/scala/scalarules/test/scala_import/nl/ScalaImportNeverLink.java

@@ -1,14 +1,3 @@
 package scalarules.test.scala_import.nl;
 
-/**
- * This class is packaged in scala_import_never_link.jar
- *
- * <p>The jar file was created with the following steps:
- *
- * <p>- javac ScalaImportNeverLink.java - makdir -p scalarules/test/scala_import/nl - mv
- * ScalaImportNeverLink.class scalarules/test/scala_import/nl - jar cf scala_import_never_link.jar
- * scalarules - rm -fr scalarules ScalaImportNeverLink.class
- *
- * <p>To stage the updated jar: git add -f scala_import_never_link.jar
- */
 public class ScalaImportNeverLink {}

test/src/main/scala/scalarules/test/twitter_scrooge/thrift/bare_jar_thrifts/BUILD

@@ -1,17 +1,8 @@
-load("@rules_java//java:defs.bzl", "java_import")
 load("//thrift:thrift.bzl", "thrift_library")
 
-java_import(
-    name = "barejar",
-    jars = ["bare-thrift.jar"],
-    visibility = ["//visibility:public"],
-)
-
 thrift_library(
     name = "bare_jar_thrifts",
-    external_jars = [
-        ":barejar",
-    ],
+    srcs = ["//test/src/main/resources/scalarules/test:Foo.thrift"],
     visibility = ["//visibility:public"],
     deps = [
         "//test/src/main/scala/scalarules/test/twitter_scrooge/thrift/bare_jar_thrifts/bare_jar_1",

test/src/main/scala/scalarules/test/twitter_scrooge/thrift/bare_jar_thrifts/bare_jar_1/BUILD

@@ -2,8 +2,6 @@ load("//thrift:thrift.bzl", "thrift_library")
 
 thrift_library(
     name = "bare_jar_1",
-    external_jars = [
-        "bare-thrift-1.jar",
-    ],
+    srcs = ["//test/src/main/resources/scalarules/test:Bar.thrift"],
     visibility = ["//visibility:public"],
 )

test/src/main/scala/scalarules/test/twitter_scrooge/thrift/bare_jar_thrifts/bare_jar_2/BUILD

@@ -2,8 +2,6 @@ load("//thrift:thrift.bzl", "thrift_library")
 
 thrift_library(
     name = "bare_jar_2",
-    external_jars = [
-        "bare-thrift-2.jar",
-    ],
+    srcs = ["//test/src/main/resources/scalarules/test:Baz.thrift"],
     visibility = ["//visibility:public"],
 )

test/src/main/scala/scalarules/test/twitter_scrooge/thrift/thrift2/thrift3/BUILD

@@ -9,7 +9,7 @@ thrift_library(
 
 scrooge_scala_import(
     name = "thrift3_import",
-    scala_jars = ["thrift3_scrooge.jar"],
+    scala_jars = ["//test/src/main/scala/scalarules/test/twitter_scrooge:scrooge3"],
     thrift_jars = ["libthrift3.jar"],
     visibility = ["//visibility:public"],
 )

test_version/version_specific_tests_dir/src/main/resources/scalarules/test/BUILD

@@ -0,0 +1,5 @@
+exports_files([
+    "Foo.thrift",
+    "Bar.thrift",
+    "Baz.thrift",
+])

test_version/version_specific_tests_dir/src/main/resources/scalarules/test/Bar.thrift

@@ -0,0 +1,5 @@
+namespace java com.foo.bar.baz
+
+struct Bar {
+  1: optional string Bar
+}

test_version/version_specific_tests_dir/src/main/resources/scalarules/test/Baz.thrift

@@ -0,0 +1,5 @@
+namespace java com.foo.bar.baz
+
+struct Baz {
+  1: optional string Baz
+}

test_version/version_specific_tests_dir/src/main/resources/scalarules/test/Foo.thrift

@@ -0,0 +1,5 @@
+namespace java com.foo.bar.baz
+
+struct Foo {
+  1: optional string foo
+}

test_version/version_specific_tests_dir/src/main/scala/scalarules/test/twitter_scrooge/thrift/bare_jar_thrifts/BUILD

@@ -1,17 +1,8 @@
-load("@rules_java//java:defs.bzl", "java_import")
 load("@io_bazel_rules_scala//thrift:thrift.bzl", "thrift_library")
 
-java_import(
-    name = "barejar",
-    jars = ["bare-thrift.jar"],
-    visibility = ["//visibility:public"],
-)
-
 thrift_library(
     name = "bare_jar_thrifts",
-    external_jars = [
-        ":barejar",
-    ],
+    srcs = ["//src/main/resources/scalarules/test:Foo.thrift"],
     visibility = ["//visibility:public"],
     deps = [
         "//src/main/scala/scalarules/test/twitter_scrooge/thrift/bare_jar_thrifts/bare_jar_1",

test_version/version_specific_tests_dir/src/main/scala/scalarules/test/twitter_scrooge/thrift/bare_jar_thrifts/bare_jar_1/BUILD

@@ -2,8 +2,6 @@ load("@io_bazel_rules_scala//thrift:thrift.bzl", "thrift_library")
 
 thrift_library(
     name = "bare_jar_1",
-    external_jars = [
-        "bare-thrift-1.jar",
-    ],
+    srcs = ["//src/main/resources/scalarules/test:Bar.thrift"],
     visibility = ["//visibility:public"],
 )

test_version/version_specific_tests_dir/src/main/scala/scalarules/test/twitter_scrooge/thrift/bare_jar_thrifts/bare_jar_2/BUILD

@@ -2,8 +2,6 @@ load("@io_bazel_rules_scala//thrift:thrift.bzl", "thrift_library")
 
 thrift_library(
     name = "bare_jar_2",
-    external_jars = [
-        "bare-thrift-2.jar",
-    ],
+    srcs = ["//src/main/resources/scalarules/test:Baz.thrift"],
     visibility = ["//visibility:public"],
 )

test_version/version_specific_tests_dir/src/main/scala/scalarules/test/twitter_scrooge/thrift/thrift2/thrift3/BUILD

@@ -9,7 +9,7 @@ thrift_library(
 
 scrooge_scala_import(
     name = "thrift3_import",
-    scala_jars = ["thrift3_scrooge.jar"],
+    scala_jars = ["//src/main/scala/scalarules/test/twitter_scrooge:scrooge3"],
     thrift_jars = ["libthrift3.jar"],
     visibility = ["//visibility:public"],
 )