Making UsageModel a class again & correcting tests

It made sense to have UsageModel be a struct before because it was a
singular value. Now it is an element in an array, it is much easier to
use an object in this situation.

Author: StanleyGoldman

src/GitHub.Exports/Models/UsageModel.cs

@@ -2,8 +2,7 @@
 
 namespace GitHub.Models
 {
-    [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Performance", "CA1815:OverrideEqualsAndOperatorEqualsOnValueTypes", Justification = "It'll use reflection by default and we're fine with that")]
-    public struct UsageModel
+    public class UsageModel
     {
         public DimensionsModel Dimensions { get; set; }
         public MeasuresModel Measures { get; set; }

src/GitHub.VisualStudio/Services/UsageService.cs

@@ -77,11 +77,6 @@ public async Task<Guid> GetUserGuid()
             return userGuid.Value;
         }
 
-        public bool IsToday(DateTimeOffset date)
-        {
-            return date.Date == DateTimeOffset.Now.Date;
-        }
-
         public IDisposable StartTimer(Func<Task> callback, TimeSpan dueTime, TimeSpan period)
         {
             return new Timer(

src/GitHub.VisualStudio/Services/UsageTracker.cs

@@ -126,7 +126,7 @@ async Task<UsageModel> GetCurrentReport(UsageData data)
         {
             var current = data.Reports.FirstOrDefault(x => x.Dimensions.Date.Date == DateTimeOffset.Now.Date);
 
-            if (Guid.Empty.Equals(current.Dimensions.Guid))
+            if (current == null)
             {
                 var guid = await service.GetUserGuid();
                 current = UsageModel.Create(guid);
@@ -140,12 +140,12 @@ async Task<UsageModel> GetCurrentReport(UsageData data)
 
             if (connectionManager.Connections.Any(x => x.HostAddress.IsGitHubDotCom()))
             {
-                current.Dimensions.IsGitHubUser |= true;
+                current.Dimensions.IsGitHubUser = true;
             }
 
             if (connectionManager.Connections.Any(x => !x.HostAddress.IsGitHubDotCom()))
             {
-                current.Dimensions.IsEnterpriseUser |= true;
+                current.Dimensions.IsEnterpriseUser = true;
             }
 
             return current;