mirrors/nomulus
1
0
Fork 0
mirror of https://github.com/google/nomulus synced 2026-01-08 23:23:32 +00:00
Code Issues Releases Wiki Activity

Respect system TMPDIR in GPG related tests

Browse Source 
With https://github.com/bazelbuild/bazel/issues/4376, bazel 0.10.0 now supports accessing system TMPDIR in its sandbox. Use this instead of hardcoding /tmp in BUILD rules to get around the gpg-agent path length restriction.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=186010932
This commit is contained in:
jianglai
2018-02-16 10:07:08 -08:00
parent edc50bbe59
commit 7f2e3695a6
2 changed files with 3 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
javatests/google/registry/rde/BUILD
View File

@@ -51,10 +51,6 @@ java_library(
GenTestRules(
name = "GeneratedTestRules",
default_test_size = "large",
# TODO (b/63619837): stop using hardcoded /tmp once
# https://github.com/bazelbuild/bazel/issues/3215 is resolved.
# If /tmp is not accessible, try changing it to one that is.
jvm_flags = ["-Djava.io.tmpdir=/tmp"],
test_files = glob(
["*Test.java"],
exclude = ["RdeUploadActionTest.java"],

4
javatests/google/registry/testing/GpgSystemCommandRule.java
View File

@@ -84,7 +84,9 @@ public final class GpgSystemCommandRule extends ExternalResource {
@Override
protected void before() throws IOException, InterruptedException {
checkState(Objects.equals(cwd, DEV_NULL));
cwd = File.createTempFile(TEMP_FILE_PREFIX, "", null);
File tmpRoot = null;
tmpRoot = new File(System.getenv("TMPDIR"));
cwd = File.createTempFile(TEMP_FILE_PREFIX, "", tmpRoot);
cwd.delete();
cwd.mkdir();
conf = new File(cwd, ".gnupg");