diff options
author | 2018-02-15 02:12:58 -0800 | |
---|---|---|
committer | 2018-02-15 02:14:44 -0800 | |
commit | f58dc51a225139fabeb452eb4b88022558f29f55 (patch) | |
tree | 5e06cdb7a39059dac51cd678f151e0a56bd26e9e /src/test/java/com/google/devtools/build | |
parent | 9d8ff35aac5006fb799584120166f8ecea1d391c (diff) |
C++: Removes CppModel class
The logic is split between CcCompilationHelper and CcLinkingHelper.
RELNOTES:none
PiperOrigin-RevId: 185809915
Diffstat (limited to 'src/test/java/com/google/devtools/build')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/rules/cpp/CompileBuildVariablesTest.java | 15 | ||||
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariablesTest.java | 2 |
2 files changed, 9 insertions, 8 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/rules/cpp/CompileBuildVariablesTest.java b/src/test/java/com/google/devtools/build/lib/rules/cpp/CompileBuildVariablesTest.java index 89b7a3b992..cbcca08e57 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/cpp/CompileBuildVariablesTest.java +++ b/src/test/java/com/google/devtools/build/lib/rules/cpp/CompileBuildVariablesTest.java @@ -59,9 +59,9 @@ public class CompileBuildVariablesTest extends BuildViewTestCase { Variables variables = getCompileBuildVariables("//x:bin", "bin"); - assertThat(variables.getStringVariable(CppModel.SOURCE_FILE_VARIABLE_NAME)) + assertThat(variables.getStringVariable(CcCompilationHelper.SOURCE_FILE_VARIABLE_NAME)) .contains("x/bin.cc"); - assertThat(variables.getStringVariable(CppModel.OUTPUT_FILE_VARIABLE_NAME)) + assertThat(variables.getStringVariable(CcCompilationHelper.OUTPUT_FILE_VARIABLE_NAME)) .contains("x/bin"); } @@ -76,7 +76,7 @@ public class CompileBuildVariablesTest extends BuildViewTestCase { Variables variables = getCompileBuildVariables("//x:bin", "bin"); ImmutableList<String> copts = - Variables.toStringList(variables, CppModel.LEGACY_COMPILE_FLAGS_VARIABLE_NAME); + Variables.toStringList(variables, CcCompilationHelper.LEGACY_COMPILE_FLAGS_VARIABLE_NAME); assertThat(copts).contains("-foo"); } @@ -91,7 +91,7 @@ public class CompileBuildVariablesTest extends BuildViewTestCase { Variables variables = getCompileBuildVariables("//x:bin", "bin"); ImmutableList<String> copts = - Variables.toStringList(variables, CppModel.USER_COMPILE_FLAGS_VARIABLE_NAME); + Variables.toStringList(variables, CcCompilationHelper.USER_COMPILE_FLAGS_VARIABLE_NAME); assertThat(copts).contains("-foo"); } @@ -108,7 +108,8 @@ public class CompileBuildVariablesTest extends BuildViewTestCase { Variables variables = getCompileBuildVariables("//x:bin", "bin"); ImmutableList<String> unfilteredCompileFlags = - Variables.toStringList(variables, CppModel.UNFILTERED_COMPILE_FLAGS_VARIABLE_NAME); + Variables.toStringList( + variables, CcCompilationHelper.UNFILTERED_COMPILE_FLAGS_VARIABLE_NAME); assertThat(unfilteredCompileFlags).contains("--i_ll_live_forever"); } @@ -121,7 +122,7 @@ public class CompileBuildVariablesTest extends BuildViewTestCase { Variables variables = getCompileBuildVariables("//x:bin", "bin"); ImmutableList<String> copts = - Variables.toStringList(variables, CppModel.USER_COMPILE_FLAGS_VARIABLE_NAME); + Variables.toStringList(variables, CcCompilationHelper.USER_COMPILE_FLAGS_VARIABLE_NAME); assertThat(copts).containsExactly("-foo").inOrder(); } @@ -137,7 +138,7 @@ public class CompileBuildVariablesTest extends BuildViewTestCase { Variables variables = getCompileBuildVariables("//x:bin", "bin"); - assertThat(variables.getStringVariable(CppModel.SYSROOT_VARIABLE_NAME)) + assertThat(variables.getStringVariable(CcCommon.SYSROOT_VARIABLE_NAME)) .isEqualTo("/usr/local/custom-sysroot"); } } diff --git a/src/test/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariablesTest.java b/src/test/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariablesTest.java index 1f039cf095..b467a7f4eb 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariablesTest.java +++ b/src/test/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariablesTest.java @@ -263,6 +263,6 @@ public class LinkBuildVariablesTest extends LinkBuildVariablesTestCase { ConfiguredTarget testTarget = getConfiguredTarget("//x:foo"); Variables testVariables = getLinkBuildVariables(testTarget, LinkTargetType.EXECUTABLE); - assertThat(testVariables.isAvailable(CppModel.SYSROOT_VARIABLE_NAME)).isTrue(); + assertThat(testVariables.isAvailable(CcCommon.SYSROOT_VARIABLE_NAME)).isTrue(); } } |