Moved sestest specific unit tests to dedicated sestest.Tests project (#108)

azure-batch-software-entitlement.sln

@@ -29,6 +29,8 @@ Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "sesclient.native", "src\ses
 		{6ACCAD67-C207-4370-B39A-63651223A21C} = {6ACCAD67-C207-4370-B39A-63651223A21C}
 	EndProjectSection
 EndProject
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "sestest.Tests", "tests\sestest.Tests\sestest.Tests.csproj", "{45ABCC13-E804-4785-A11E-56F083158AB5}"
+EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
 		Debug|Any CPU = Debug|Any CPU
@@ -133,6 +135,18 @@ Global
 		{DA18B560-4FCD-4C16-9E37-3715EB19C2CE}.Release|x64.Build.0 = Release|x64
 		{DA18B560-4FCD-4C16-9E37-3715EB19C2CE}.Release|x86.ActiveCfg = Release|Win32
 		{DA18B560-4FCD-4C16-9E37-3715EB19C2CE}.Release|x86.Build.0 = Release|Win32
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Debug|x64.ActiveCfg = Debug|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Debug|x64.Build.0 = Debug|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Debug|x86.ActiveCfg = Debug|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Debug|x86.Build.0 = Debug|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Release|Any CPU.Build.0 = Release|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Release|x64.ActiveCfg = Release|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Release|x64.Build.0 = Release|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Release|x86.ActiveCfg = Release|Any CPU
+		{45ABCC13-E804-4785-A11E-56F083158AB5}.Release|x86.Build.0 = Release|Any CPU
 	EndGlobalSection
 	GlobalSection(SolutionProperties) = preSolution
 		HideSolutionNode = FALSE

tests/Microsoft.Azure.Batch.SoftwareEntitlement.Common.Tests/GlobalSuppressions.cs

@@ -29,7 +29,7 @@ using System.Diagnostics.CodeAnalysis;
 [assembly: SuppressMessage(
     "Performance",
     "CA1822:Mark members as static",
-    Justification = "Some tests don't share intiialization but still need to be non-static")]
+    Justification = "Some tests don't share intialization but still need to be non-static")]
 
 [assembly: SuppressMessage(
     "Redundancy",

tests/Microsoft.Azure.Batch.SoftwareEntitlement.Tests/GlobalSuppressions.cs

@@ -24,7 +24,7 @@ using System.Diagnostics.CodeAnalysis;
 [assembly: SuppressMessage(
     "Performance",
     "CA1822:Mark members as static",
-    Justification = "Some tests don't share intiialization but still need to be non-static")]
+    Justification = "Some tests don't share intialization but still need to be non-static")]
 
 [assembly: SuppressMessage(
     "Performance",

tests/Microsoft.Azure.Batch.SoftwareEntitlement.Tests/Microsoft.Azure.Batch.SoftwareEntitlement.Tests.csproj

@@ -22,9 +22,7 @@
 
   <ItemGroup>
     <ProjectReference Include="..\..\src\Microsoft.Azure.Batch.SoftwareEntitlement.Common\Microsoft.Azure.Batch.SoftwareEntitlement.Common.csproj" />
-    <ProjectReference Include="..\..\src\Microsoft.Azure.Batch.SoftwareEntitlement.Server\Microsoft.Azure.Batch.SoftwareEntitlement.Server.csproj" />
     <ProjectReference Include="..\..\src\Microsoft.Azure.Batch.SoftwareEntitlement\Microsoft.Azure.Batch.SoftwareEntitlement.csproj" />
-    <ProjectReference Include="..\..\src\sestest\sestest.csproj" />
   </ItemGroup>
 
   <ItemGroup>

tests/sestest.Tests/GlobalSuppressions.cs

@@ -0,0 +1,32 @@
+
+// This file is used by Code Analysis to maintain SuppressMessage 
+// attributes that are applied to this project.
+// Project-level suppressions either have no target or are given 
+// a specific target and scoped to a namespace, type, member, etc.
+
+using System.Diagnostics.CodeAnalysis;
+
+[assembly: SuppressMessage(
+    "Design",
+    "CA1034:Nested types should not be visible",
+    Justification = "This project uses nested public classes to group related tests.")]
+
+[assembly: SuppressMessage(
+    "Design",
+    "CA1052",
+    Justification = "Grouping tests in nested private classes for structure.")]
+
+[assembly: SuppressMessage(
+    "Naming",
+    "CA1707: Remove the underscores from member names",
+    Justification = "This project uses '_' to separate parts of test names.")]
+
+[assembly: SuppressMessage(
+    "Performance",
+    "CA1822:Mark members as static",
+    Justification = "Some tests don't share intialization but still need to be non-static")]
+
+[assembly: SuppressMessage(
+    "Performance",
+    "RCS1096:Use bitwise operation instead of calling 'HasFlag'.",
+    Justification = "This project prefers the clarity of HasFlag().")]

tests/sestest.Tests/sestest.Tests.csproj

@@ -0,0 +1,29 @@
+<Project Sdk="Microsoft.NET.Sdk">
+
+  <PropertyGroup>
+    <TargetFramework>netcoreapp2.0</TargetFramework>
+    <RootNamespace>Microsoft.Azure.Batch.SoftwareEntitlement.Tests</RootNamespace>
+  </PropertyGroup>
+
+  <ItemGroup>
+    <PackageReference Include="Castle.Core" Version="4.2.1" />
+    <PackageReference Include="FluentAssertions" Version="4.19.4" />
+    <PackageReference Include="Microsoft.AnalyzerPowerPack" Version="1.1.0" />
+    <PackageReference Include="Microsoft.CodeAnalysis.FxCopAnalyzers" Version="2.6.0" />
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="15.6.0" />
+    <PackageReference Include="Newtonsoft.Json" Version="10.0.3" />
+    <PackageReference Include="NSubstitute" Version="3.1.0" />
+    <PackageReference Include="OpenCover" Version="4.6.519" />
+    <PackageReference Include="ReportGenerator" Version="3.1.2" />
+    <PackageReference Include="Roslynator.Analyzers" Version="1.6.0" />
+    <PackageReference Include="System.Runtime.Analyzers" Version="1.1.0" />
+    <PackageReference Include="xunit" Version="2.3.1" />
+    <PackageReference Include="xunit.runner.visualstudio" Version="2.3.1" />
+    <DotNetCliToolReference Include="dotnet-xunit" Version="2.3.1" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <ProjectReference Include="..\..\src\sestest\sestest.csproj" />
+  </ItemGroup>
+
+</Project>