[ARVADOS] updated: 5824ee2e5198dd46a7813fe2adbd380a114f9ac4

git at public.curoverse.com git at public.curoverse.com
Tue May 19 15:28:56 EDT 2015


Summary of changes:
 services/datamanager/summary/pull_list_test.go | 44 ++++++++++++--------------
 1 file changed, 20 insertions(+), 24 deletions(-)

       via  5824ee2e5198dd46a7813fe2adbd380a114f9ac4 (commit)
      from  999c05af58ef1bb6f2b6804301f6ef8d02544a2b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 5824ee2e5198dd46a7813fe2adbd380a114f9ac4
Author: mishaz <misha at curoverse.com>
Date:   Tue May 19 19:28:36 2015 +0000

    More reformatting of unittests to improve readability.

diff --git a/services/datamanager/summary/pull_list_test.go b/services/datamanager/summary/pull_list_test.go
index 5638dc4..9d7075c 100644
--- a/services/datamanager/summary/pull_list_test.go
+++ b/services/datamanager/summary/pull_list_test.go
@@ -150,15 +150,15 @@ func (s *MySuite) TestBuildPullLists(c *C) {
 		BuildPullLists(map[Locator]PullServers{
 			locator1: PullServers{To: []string{"t1"}, From: []string{"f1", "f2"}}}),
 		PullListMapEquals,
-		map[string]PullList{"t1": PullList{
-			PullRequest{Locator: locator1, Servers: []string{"f1", "f2"}}}})
+		map[string]PullList{
+			"t1": PullList{PullRequest{locator1, []string{"f1", "f2"}}}})
 
 	c.Check(
 		BuildPullLists(map[Locator]PullServers{
 			locator1: PullServers{To: []string{"t1"}, From: []string{}}}),
 		PullListMapEquals,
 		map[string]PullList{"t1": PullList{
-			PullRequest{Locator: locator1, Servers: []string{}}}})
+			PullRequest{locator1, []string{}}}})
 
 	c.Check(
 		BuildPullLists(map[Locator]PullServers{
@@ -168,10 +168,8 @@ func (s *MySuite) TestBuildPullLists(c *C) {
 			}}),
 		PullListMapEquals,
 		map[string]PullList{
-			"t1": PullList{
-				PullRequest{Locator: locator1, Servers: []string{"f1", "f2"}}},
-			"t2": PullList{
-				PullRequest{Locator: locator1, Servers: []string{"f1", "f2"}}},
+			"t1": PullList{PullRequest{locator1, []string{"f1", "f2"}}},
+			"t2": PullList{PullRequest{locator1, []string{"f1", "f2"}}},
 		})
 
 	locator2 := Locator{Digest: blockdigest.MakeTestBlockDigest(0xCabbed)}
@@ -181,10 +179,8 @@ func (s *MySuite) TestBuildPullLists(c *C) {
 			locator2: PullServers{To: []string{"t2"}, From: []string{"f3", "f4"}}}),
 		PullListMapEquals,
 		map[string]PullList{
-			"t1": PullList{
-				PullRequest{Locator: locator1, Servers: []string{"f1", "f2"}}},
-			"t2": PullList{
-				PullRequest{Locator: locator2, Servers: []string{"f3", "f4"}}},
+			"t1": PullList{PullRequest{locator1, []string{"f1", "f2"}}},
+			"t2": PullList{PullRequest{locator2, []string{"f3", "f4"}}},
 		})
 
 	c.Check(
@@ -199,11 +195,11 @@ func (s *MySuite) TestBuildPullLists(c *C) {
 		PullListMapEquals,
 		map[string]PullList{
 			"t1": PullList{
-				PullRequest{Locator: locator1, Servers: []string{"f1", "f2"}},
-				PullRequest{Locator: locator2, Servers: []string{"f3", "f4"}},
+				PullRequest{locator1, []string{"f1", "f2"}},
+				PullRequest{locator2, []string{"f3", "f4"}},
 			},
 			"t2": PullList{
-				PullRequest{Locator: locator2, Servers: []string{"f3", "f4"}},
+				PullRequest{locator2, []string{"f3", "f4"}},
 			},
 		})
 
@@ -227,22 +223,22 @@ func (s *MySuite) TestBuildPullLists(c *C) {
 		PullListMapEquals,
 		map[string]PullList{
 			"t1": PullList{
-				PullRequest{Locator: locator1, Servers: []string{"f1", "f2"}},
-				PullRequest{Locator: locator2, Servers: []string{"f3", "f4"}},
-				PullRequest{Locator: locator3, Servers: []string{"f4", "f5"}},
-				PullRequest{Locator: locator4, Servers: []string{"f1", "f5"}},
+				PullRequest{locator1, []string{"f1", "f2"}},
+				PullRequest{locator2, []string{"f3", "f4"}},
+				PullRequest{locator3, []string{"f4", "f5"}},
+				PullRequest{locator4, []string{"f1", "f5"}},
 			},
 			"t2": PullList{
-				PullRequest{Locator: locator2, Servers: []string{"f3", "f4"}},
-				PullRequest{Locator: locator3, Servers: []string{"f4", "f5"}},
-				PullRequest{Locator: locator4, Servers: []string{"f1", "f5"}},
+				PullRequest{locator2, []string{"f3", "f4"}},
+				PullRequest{locator3, []string{"f4", "f5"}},
+				PullRequest{locator4, []string{"f1", "f5"}},
 			},
 			"t3": PullList{
-				PullRequest{Locator: locator3, Servers: []string{"f4", "f5"}},
-				PullRequest{Locator: locator4, Servers: []string{"f1", "f5"}},
+				PullRequest{locator3, []string{"f4", "f5"}},
+				PullRequest{locator4, []string{"f1", "f5"}},
 			},
 			"t4": PullList{
-				PullRequest{Locator: locator4, Servers: []string{"f1", "f5"}},
+				PullRequest{locator4, []string{"f1", "f5"}},
 			},
 		})
 }

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list