The great Skylark -> Starlark class migration:
PackageSerializationContext.SkylarkRuleClass Table
PackageSerializer.DeserializedSkylarkRuleKey
Getting feet wet with class name changes.
Algorithm:
- Convert these class names
- Convert methods, variables, other symbols encountered in the files these
classes are defined in.
- *Don't* replace class names for classes defined in other files. These will
be handled in their own order. This helps keep individual changelist scope
small.
PiperOrigin-RevId: 307127450
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java
index 6760733..3f7988f 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java
@@ -577,7 +577,7 @@
SkyKey skyKey = PackageValue.key(PackageIdentifier.parse("@//foo"));
Package pkg = validPackageWithoutErrors(skyKey);
- assertThat(pkg.getSkylarkFileDependencies())
+ assertThat(pkg.getStarlarkFileDependencies())
.containsExactly(
Label.parseAbsolute("//bar:ext.bzl", ImmutableMap.of()),
Label.parseAbsolute("//baz:ext.bzl", ImmutableMap.of()));
@@ -590,7 +590,7 @@
Root.fromPath(rootDirectory));
pkg = validPackageWithoutErrors(skyKey);
- assertThat(pkg.getSkylarkFileDependencies())
+ assertThat(pkg.getStarlarkFileDependencies())
.containsExactly(
Label.parseAbsolute("//bar:ext.bzl", ImmutableMap.of()),
Label.parseAbsolute("//qux:ext.bzl", ImmutableMap.of()));