Add PklProject.deps.json file with newline for POSIX compliance (ap…

…ple#664) * Add Project.deps.json file with newline for POSIX compliance * Fix test cases
stackoverflow · Oct 1, 2024 · cc1e432 · cc1e432
1 parent 47f2143
commit cc1e432
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 1 deletion.
diff --git a/pkl-core/src/main/java/org/pkl/core/project/ProjectDeps.java b/pkl-core/src/main/java/org/pkl/core/project/ProjectDeps.java
@@ -218,6 +218,7 @@ private void write() throws IOException {
       }
       jsonWriter.endObject();
       jsonWriter.endObject();
+      jsonWriter.newline();
       jsonWriter.close();
     }
   }

diff --git a/pkl-core/src/test/files/LanguageSnippetTests/input/projects/project6/PklProject.deps.json b/pkl-core/src/test/files/LanguageSnippetTests/input/projects/project6/PklProject.deps.json
@@ -1,4 +1,4 @@
 {
   "schemaVersion": 1,
   "resolvedDependencies": {}
-}
+}
diff --git a/pkl-core/src/test/kotlin/org/pkl/core/project/ProjectDependenciesResolverTest.kt b/pkl-core/src/test/kotlin/org/pkl/core/project/ProjectDependenciesResolverTest.kt
@@ -86,6 +86,7 @@ class ProjectDependenciesResolverTest {
           }
         }
       }
+      
     """
           .trimIndent()
       )

diff --git a/pkl-core/src/test/kotlin/org/pkl/core/project/ProjectDepsTest.kt b/pkl-core/src/test/kotlin/org/pkl/core/project/ProjectDepsTest.kt
@@ -44,6 +44,7 @@ class ProjectDepsTest {
             }
           }
         }
+        
       """
       .trimIndent()
-Original file line number
+Diff line change
@@ Expand Up / @@ -86,6 +86,7 @@ class ProjectDependenciesResolverTest { @@
               }
             }
           }
         """
               .trimIndent()
           )
@@ Expand Down @@