Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1327)

Unified Diff: tools/gn/config_unittest.cc

Issue 2940873002: Implement tracking of BUILD.gn files used to define target, toolchain or (Closed)
Patch Set: Fix compilation after rebase. Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/gn/config.cc ('k') | tools/gn/config_values_extractors_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/gn/config_unittest.cc
diff --git a/tools/gn/config_unittest.cc b/tools/gn/config_unittest.cc
index 9a1e01df62f1f2eba6a55fc50fda3fc5f679ca0d..8eb9b8253f8cbefc4df05392cf52590208349f2d 100644
--- a/tools/gn/config_unittest.cc
+++ b/tools/gn/config_unittest.cc
@@ -12,7 +12,7 @@ TEST(Config, ResolvedNoSub) {
TestWithScope setup;
Err err;
- Config config(setup.settings(), Label(SourceDir("//foo/"), "bar"));
+ Config config(setup.settings(), Label(SourceDir("//foo/"), "bar"), {});
config.own_values().defines().push_back("FOO");
ASSERT_TRUE(config.OnResolved(&err));
@@ -31,15 +31,15 @@ TEST(Config, ResolvedSub) {
TestWithScope setup;
Err err;
- Config sub1(setup.settings(), Label(SourceDir("//foo/"), "1"));
+ Config sub1(setup.settings(), Label(SourceDir("//foo/"), "1"), {});
sub1.own_values().defines().push_back("ONE");
ASSERT_TRUE(sub1.OnResolved(&err));
- Config sub2(setup.settings(), Label(SourceDir("//foo/"), "2"));
+ Config sub2(setup.settings(), Label(SourceDir("//foo/"), "2"), {});
sub2.own_values().defines().push_back("TWO");
ASSERT_TRUE(sub2.OnResolved(&err));
- Config config(setup.settings(), Label(SourceDir("//foo/"), "bar"));
+ Config config(setup.settings(), Label(SourceDir("//foo/"), "bar"), {});
config.own_values().defines().push_back("FOO");
config.configs().push_back(LabelConfigPair(&sub1));
config.configs().push_back(LabelConfigPair(&sub2));
@@ -63,16 +63,16 @@ TEST(Config, SubSub) {
Err err;
// Set up first -> middle -> last configs.
- Config last(setup.settings(), Label(SourceDir("//foo/"), "last"));
+ Config last(setup.settings(), Label(SourceDir("//foo/"), "last"), {});
last.own_values().defines().push_back("LAST");
ASSERT_TRUE(last.OnResolved(&err));
- Config middle(setup.settings(), Label(SourceDir("//foo/"), "middle"));
+ Config middle(setup.settings(), Label(SourceDir("//foo/"), "middle"), {});
middle.own_values().defines().push_back("MIDDLE");
middle.configs().push_back(LabelConfigPair(&last));
ASSERT_TRUE(middle.OnResolved(&err));
- Config first(setup.settings(), Label(SourceDir("//foo/"), "first"));
+ Config first(setup.settings(), Label(SourceDir("//foo/"), "first"), {});
first.own_values().defines().push_back("FIRST");
first.configs().push_back(LabelConfigPair(&middle));
ASSERT_TRUE(first.OnResolved(&err));
« no previous file with comments | « tools/gn/config.cc ('k') | tools/gn/config_values_extractors_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698