Merge lp:~elopio/snappy-tests-job/get_branch into lp:~fgimenez/snappy-tests-job/trunk

[Leo Arias]

Renamed Run to Get in the source interface.

I'm adding a Merge func to the interface, and I thought that Run is not as accurate as Get. So I'm renaming it here.

Unmerged revisions

62. By Leo Arias on 2015-08-21

Updated the call to source.Get in main.

61. By Leo Arias on 2015-08-21

Renamed Run to Get in source.go.

60. By Leo Arias on 2015-08-21

Renamed Run to Get in the source_tests.

=== modified file 'snappy-tests-job/main.go'
--- snappy-tests-job/main.go	2015-08-13 15:56:06 +0000
+++ snappy-tests-job/main.go	2015-08-21 02:27:54 +0000
@@ -68,7 +68,7 @@
 	// TODO: we need a waitGroup here to run things more efficiently
 	utilHandler := utils.NewBasicHandler()
 	sourceHandler := source.NewBzrHandler(utilHandler)
-	sourcePath, err := sourceHandler.Run(*src)
+	sourcePath, err := sourceHandler.Get(*src)
 	if err != nil {
 		log.Panicf("Error getting %s into %s", *src, sourcePath)
 	}
 
=== modified file 'source/source.go'
--- source/source.go	2015-07-24 15:28:01 +0000
+++ source/source.go	2015-08-21 02:27:54 +0000
@@ -36,7 +36,7 @@
 
 // Sourcer is the interface satisfied by all the source handlers
 type Sourcer interface {
-	Run(string) (string, error)
+	Get(string) (string, error)
 }
 
 // BzrHandler is a source handler for bazaar
@@ -44,8 +44,8 @@
 	util utils.Utilizer
 }
 
-// Run is the method for getting the source code
-func (handler *BzrHandler) Run(repo string) (string, error) {
+// Get is the method for getting the source code
+func (handler *BzrHandler) Get(repo string) (string, error) {
 	path := filepath.Join(targetPath, strconv.Itoa(os.Getpid()), basePkg)
 
 	log.Printf("*** Branching %s in %s ***", repo, path)
 
=== modified file 'source/source_test.go'
--- source/source_test.go	2015-07-24 09:59:38 +0000
+++ source/source_test.go	2015-08-21 02:27:54 +0000
@@ -54,8 +54,8 @@
 		check.Commentf("Expected type Sourcer, got %T", s.subject))
 }
 
-func (s *sourceSuite) TestRunCreatesTargetPath(c *check.C) {
-	s.subject.Run(testRepo)
+func (s *sourceSuite) TestGetCreatesTargetPath(c *check.C) {
+	s.subject.Get(testRepo)
 
 	expected := expectedBasePath()
 
@@ -63,26 +63,26 @@
 		check.Commentf("The target directory %s was not created", expected))
 }
 
-func (s *sourceSuite) TestRunCallsBranchCommand(c *check.C) {
-	s.subject.Run(testRepo)
+func (s *sourceSuite) TestGetCallsBranchCommand(c *check.C) {
+	s.subject.Get(testRepo)
 
 	pullCmd := pullCmd()
 	c.Assert(s.fakeUtil.ExecCalls[pullCmd], check.Equals, 1,
 		check.Commentf("The pull command %s was not called", pullCmd))
 }
 
-func (s *sourceSuite) TestRunReturnValue(c *check.C) {
-	path, err := s.subject.Run(testRepo)
+func (s *sourceSuite) TestGetReturnValue(c *check.C) {
+	path, err := s.subject.Get(testRepo)
 	expected := expectedReturnPath()
 
 	c.Assert(path, check.Equals, expected, check.Commentf("Unexpected return value path %s", path))
 	c.Assert(err, check.IsNil, check.Commentf("Unexpected return error %s", err))
 }
 
-func (s *sourceSuite) TestRunDoesNotCallBranchCommandOnError(c *check.C) {
+func (s *sourceSuite) TestGetDoesNotCallBranchCommandOnError(c *check.C) {
 	s.fakeUtil.FailMkDir = true
 
-	s.subject.Run(testRepo)
+	s.subject.Get(testRepo)
 
 	pullCmd := pullCmd()
 	c.Assert(s.fakeUtil.ExecCalls[pullCmd], check.Equals, 0,