Merge pull request #1956 from github/metrics/NumberOfClonesToDefaultClonePath

Count the number of clones to the default clone path

Author: jcansdale

src/GitHub.App/Services/RepositoryCloneService.cs

@@ -135,6 +135,12 @@ public async Task CloneRepository(
                     // If it isn't a GitHub URL, assume it's an Enterprise URL
                     await usageTracker.IncrementCounter(x => x.NumberOfEnterpriseClones);
                 }
+
+                if (repositoryPath.StartsWith(DefaultClonePath, StringComparison.OrdinalIgnoreCase))
+                {
+                    // Count the number of times users clone into the Default Repository Location
+                    await usageTracker.IncrementCounter(x => x.NumberOfClonesToDefaultClonePath);
+                }
             }
             catch (Exception ex)
             {

src/GitHub.Exports/Models/UsageModel.cs

@@ -91,6 +91,7 @@ public class MeasuresModel
             public int NumberOfCloneViewUrlTab { get; set; }
             public int NumberOfGitHubClones { get; set; }
             public int NumberOfEnterpriseClones { get; set; }
+            public int NumberOfClonesToDefaultClonePath { get; set; }
         }
     }
 }

test/GitHub.App.UnitTests/Services/RepositoryCloneServiceTests.cs

@@ -49,5 +49,25 @@ await usageTracker.Received(numberOfCalls).IncrementCounter(
                 Arg.Is<Expression<Func<UsageModel.MeasuresModel, int>>>(x =>
                     ((MemberExpression)x.Body).Member.Name == counterName));
         }
+
+        [TestCase(@"c:\default\repo", @"c:\default", 1, nameof(UsageModel.MeasuresModel.NumberOfClonesToDefaultClonePath))]
+        [TestCase(@"c:\not_default\repo", @"c:\default", 0, nameof(UsageModel.MeasuresModel.NumberOfClonesToDefaultClonePath))]
+        public async Task UpdatesMetricsWhenDefaultClonePath(string targetPath, string defaultPath, int numberOfCalls, string counterName)
+        {
+            var serviceProvider = Substitutes.ServiceProvider;
+            var operatingSystem = serviceProvider.GetOperatingSystem();
+            var vsGitServices = serviceProvider.GetVSGitServices();
+            vsGitServices.GetLocalClonePathFromGitProvider().Returns(defaultPath);
+            var graphqlFactory = Substitute.For<IGraphQLClientFactory>();
+            var usageTracker = Substitute.For<IUsageTracker>();
+            var cloneService = new RepositoryCloneService(operatingSystem, vsGitServices, graphqlFactory, usageTracker);
+
+            await cloneService.CloneRepository("https://github.com/foo/bar", targetPath);
+            var model = UsageModel.Create(Guid.NewGuid());
+
+            await usageTracker.Received(numberOfCalls).IncrementCounter(
+                Arg.Is<Expression<Func<UsageModel.MeasuresModel, int>>>(x =>
+                    ((MemberExpression)x.Body).Member.Name == counterName));
+        }
     }
 }