Merge pull request #19 from JSkimming/CodeAnalysis

Introduced code analysis inspection on build

Author: JSkimming

src/AutoTest.ArgumentNullException.Xunit/AutoTest.ArgumentNullException.Xunit.csproj

@@ -20,6 +20,7 @@
     <DefineConstants>DEBUG;TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <RunCodeAnalysis>true</RunCodeAnalysis>
     <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
     <DocumentationFile>bin\Debug\AutoTest.ArgumentNullException.Xunit.xml</DocumentationFile>
   </PropertyGroup>
@@ -30,7 +31,9 @@
     <DefineConstants>TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <RunCodeAnalysis>true</RunCodeAnalysis>
     <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
+    <CodeAnalysisTreatWarningsAsErrors Condition="'$(CodeAnalysisTreatWarningsAsErrors)'==''">true</CodeAnalysisTreatWarningsAsErrors>
     <DocumentationFile>bin\Release\AutoTest.ArgumentNullException.Xunit.xml</DocumentationFile>
   </PropertyGroup>
   <ItemGroup>

src/AutoTest.ArgumentNullException/ArgNullExCompositeCustomization.cs

@@ -49,7 +49,7 @@ public IEnumerable<IArgNullExCustomization> Customizations
         /// </summary>
         /// <param name="fixture">The fixture to customize.</param>
         /// <exception cref="ArgumentNullException">The <paramref name="fixture"/> parameter is <see langword="null"/>.</exception>
-        void IArgNullExCustomization.Customize(IArgumentNullExceptionFixture fixture)
+        public void Customize(IArgumentNullExceptionFixture fixture)
         {
             if (fixture == null)
                 throw new ArgumentNullException("fixture");

src/AutoTest.ArgumentNullException/AutoTest.ArgumentNullException.csproj

@@ -20,6 +20,7 @@
     <DefineConstants>DEBUG;TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <RunCodeAnalysis>true</RunCodeAnalysis>
     <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
     <DocumentationFile>bin\Debug\AutoTest.ArgumentNullException.xml</DocumentationFile>
   </PropertyGroup>
@@ -30,7 +31,9 @@
     <DefineConstants>TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <RunCodeAnalysis>true</RunCodeAnalysis>
     <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
+    <CodeAnalysisTreatWarningsAsErrors Condition="'$(CodeAnalysisTreatWarningsAsErrors)'==''">true</CodeAnalysisTreatWarningsAsErrors>
     <DocumentationFile>bin\Release\AutoTest.ArgumentNullException.xml</DocumentationFile>
   </PropertyGroup>
   <ItemGroup>

src/AutoTest.ArgumentNullException/CompositionException.cs

@@ -8,6 +8,7 @@
     /// <summary>
     /// An <see cref="Exception"/> representing an error composing a method execution.
     /// </summary>
+    [Serializable]
     public class CompositionException : Exception
     {
         /// <summary>

src/AutoTest.ArgumentNullException/Execution/DefaultExecutionSetup.cs

@@ -10,7 +10,7 @@
     /// <summary>
     /// The default <see cref="IExecutionSetup"/>.
     /// </summary>
-    public class DefaultExecutionSetup : IExecutionSetup
+    public sealed class DefaultExecutionSetup : IExecutionSetup
     {
         /// <summary>
         /// A singleton completed task.

src/AutoTest.ArgumentNullException/Execution/ErroredExecutionSetup.cs

@@ -8,7 +8,7 @@
     /// <summary>
     /// The <see cref="IExecutionSetup"/> used to throw a setup error.
     /// </summary>
-    public class ErroredExecutionSetup : IExecutionSetup
+    public sealed class ErroredExecutionSetup : IExecutionSetup
     {
         /// <summary>
         /// Initializes a new instance of the <see cref="ErroredExecutionSetup"/> class.

src/AutoTest.ArgumentNullException/Filter/HasNullableParametersWithoutNullDefaults.cs

@@ -8,7 +8,7 @@
     /// <summary>
     /// Filters out the methods that have nullable parameters, but have null defaults.
     /// </summary>
-    public class HasNullableParametersWithoutNullDefaults : FilterBase, IMethodFilter
+    public sealed class HasNullableParametersWithoutNullDefaults : FilterBase, IMethodFilter
     {
         /// <summary>
         /// Filters out the methods that have nullable parameters, but have null defaults.

src/AutoTest.ArgumentNullException/Filter/IsClassOrStruct.cs

@@ -7,7 +7,7 @@
     /// <summary>
     /// Filters out types that are not classes or structs.
     /// </summary>
-    public class IsClassOrStruct : FilterBase, ITypeFilter
+    public sealed class IsClassOrStruct : FilterBase, ITypeFilter
     {
         /// <summary>
         /// Filters out types that are not classes.

src/AutoTest.ArgumentNullException/Filter/IsNullable.cs

@@ -8,7 +8,7 @@
     /// <summary>
     /// Filters out parameters that are not nullable.
     /// </summary>
-    public class IsNullable : FilterBase, IParameterFilter
+    public sealed class IsNullable : FilterBase, IParameterFilter
     {
         /// <summary>
         /// Filters out parameters that are not nullable.

src/AutoTest.ArgumentNullException/Filter/NotAbstractMethod.cs

@@ -8,7 +8,7 @@
     /// <summary>
     /// Filters out abstract methods.
     /// </summary>
-    public class NotAbstractMethod : FilterBase, IMethodFilter
+    public sealed class NotAbstractMethod : FilterBase, IMethodFilter
     {
         /// <summary>
         /// Filters out abstract methods.

src/AutoTest.ArgumentNullException/Filter/NotCompilerGenerated.cs

@@ -8,7 +8,7 @@
     /// <summary>
     /// Filters out methods that are compiler generated.
     /// </summary>
-    public class NotCompilerGenerated : FilterBase, ITypeFilter, IMethodFilter
+    public sealed class NotCompilerGenerated : FilterBase, ITypeFilter, IMethodFilter
     {
         /// <summary>
         /// Filters out types that are compiler generated.

src/AutoTest.ArgumentNullException/Filter/NotNullDefault.cs

@@ -8,7 +8,7 @@
     /// <summary>
     /// Filters out parameters that have a <see langword="null"/> default value.
     /// </summary>
-    public class NotNullDefault : FilterBase, IParameterFilter
+    public sealed class NotNullDefault : FilterBase, IParameterFilter
     {
         /// <summary>
         /// Filters out parameters that have a <see langword="null"/> default value.

src/AutoTest.ArgumentNullException/Filter/NotOutParameter.cs

@@ -8,7 +8,7 @@
     /// <summary>
     /// Filters out output parameters.
     /// </summary>
-    public class NotOutParameter : FilterBase, IParameterFilter
+    public sealed class NotOutParameter : FilterBase, IParameterFilter
     {
         /// <summary>
         /// Filters out output parameters.

src/AutoTest.ArgumentNullException/Filter/NotPropertySetter.cs

@@ -8,7 +8,7 @@
     /// <summary>
     /// Filters out property setters.
     /// </summary>
-    public class NotPropertySetter : FilterBase, IMethodFilter
+    public sealed class NotPropertySetter : FilterBase, IMethodFilter
     {
         /// <summary>
         /// Filters out property setters.

src/AutoTest.ArgumentNullException/Filter/RegexFilter.cs

@@ -9,7 +9,7 @@
     /// <summary>
     /// A filter to include or exclude using <see cref="Regex"/> matching.
     /// </summary>
-    public class RegexFilter : FilterBase, ITypeFilter, IMethodFilter, IParameterFilter, IRegexFilter
+    public sealed class RegexFilter : FilterBase, ITypeFilter, IMethodFilter, IParameterFilter, IRegexFilter
     {
         /// <summary>
         /// The list of rules.

src/AutoTest.ArgumentNullException/SpecimenProvider.cs

@@ -10,7 +10,7 @@
     /// <summary>
     /// Provides parameter and instance specimens for a null parameter invocation of a method.
     /// </summary>
-    public class SpecimenProvider : ISpecimenProvider
+    public sealed class SpecimenProvider : ISpecimenProvider
     {
         /// <summary>
         /// The specimen builder.