From: Hyoun Jiil <jiil.hyoun@samsung.com>
Date: Thu, 25 Apr 2013 02:34:52 +0000 (+0900)
Subject: [Title] add git clone or checkout fail error code
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=780d9d0fcfa5dfe0c65207bba3b3bb8d3b663940;p=sdk%2Ftools%2Fsdk-build.git

[Title] add git clone or checkout fail error code
[Type] Enhancement
[Module] Toolchain /
[Priority] Minor
[Jira#]
[Redmine#] 9187
[Problem]
[Cause]
[Solution]
[TestCase]

Change-Id: I215b22685e4dd66ef68a2392391470812ee8b453
---

diff --git a/src/build_server/BuildServerException.rb b/src/build_server/BuildServerException.rb
index 1daf4a3..7611d3f 100644
--- a/src/build_server/BuildServerException.rb
+++ b/src/build_server/BuildServerException.rb
@@ -38,6 +38,8 @@ class BuildServerException < Exception
 		"ERR108" => "Unsupported OS used in package file name!",
 		"ERR109" => "Extracting package file failed!",
 		"ERR110" => "Initializing sub jobs failed!",
+		"ERR111" => "Git clone failed!!",
+		"ERR112" => "Git checkout failed!!",
 
 		"ERR200" => "Unmet dependency found!",
 		"ERR201" => "Creating job builder failed!",
diff --git a/src/build_server/GitBuildJob.rb b/src/build_server/GitBuildJob.rb
index b66d081..9f9c800 100644
--- a/src/build_server/GitBuildJob.rb
+++ b/src/build_server/GitBuildJob.rb
@@ -144,10 +144,6 @@ class GitBuildJob < BuildJob
 
 		# download source code
 		@git_commit = get_source_code()
-		if @git_commit.nil? then
-			@status = "ERROR"
-			return false
-		end
 
 		# check pkginfo.manifest
 		if not File.exist? "#{@source_path}/package/pkginfo.manifest" then
@@ -287,13 +283,11 @@ class GitBuildJob < BuildJob
 		if  not File.exist? git_path then
 			# if "git pull" failed, try to "git clone"
 			if not git_cmd("clone #{@git_repos}", cache_path, @log) then
-				@log.error( "Failed on \"git clone #{@git_repos}\"", Log::LV_USER)
-				return nil
+				raise BuildServerException.new("ERR111"), "git clone #{@git_repos}"
 			end
 			# git checkout
 			if not git_cmd("checkout #{@git_branch}", git_path, @log) then
-				@log.error( "Failed on \"git checkout #{@git_branch}\"", Log::LV_USER)
-				return nil
+				raise BuildServerException.new("ERR112"), "git checkout #{@git_branch}"
 			end
 		end
 
diff --git a/test/build-server.basic1/build-cli-37.testcase b/test/build-server.basic1/build-cli-37.testcase
new file mode 100644
index 0000000..9362416
--- /dev/null
+++ b/test/build-server.basic1/build-cli-37.testcase
@@ -0,0 +1,12 @@
+#PRE-EXEC
+echo "error case when wrong git repository setted"
+../../build-svr add-prj -n testserver3 -N testaErr -g `pwd`/git01/aErr -b master
+#EXEC
+../../build-cli build -N testaErr -d 127.0.0.1:2223 -o ubuntu-32 -U admin@user -w 1234
+#POST-EXEC
+#EXPECT
+Info: Added new job
+Info: Initializing job...
+Error: Git clone failed!!: git clone 
+Error: Job is stopped by ERROR
+Error: Building job on remote server failed!
diff --git a/test/build-server.basic1/build-cli-38.testcase b/test/build-server.basic1/build-cli-38.testcase
new file mode 100644
index 0000000..544cc9c
--- /dev/null
+++ b/test/build-server.basic1/build-cli-38.testcase
@@ -0,0 +1,12 @@
+#PRE-EXEC
+echo "error case when wrong git branch setted"
+../../build-svr add-prj -n testserver3 -N testbErr -g `pwd`/git01/b -b testError
+#EXEC
+../../build-cli build -N testbErr -d 127.0.0.1:2223 -o ubuntu-32 -U admin@user -w 1234
+#POST-EXEC
+#EXPECT
+Info: Added new job
+Info: Initializing job...
+Error: Git checkout failed!!: git checkout
+Error: Job is stopped by ERROR
+Error: Building job on remote server failed!
diff --git a/test/build-server.basic1/testsuite b/test/build-server.basic1/testsuite
index cfbf416..babed08 100644
--- a/test/build-server.basic1/testsuite
+++ b/test/build-server.basic1/testsuite
@@ -34,3 +34,5 @@ build-cli-33.testcase
 build-cli-34.testcase
 build-cli-35.testcase
 build-cli-36.testcase
+build-cli-37.testcase
+build-cli-38.testcase