Skip modules without artifacts in command summary

artifactory/commands/commandssummaries/buildinfosummary.go

@@ -60,17 +60,28 @@ func (bis *BuildInfoSummary) buildInfoTable(builds []*buildInfo.BuildInfo) strin
 func (bis *BuildInfoSummary) buildInfoModules(builds []*buildInfo.BuildInfo) string {
 	var markdownBuilder strings.Builder
 	markdownBuilder.WriteString("\n\n ### Modules Published As Part of This Build  \n\n")
+	var shouldGenerate bool
 	for _, build := range builds {
 		for _, module := range build.Modules {
+			if len(module.Artifacts) == 0 {
+				continue
+			}
+
 			switch module.Type {
 			case buildInfo.Docker, buildInfo.Maven, buildInfo.Npm, buildInfo.Go, buildInfo.Generic, buildInfo.Terraform:
 				markdownBuilder.WriteString(bis.generateModuleMarkdown(module))
+				shouldGenerate = true
 			default:
 				// Skip unsupported module types.
 				continue
 			}
 		}
 	}
+
+	// If no supported module with artifacts was found, avoid generating the markdown.
+	if !shouldGenerate {
+		return ""
+	}
 	return markdownBuilder.String()
 }
 

artifactory/commands/commandssummaries/buildinfosummary_test.go

@@ -71,6 +71,41 @@ func TestBuildInfoModules(t *testing.T) {
 	assert.Equal(t, getTestDataFile(t, "modules.md"), gh.buildInfoModules(builds))
 }
 
+// Validate that if no supported module with artifacts was found, we avoid generating the markdown.
+func TestBuildInfoModulesEmpty(t *testing.T) {
+	gh := &BuildInfoSummary{}
+	var builds = []*buildinfo.BuildInfo{
+		{
+			Name:     "buildName",
+			Number:   "123",
+			Started:  "2024-05-05T12:47:20.803+0300",
+			BuildUrl: "http://myJFrogPlatform/builds/buildName/123",
+			Modules: []buildinfo.Module{
+				{
+					Type:      buildinfo.Maven,
+					Artifacts: []buildinfo.Artifact{},
+					Dependencies: []buildinfo.Dependency{{
+						Id: "dep1",
+					},
+					},
+				},
+				{
+					Type: buildinfo.Gradle,
+					Artifacts: []buildinfo.Artifact{
+						{
+							Name:                   "gradleArtifact",
+							Path:                   "dir/gradleArtifact",
+							OriginalDeploymentRepo: "gradle-local",
+						},
+					},
+				},
+			},
+		},
+	}
+
+	assert.Empty(t, gh.buildInfoModules(builds))
+}
+
 func getTestDataFile(t *testing.T, fileName string) string {
 	modulesPath := filepath.Join(".", "testdata", fileName)
 	content, err := os.ReadFile(modulesPath)