update empty artifacts message

Signed-off-by: Jeeva Kandasamy <[email protected]>

cmd/command/download.go

@@ -33,18 +33,20 @@ var downloadArtifacts = &cobra.Command{
 	Use:     "artifact",
 	Aliases: []string{"artifacts"},
 	Short:   "Download artifact of a build",
-	Example: `  jenkinsctl download artifact 2101 --to-dir /tmp/artifacts`,
+	Example: `  jenkinsctl download artifact --build-number 2101 --to-dir /tmp/artifacts`,
 	Run: func(cmd *cobra.Command, args []string) {
 		client := jenkins.NewClient(CONFIG)
 		if client == nil {
 			return
 		}
 
-		savedLocation, err := client.DownloadArtifacts(CONFIG.JobContext, buildNumber, artifactSaveLocation)
+		savedLocation, err := client.DownloadArtifacts(ioStreams.Out, CONFIG.JobContext, buildNumber, artifactSaveLocation)
 		if err != nil {
 			fmt.Fprintf(ioStreams.ErrOut, "error on downloading artifacts. error:[%s]\n", err)
 			return
 		}
-		fmt.Fprintf(ioStreams.Out, "artifacts are downloaded on the directory: %s\n", savedLocation)
+		if savedLocation != "" {
+			fmt.Fprintf(ioStreams.Out, "artifacts are downloaded on the directory: %s\n", savedLocation)
+		}
 	},
 }

pkg/jenkins/client.go

@@ -186,15 +186,23 @@ func (jc *Client) ListParameters(jobName string) ([]gojenkins.ParameterDefinitio
 }
 
 // DownloadArtifacts of a build
-func (jc *Client) DownloadArtifacts(jobName string, buildNumber int, toDirectory string) (string, error) {
+func (jc *Client) DownloadArtifacts(out io.Writer, jobName string, buildNumber int, toDirectory string) (string, error) {
 	directoryFinal := filepath.Join(toDirectory, jobName, strconv.Itoa(buildNumber))
 	build, err := jc.api.GetBuild(jc.ctx, jobName, int64(buildNumber))
 	if err != nil {
 		return directoryFinal, err
 	}
 
 	dirSplitter := fmt.Sprintf("%d/artifact/", buildNumber)
-	for _, a := range build.GetArtifacts() {
+
+	artifacts := build.GetArtifacts()
+
+	if len(artifacts) == 0 {
+		fmt.Fprintln(out, "no artifacts found")
+		return "", nil
+	}
+
+	for _, a := range artifacts {
 		subDir := ""
 		if dirs := strings.SplitAfterN(a.Path, dirSplitter, 2); len(dirs) > 1 {
 			subDir = filepath.Dir(dirs[1])