Skip to content
This repository was archived by the owner on Sep 9, 2020. It is now read-only.

Commit 0a4f68a

Browse files
ibrashoibrasho
committed
dep: add NewManifest
This change adds a constructor for dep.Manifest. Signed-off-by: Ibrahim AshShohail <[email protected]>
1 parent 53e80dc commit 0a4f68a

14 files changed

+123
-112
lines changed

‎Gopkg.toml‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,3 +18,8 @@
1818
[[constraint]]
1919
name = "github.com/pkg/errors"
2020
version = "0.8.0"
21+
22+
[prune]
23+
non-go = true
24+
go-tests = true
25+
unused-packages = true

‎cmd/dep/ensure.go‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -603,9 +603,8 @@ func (cmd *ensureCommand) runAdd(ctx *dep.Ctx, args []string, p *dep.Project, sm
603603

604604
// Prep post-actions and feedback from adds.
605605
var reqlist []string
606-
appender := &dep.Manifest{
607-
Constraints: make(gps.ProjectConstraints),
608-
}
606+
appender := dep.NewManifest()
607+
609608
for pr, instr := range addInstructions {
610609
for path := range instr.ephReq {
611610
reqlist = append(reqlist, path)

‎cmd/dep/glide_importer.go‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -141,9 +141,7 @@ func (g *glideImporter) convert(pr gps.ProjectRoot) (*dep.Manifest, *dep.Lock, e
141141
task.WriteString("...")
142142
g.logger.Println(task)
143143

144-
manifest := &dep.Manifest{
145-
Constraints: make(gps.ProjectConstraints),
146-
}
144+
manifest := dep.NewManifest()
147145

148146
for _, pkg := range append(g.yaml.Imports, g.yaml.TestImports...) {
149147
pc, err := g.buildProjectConstraint(pkg)

‎cmd/dep/glide_importer_test.go‎

Lines changed: 16 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -58,12 +58,13 @@ func TestGlideConfig_Convert(t *testing.T) {
5858
},
5959
},
6060
convertTestCase: &convertTestCase{
61-
projectRoot: "github.com/sdboyer/deptest",
62-
wantSourceRepo: "https://github.com/sdboyer/deptest.git",
63-
wantConstraint: "^1.0.0",
64-
wantLockCount: 1,
65-
wantRevision: gps.Revision("ff2948a2ac8f538c4ecd55962e919d1e13e74baf"),
66-
wantVersion: "v1.0.0",
61+
projectRoot: "github.com/sdboyer/deptest",
62+
wantSourceRepo: "https://github.com/sdboyer/deptest.git",
63+
wantConstraint: "^1.0.0",
64+
wantLockCount: 1,
65+
wantRevision: gps.Revision("ff2948a2ac8f538c4ecd55962e919d1e13e74baf"),
66+
wantVersion: "v1.0.0",
67+
wantIgnoredPackages: []string{},
6768
},
6869
},
6970
"test project": {
@@ -84,10 +85,11 @@ func TestGlideConfig_Convert(t *testing.T) {
8485
},
8586
},
8687
convertTestCase: &convertTestCase{
87-
projectRoot: "github.com/sdboyer/deptest",
88-
wantLockCount: 1,
89-
wantConstraint: "^1.0.0",
90-
wantVersion: "v1.0.0",
88+
projectRoot: "github.com/sdboyer/deptest",
89+
wantLockCount: 1,
90+
wantConstraint: "^1.0.0",
91+
wantVersion: "v1.0.0",
92+
wantIgnoredPackages: []string{},
9193
},
9294
},
9395
"revision only": {
@@ -107,9 +109,10 @@ func TestGlideConfig_Convert(t *testing.T) {
107109
},
108110
},
109111
convertTestCase: &convertTestCase{
110-
projectRoot: "github.com/sdboyer/deptest",
111-
wantLockCount: 1,
112-
wantRevision: gps.Revision("ff2948a2ac8f538c4ecd55962e919d1e13e74baf"),
112+
projectRoot: "github.com/sdboyer/deptest",
113+
wantLockCount: 1,
114+
wantRevision: gps.Revision("ff2948a2ac8f538c4ecd55962e919d1e13e74baf"),
115+
wantIgnoredPackages: []string{},
113116
},
114117
},
115118
"with ignored package": {

‎cmd/dep/godep_importer.go‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -88,9 +88,7 @@ func (g *godepImporter) load(projectDir string) error {
8888
func (g *godepImporter) convert(pr gps.ProjectRoot) (*dep.Manifest, *dep.Lock, error) {
8989
g.logger.Println("Converting from Godeps.json ...")
9090

91-
manifest := &dep.Manifest{
92-
Constraints: make(gps.ProjectConstraints),
93-
}
91+
manifest := dep.NewManifest()
9492
lock := &dep.Lock{}
9593

9694
for _, pkg := range g.json.Imports {

‎cmd/dep/godep_importer_test.go‎

Lines changed: 17 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,12 @@ func TestGodepConfig_Convert(t *testing.T) {
3434
},
3535
},
3636
convertTestCase: &convertTestCase{
37-
projectRoot: gps.ProjectRoot("github.com/sdboyer/deptest"),
38-
wantConstraint: "^0.8.0",
39-
wantRevision: gps.Revision("ff2948a2ac8f538c4ecd55962e919d1e13e74baf"),
40-
wantVersion: "v0.8.0",
41-
wantLockCount: 1,
37+
projectRoot: gps.ProjectRoot("github.com/sdboyer/deptest"),
38+
wantConstraint: "^0.8.0",
39+
wantRevision: gps.Revision("ff2948a2ac8f538c4ecd55962e919d1e13e74baf"),
40+
wantVersion: "v0.8.0",
41+
wantLockCount: 1,
42+
wantIgnoredPackages: []string{},
4243
},
4344
},
4445
"with semver suffix": {
@@ -52,11 +53,11 @@ func TestGodepConfig_Convert(t *testing.T) {
5253
},
5354
},
5455
convertTestCase: &convertTestCase{
55-
56-
projectRoot: gps.ProjectRoot("github.com/sdboyer/deptest"),
57-
wantConstraint: "^1.12.0-12-g2fd980e",
58-
wantLockCount: 1,
59-
wantVersion: "v1.0.0",
56+
projectRoot: gps.ProjectRoot("github.com/sdboyer/deptest"),
57+
wantConstraint: "^1.12.0-12-g2fd980e",
58+
wantLockCount: 1,
59+
wantVersion: "v1.0.0",
60+
wantIgnoredPackages: []string{},
6061
},
6162
},
6263
"empty comment": {
@@ -70,20 +71,19 @@ func TestGodepConfig_Convert(t *testing.T) {
7071
},
7172
},
7273
convertTestCase: &convertTestCase{
73-
74-
projectRoot: gps.ProjectRoot("github.com/sdboyer/deptest"),
75-
wantConstraint: "^1.0.0",
76-
wantRevision: gps.Revision("ff2948a2ac8f538c4ecd55962e919d1e13e74baf"),
77-
wantVersion: "v1.0.0",
78-
wantLockCount: 1,
74+
projectRoot: gps.ProjectRoot("github.com/sdboyer/deptest"),
75+
wantConstraint: "^1.0.0",
76+
wantRevision: gps.Revision("ff2948a2ac8f538c4ecd55962e919d1e13e74baf"),
77+
wantVersion: "v1.0.0",
78+
wantLockCount: 1,
79+
wantIgnoredPackages: []string{},
7980
},
8081
},
8182
"bad input - empty package name": {
8283
json: godepJSON{
8384
Imports: []godepPackage{{ImportPath: ""}},
8485
},
8586
convertTestCase: &convertTestCase{
86-
8787
wantConvertErr: true,
8888
},
8989
},
@@ -96,7 +96,6 @@ func TestGodepConfig_Convert(t *testing.T) {
9696
},
9797
},
9898
convertTestCase: &convertTestCase{
99-
10099
wantConvertErr: true,
101100
},
102101
},
@@ -116,7 +115,6 @@ func TestGodepConfig_Convert(t *testing.T) {
116115
},
117116
},
118117
convertTestCase: &convertTestCase{
119-
120118
projectRoot: gps.ProjectRoot("github.com/sdboyer/deptest"),
121119
wantLockCount: 1,
122120
wantConstraint: "^1.0.0",

‎cmd/dep/gopath_scanner.go‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,9 @@ func (g *gopathScanner) InitializeRootManifestAndLock(rootM *dep.Manifest, rootL
5252
return err
5353
}
5454

55-
g.origM = &dep.Manifest{
56-
Constraints: g.pd.constraints,
57-
Ovr: make(gps.ProjectConstraints),
58-
}
55+
g.origM = dep.NewManifest()
56+
g.origM.Constraints = g.pd.constraints
57+
5958
g.origL = &dep.Lock{
6059
P: make([]gps.LockedProject, 0, len(g.pd.ondisk)),
6160
}

‎cmd/dep/gopath_scanner_test.go‎

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -27,19 +27,14 @@ func TestGopathScanner_OverlayManifestConstraints(t *testing.T) {
2727
v1 := gps.NewVersion("v1.0.0")
2828
v2 := gps.NewVersion("v2.0.0")
2929
v3 := gps.NewVersion("v3.0.0")
30-
rootM := &dep.Manifest{
31-
Constraints: gps.ProjectConstraints{
32-
pi1.ProjectRoot: gps.ProjectProperties{Constraint: v1},
33-
},
34-
}
30+
rootM := dep.NewManifest()
31+
rootM.Constraints[pi1.ProjectRoot] = gps.ProjectProperties{Constraint: v1}
3532
rootL := &dep.Lock{}
33+
origM := dep.NewManifest()
34+
origM.Constraints[pi1.ProjectRoot] = gps.ProjectProperties{Constraint: v2}
35+
origM.Constraints[pi2.ProjectRoot] = gps.ProjectProperties{Constraint: v3}
3636
gs := gopathScanner{
37-
origM: &dep.Manifest{
38-
Constraints: gps.ProjectConstraints{
39-
pi1.ProjectRoot: gps.ProjectProperties{Constraint: v2},
40-
pi2.ProjectRoot: gps.ProjectProperties{Constraint: v3},
41-
},
42-
},
37+
origM: origM,
4338
origL: &dep.Lock{},
4439
ctx: ctx,
4540
pd: projectData{
@@ -79,7 +74,7 @@ func TestGopathScanner_OverlayLockProjects(t *testing.T) {
7974
h := test.NewHelper(t)
8075
ctx := newTestContext(h)
8176

82-
rootM := &dep.Manifest{}
77+
rootM := dep.NewManifest()
8378
pi1 := gps.ProjectIdentifier{ProjectRoot: gps.ProjectRoot(testProject1)}
8479
pi2 := gps.ProjectIdentifier{ProjectRoot: gps.ProjectRoot(testProject2)}
8580
v1 := gps.NewVersion("v1.0.0")
@@ -89,7 +84,7 @@ func TestGopathScanner_OverlayLockProjects(t *testing.T) {
8984
P: []gps.LockedProject{gps.NewLockedProject(pi1, v1, []string{})},
9085
}
9186
gs := gopathScanner{
92-
origM: &dep.Manifest{Constraints: make(gps.ProjectConstraints)},
87+
origM: dep.NewManifest(),
9388
origL: &dep.Lock{
9489
P: []gps.LockedProject{
9590
gps.NewLockedProject(pi1, v2, []string{}), // ignored, already exists in lock

‎cmd/dep/root_analyzer.go‎

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,7 @@ func (a *rootAnalyzer) InitializeRootManifestAndLock(dir string, pr gps.ProjectR
5252
}
5353

5454
if rootM == nil {
55-
rootM = &dep.Manifest{
56-
Constraints: make(gps.ProjectConstraints),
57-
Ovr: make(gps.ProjectConstraints),
58-
}
55+
rootM = dep.NewManifest()
5956
}
6057
if rootL == nil {
6158
rootL = &dep.Lock{}
@@ -88,7 +85,8 @@ func (a *rootAnalyzer) importManifestAndLock(dir string, pr gps.ProjectRoot, sup
8885
}
8986
}
9087

91-
var emptyManifest = &dep.Manifest{Constraints: make(gps.ProjectConstraints), Ovr: make(gps.ProjectConstraints)}
88+
var emptyManifest = dep.NewManifest()
89+
9290
return emptyManifest, nil, nil
9391
}
9492

‎cmd/dep/vndr_importer.go‎

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -86,11 +86,9 @@ func (v *vndrImporter) loadVndrFile(dir string) error {
8686

8787
func (v *vndrImporter) convert(pr gps.ProjectRoot) (*dep.Manifest, *dep.Lock, error) {
8888
var (
89-
manifest = &dep.Manifest{
90-
Constraints: make(gps.ProjectConstraints),
91-
}
92-
lock = &dep.Lock{}
93-
err error
89+
manifest = dep.NewManifest()
90+
lock = &dep.Lock{}
91+
err error
9492
)
9593

9694
for _, pkg := range v.packages {

0 commit comments

Comments
 (0)