Sorry for the large Commit

* Cleanup using Resharper (code reformat for future pull requests and readability)
* Added background jobs to delete old Reports
* Migrated to claims based authorization (ASP.NET Identity without EF)

Author: gauffininteractive

src/Server/OneTrueError.Api.Client.Tests/Properties/AssemblyInfo.cs

@@ -1,10 +1,10 @@
 using System.Reflection;
-using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices;
 
 // General Information about an assembly is controlled through the following 
 // set of attributes. Change these attribute values to modify the information
 // associated with an assembly.
+
 [assembly: AssemblyTitle("OneTrueError.Api.Client.Tests")]
 [assembly: AssemblyDescription("")]
 [assembly: AssemblyConfiguration("")]
@@ -17,9 +17,11 @@
 // Setting ComVisible to false makes the types in this assembly not visible 
 // to COM components.  If you need to access a type in this assembly from 
 // COM, set the ComVisible attribute to true on that type.
+
 [assembly: ComVisible(false)]
 
 // The following GUID is for the ID of the typelib if this project is exposed to COM
+
 [assembly: Guid("62989500-31bc-44fa-97ca-84f484c6f1aa")]
 
 // Version information for an assembly consists of the following four values:
@@ -32,5 +34,6 @@
 // You can specify all the values or you can default the Build and Revision Numbers 
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
+
 [assembly: AssemblyVersion("1.0.0.0")]
-[assembly: AssemblyFileVersion("1.0.0.0")]
+[assembly: AssemblyFileVersion("1.0.0.0")]

src/Server/OneTrueError.Api.Client.Tests/TryTheClient.cs

@@ -1,8 +1,5 @@
 using System;
-using System.Collections.Generic;
-using System.Linq;
 using System.Net;
-using System.Text;
 using System.Threading.Tasks;
 using FluentAssertions;
 using OneTrueError.Api.Core.Accounts.Queries;
@@ -16,21 +13,20 @@ public class TryTheClient
         [Fact]
         public async Task Test()
         {
-            OneTrueApiClient client = new OneTrueApiClient();
+            var client = new OneTrueApiClient();
             client.Open(new Uri("http://localhost/onetrueerror/"), "", "");
             FindAccountByUserNameResult result = null;
             try
             {
                 result = await client.QueryAsync(new FindAccountByUserName("admin"));
             }
-            catch (WebException ex) 
+            catch (WebException ex)
             {
-                
             }
 
 
             result.Should().NotBeNull();
         }
     }
 #endif
-}
+}

src/Server/OneTrueError.Api.Client.Tests/packages.config

@@ -1,4 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
+
 <packages>
   <package id="DotNetCqs" version="1.0.0" targetFramework="net452" />
   <package id="FluentAssertions" version="4.14.0" targetFramework="net452" />

src/Server/OneTrueError.Api.Client/OneTrueClient.cs

@@ -14,23 +14,23 @@ namespace OneTrueError.Api.Client
     /// </summary>
     public class OneTrueApiClient : IQueryBus, ICommandBus, IEventBus
     {
-        private string _apiKey;
-
         private readonly JsonSerializerSettings _jsonSerializerSettings = new JsonSerializerSettings
         {
             ConstructorHandling = ConstructorHandling.AllowNonPublicDefaultConstructor,
             Formatting = Formatting.Indented
         };
 
+        private string _apiKey;
+
         private string _sharedSecret;
         private Uri _uri;
 
 
         /// <summary>
         ///     Creates a new instance of <see cref="OneTrueApiClient" />.
         /// </summary>
-        public OneTrueApiClient() { 
-
+        public OneTrueApiClient()
+        {
             _jsonSerializerSettings.ContractResolver = new IncludeNonPublicMembersContractResolver();
         }
 
@@ -111,9 +111,9 @@ private async Task<HttpWebResponse> RequestAsync(string httpMethod, string cqsTy
             var json = JsonConvert.SerializeObject(cqsObject, _jsonSerializerSettings);
             var buffer = Encoding.UTF8.GetBytes(json);
 
-var hamc = new HMACSHA256(Encoding.UTF8.GetBytes(_sharedSecret.ToLower()));
-var hash = hamc.ComputeHash(buffer);
-var signature = Convert.ToBase64String(hash);
+            var hamc = new HMACSHA256(Encoding.UTF8.GetBytes(_sharedSecret.ToLower()));
+            var hash = hamc.ComputeHash(buffer);
+            var signature = Convert.ToBase64String(hash);
 
             await stream.WriteAsync(buffer, 0, buffer.Length);
 

src/Server/OneTrueError.Api.Client/Properties/AssemblyInfo.cs

@@ -1,10 +1,10 @@
 using System.Reflection;
-using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices;
 
 // General Information about an assembly is controlled through the following 
 // set of attributes. Change these attribute values to modify the information
 // associated with an assembly.
+
 [assembly: AssemblyTitle("OneTrueError.Api.Client")]
 [assembly: AssemblyDescription("")]
 [assembly: AssemblyConfiguration("")]
@@ -17,9 +17,11 @@
 // Setting ComVisible to false makes the types in this assembly not visible 
 // to COM components.  If you need to access a type in this assembly from 
 // COM, set the ComVisible attribute to true on that type.
+
 [assembly: ComVisible(false)]
 
 // The following GUID is for the ID of the typelib if this project is exposed to COM
+
 [assembly: Guid("017f8863-3de0-4ad2-9ed3-5acb87bbbcd0")]
 
 // Version information for an assembly consists of the following four values:
@@ -32,5 +34,6 @@
 // You can specify all the values or you can default the Build and Revision Numbers 
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
+
 [assembly: AssemblyVersion("1.0.0.0")]
-[assembly: AssemblyFileVersion("1.0.0.0")]
+[assembly: AssemblyFileVersion("1.0.0.0")]

src/Server/OneTrueError.Api.Client/StringBuilderExtensions.cs

@@ -1,12 +1,9 @@
 using System;
-using System.Collections.Generic;
-using System.Linq;
 using System.Text;
-using System.Threading.Tasks;
 
-namespace OneTrueError.Api.Client.Tests
+namespace OneTrueError.Api.Client
 {
-    static class StringBuilderExtensions
+    internal static class StringBuilderExtensions
     {
         public static void AppendUrlEncoded(this StringBuilder sb, string name, string value)
         {
@@ -16,6 +13,5 @@ public static void AppendUrlEncoded(this StringBuilder sb, string name, string v
             sb.Append("=");
             sb.Append(Uri.EscapeDataString(value));
         }
-
     }
-}
+}

src/Server/OneTrueError.Api.Client/packages.config

@@ -1,4 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
+
 <packages>
   <package id="DotNetCqs" version="1.0.0" targetFramework="net452" />
   <package id="Newtonsoft.Json" version="9.0.1" targetFramework="net452" />

src/Server/OneTrueError.Api/Core/Accounts/Commands/RegisterAccount.cs

@@ -4,12 +4,12 @@
 namespace OneTrueError.Api.Core.Accounts.Commands
 {
     /// <summary>
-    /// Register a new account and send out an activation email.
+    ///     Register a new account and send out an activation email.
     /// </summary>
     public class RegisterAccount : Command
     {
         /// <summary>
-        /// Creates a new instance of <see cref="RegisterAccount"/>
+        ///     Creates a new instance of <see cref="RegisterAccount" />
         /// </summary>
         /// <param name="userName">User name</param>
         /// <param name="password">Password as entered by the user</param>
@@ -25,26 +25,25 @@ public RegisterAccount(string userName, string password, string email)
         }
 
         /// <summary>
-        /// Serialization constructor.
+        ///     Serialization constructor.
         /// </summary>
         protected RegisterAccount()
         {
-            
         }
 
         /// <summary>
-        /// User name
+        ///     Email address.
         /// </summary>
-        public string UserName { get; private set; }
+        public string Email { get; private set; }
 
         /// <summary>
-        /// Password as entered by the user.
+        ///     Password as entered by the user.
         /// </summary>
         public string Password { get; private set; }
 
         /// <summary>
-        /// Email address.
+        ///     User name
         /// </summary>
-        public string Email { get; private set; }
+        public string UserName { get; private set; }
     }
 }

src/Server/OneTrueError.Api/Core/Accounts/Events/AccountRegistered.cs

@@ -9,7 +9,7 @@ namespace OneTrueError.Api.Core.Accounts.Events
     public class AccountRegistered : ApplicationEvent
     {
         /// <summary>
-        /// Create a new instance of <see cref="AccountRegistered"/>-
+        ///     Create a new instance of <see cref="AccountRegistered" />-
         /// </summary>
         /// <param name="accountId">Account id (primary key).</param>
         /// <param name="userName">User name as entered by the user.</param>
@@ -22,20 +22,19 @@ public AccountRegistered(int accountId, string userName)
         }
 
         /// <summary>
-        /// Serialization constructor
+        ///     Serialization constructor
         /// </summary>
         protected AccountRegistered()
         {
-            
         }
 
         /// <summary>
-        /// Account id (primary key).
+        ///     Account id (primary key).
         /// </summary>
         public int AccountId { get; private set; }
 
         /// <summary>
-        /// User name as entered by the user.
+        ///     User name as entered by the user.
         /// </summary>
         public string UserName { get; private set; }
     }

src/Server/OneTrueError.Api/Core/Accounts/Events/InvitationAccepted.cs

@@ -9,7 +9,7 @@ namespace OneTrueError.Api.Core.Accounts.Events
     public class InvitationAccepted : ApplicationEvent
     {
         /// <summary>
-        /// Creates a new instance of <see cref="InvitationAccepted"/>.
+        ///     Creates a new instance of <see cref="InvitationAccepted" />.
         /// </summary>
         /// <param name="accountId">account that accepted the inviation</param>
         /// <param name="invitedByUserName">user that made the invite</param>
@@ -27,12 +27,17 @@ public InvitationAccepted(int accountId, string invitedByUserName, string userNa
         }
 
         /// <summary>
-        /// Serialization constructor.
+        ///     Serialization constructor.
         /// </summary>
         protected InvitationAccepted()
         {
         }
 
+        /// <summary>
+        ///     The email that the inviation was accepted by.
+        /// </summary>
+        public string AcceptedEmailAddress { get; set; }
+
         /// <summary>
         ///     Id of the user that accepted the invitation
         /// </summary>
@@ -49,13 +54,13 @@ protected InvitationAccepted()
         public string InvitedByUserName { get; set; }
 
         /// <summary>
-        ///     The user that accepted the invitation
+        ///     Email address that the invitation was sent to.
         /// </summary>
-        public string UserName { get; set; }
+        public string InvitedEmailAddress { get; set; }
 
         /// <summary>
-        ///     Email address of the user that accepted the invitation
+        ///     The user that accepted the invitation
         /// </summary>
-        public string EmailAddress { get; set; }
+        public string UserName { get; set; }
     }
 }