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

Unified Diff: cipd/client/cipd/ensure/bad_test.go

Issue 2663473002: [cipd] Rename new Root terminology to Subdir. (Closed)
Patch Set: Created 3 years, 11 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 | « cipd/client/cipd/common/common_test.go ('k') | cipd/client/cipd/ensure/doc.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cipd/client/cipd/ensure/bad_test.go
diff --git a/cipd/client/cipd/ensure/bad_test.go b/cipd/client/cipd/ensure/bad_test.go
index 794a38c418614fe719276fe112bcf72a57c233d7..7fde4326ab1710ac4b578fd3277bfef6d92b2248 100644
--- a/cipd/client/cipd/ensure/bad_test.go
+++ b/cipd/client/cipd/ensure/bad_test.go
@@ -42,39 +42,39 @@ var badEnsureFiles = []struct {
},
{
- "windows root",
- "@root folder\\thing",
- `bad root path: backslashes not allowed`,
+ "windows subdir",
+ "@subdir folder\\thing",
+ `bad subdir: backslashes not allowed`,
},
{
- "messy root",
- "@root folder/../something",
- `bad root path: "folder/../something" (should be "something")`,
+ "messy subdir",
+ "@subdir folder/../something",
+ `bad subdir: "folder/../something" (should be "something")`,
},
{
- "relative root",
- "@root ../../something",
- `bad root path: invalid ".": "../../something"`,
+ "relative subdir",
+ "@subdir ../../something",
+ `bad subdir: invalid ".": "../../something"`,
},
{
- "absolute root",
- "@root /etc",
- `bad root path: absolute paths not allowed`,
+ "absolute subdir",
+ "@subdir /etc",
+ `bad subdir: absolute paths not allowed`,
},
{
"extra slashes",
- "@root //foo/bar/baz",
- `bad root path`,
+ "@subdir //foo/bar/baz",
+ `bad subdir`,
},
{
"windows style",
- "@root c:/foo/bar/baz",
- `bad root path`,
+ "@subdir c:/foo/bar/baz",
+ `bad subdir`,
},
{
@@ -122,7 +122,7 @@ var badEnsureFiles = []struct {
"some/package/something version",
"some/package/something latest",
),
- `duplicate package in root "": "some/package/something": defined on line 1 and 2`,
+ `duplicate package in subdir "": "some/package/something": defined on line 1 and 2`,
},
{
@@ -132,7 +132,7 @@ var badEnsureFiles = []struct {
"some/other/package canary",
"some/package/test_arch latest",
),
- `duplicate package in root "": "some/package/test_arch": defined on line 1 and 3`,
+ `duplicate package in subdir "": "some/package/test_arch": defined on line 1 and 3`,
},
{
@@ -156,7 +156,7 @@ var badEnsureFiles = []struct {
{
"late setting (directive)",
f(
- "@Root some/path",
+ "@Subdir some/path",
"",
"$ServiceURL https://something.example.com",
),
« no previous file with comments | « cipd/client/cipd/common/common_test.go ('k') | cipd/client/cipd/ensure/doc.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698