Testing correct invalidation of Skylark Remote Repositories

A Skylark remote repository should be invalidated only when
the WORKSPACE file change, or one of its dependency or the Skylark
file change.

This change include two fixes:
  - The path of the RepositoryDirectoryValue was incorrect when
    the directory root is a symlink and the repository is not local
    (and not refetching). This was never triggered before because
    the only rule that were symlinking their root were the local
    one.
  - Directories were unitialized for the SkylarkRepositoryFunction
    (was forgotten as part of a refactor when introducing it).

--
MOS_MIGRATED_REVID=115547540

1 files changed, 1 insertions, 0 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/bazel/BazelRepositoryModule.java b/src/main/java/com/google/devtools/build/lib/bazel/BazelRepositoryModule.java
index 5e16483f6a..6d7c4f8886 100644
--- a/src/main/java/com/google/devtools/build/lib/bazel/BazelRepositoryModule.java
+++ b/src/main/java/com/google/devtools/build/lib/bazel/BazelRepositoryModule.java
@@ -110,6 +110,7 @@ public class BazelRepositoryModule extends BlazeModule {
     for (RepositoryFunction handler : repositoryHandlers.values()) {
       handler.setDirectories(directories);
     }
+    skylarkRepositoryFunction.setDirectories(directories);
   }
 
   /**