Tests

Author: tmat

src/Tests/Microsoft.NET.Build.Tests/SourceLinkTests.cs

@@ -25,15 +25,19 @@ public SourceLinkTests(ITestOutputHelper log)
         {
         }
 
-        private void CreateGitFiles(string repoDir, string originUrl)
+        private void CreateGitFiles(string repoDir, string originUrl, string commitSha = "1200000000000000000000000000000000000000")
         {
             var gitDir = Path.Combine(repoDir, ".git");
             var headsDir = Path.Combine(gitDir, "refs", "heads");
 
             Directory.CreateDirectory(gitDir);
             File.WriteAllText(Path.Combine(gitDir, "HEAD"), "ref: refs/heads/master");
             Directory.CreateDirectory(headsDir);
-            File.WriteAllText(Path.Combine(headsDir, "master"), "1200000000000000000000000000000000000000");
+
+            if (commitSha != null)
+            {
+                File.WriteAllText(Path.Combine(headsDir, "master"), commitSha);
+            }
 
             if (originUrl != null)
             {
@@ -81,6 +85,28 @@ public void WithNoGitMetadata()
             intermediateDir.Should().NotHaveFile("SourceLinkTestApp.sourcelink.json");
         }
 
+        /// <summary>
+        /// When creating a new repository locally we want the build to work and not report warnings even before the remote is set.
+        /// </summary>
+        [Fact]
+        public void WithNoRemoteNoCommit()
+        {
+            var testAsset = _testAssetsManager
+                .CopyTestAsset("SourceLinkTestApp")
+                .WithSource();
+
+            CreateGitFiles(testAsset.Path, originUrl: null, commitSha: null);
+
+            var buildCommand = new BuildCommand(testAsset);
+            buildCommand.Execute().Should().HaveStdOutContaining($"warning : Repository '{testAsset.Path}' has no remote.");
+
+            var intermediateDir = buildCommand.GetIntermediateDirectory();
+            intermediateDir.Should().NotHaveFile("SourceLinkTestApp.sourcelink.json");
+        }
+
+        /// <summary>
+        /// When creating a new repository locally we want the build to work and not report warnings even before the remote is set.
+        /// </summary>
         [Fact]
         public void WithNoRemote()
         {