Merge pull request #427 from Manoramsharma/pathch-new-line

BUG: Removed extra newline in kcl.mod

pkg/package/toml.go

@@ -74,7 +74,6 @@ func (dep *Dependencies) MarshalTOML() string {
 			sb.WriteString(NEWLINE)
 			sb.WriteString(dep.MarshalTOML())
 		}
-		sb.WriteString(NEWLINE)
 	}
 	return sb.String()
 }

pkg/package/toml_test.go

@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"os"
 	"path/filepath"
+	"strings"
 	"testing"
 
 	"github.com/BurntSushi/toml"
@@ -286,3 +287,56 @@ func TestMarshalOciUrlIntoFile(t *testing.T) {
 		assert.Equal(t, utils.RmNewline(string(expectKclModFileContents)), utils.RmNewline(writeKclModFileContents))
 	}
 }
+
+func TestInitEmptyPkg(t *testing.T) {
+	modfile := ModFile{
+		Pkg: Package{
+			Name:    "MyKcl",
+			Edition: "v0.0.1",
+			Version: "v0.0.1",
+			Include: []string{"src/", "README.md", "LICENSE"},
+			Exclude: []string{"target/", ".git/", "*.log"},
+		},
+		Dependencies: Dependencies{
+			orderedmap.NewOrderedMap[string, Dependency](),
+		},
+	}
+
+	dep := Dependency{
+		Name:     "MyKcl1",
+		FullName: "MyKcl1_v0.0.2",
+		Source: downloader.Source{
+			Git: &downloader.Git{
+				Url: "https://github.com/test/MyKcl1.git",
+				Tag: "v0.0.2",
+			},
+		},
+	}
+
+	ociDep := Dependency{
+		Name:     "MyOciKcl1",
+		FullName: "MyOciKcl1_0.0.1",
+		Version:  "0.0.1",
+		Source: downloader.Source{
+			Oci: &downloader.Oci{
+				Tag: "0.0.1",
+			},
+		},
+	}
+
+	modfile.Dependencies.Deps.Set("MyOciKcl1_0.0.1", ociDep)
+	modfile.Dependencies.Deps.Set("MyKcl1_v0.0.2", dep)
+
+	got_data := modfile.MarshalTOML()
+
+	expected_data, _ := os.ReadFile(filepath.Join(getTestDir(testTomlDir), "expected.toml"))
+	expected_toml := string(expected_data)
+
+	expected_toml = strings.ReplaceAll(expected_toml, "\r\n", "\n")
+	got_data = strings.ReplaceAll(got_data, "\r\n", "\n")
+
+	fmt.Printf("expected_toml: '%q'\n", expected_toml)
+
+	fmt.Printf("modfile: '%q'\n", got_data)
+	assert.Equal(t, expected_toml, got_data)
+}