Merge pull request #504 from BHoM/Test_Toolkit-#502-FixCodeCompliance

Fix code compliance on oM

Author: IsakNaslundBh

CodeComplianceTest_Engine/CodeComplianceTest_Engine.csproj

@@ -156,7 +156,6 @@
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="Query\Checks\AttributeHasEndingPunctuation.cs" />
     <Compile Include="Query\Checks\HasConstructor.cs" />
-    <Compile Include="Query\Checks\HasOneConstructor.cs" />
     <Compile Include="Query\Checks\HasUniqueMultiOutputAttributes.cs" />
     <Compile Include="Query\Checks\HasValidConstructor.cs" />
     <Compile Include="Query\Checks\HasValidCopyright.cs" />
@@ -171,7 +170,6 @@
     <Compile Include="Query\Checks\EngineClassMatchesFilePath.cs" />
     <Compile Include="Query\Checks\HasValidOutputAttribute.cs" />
     <Compile Include="Query\Checks\IsInputAttributePresent.cs" />
-    <Compile Include="Query\Checks\HasSingleClass.cs" />
     <Compile Include="Query\Checks\InputAttributeIsUnique.cs" />
     <Compile Include="Query\Checks\InputParameterStartsLower.cs" />
     <Compile Include="Query\Checks\IsValidIImmutableObject.cs" />
@@ -289,4 +287,4 @@ xcopy "$(TargetDir)System.Threading.Tasks.Extensions.dll"  "C:\ProgramData\BHoM\
   <Target Name="AfterBuild">
   </Target>
   -->
-</Project>
+</Project>

CodeComplianceTest_Engine/Query/Checks/HasConstructor.cs

@@ -44,7 +44,7 @@ public static partial class Query
         [Path(@"([a-zA-Z0-9]+)_Tests\\.*\.cs$", false)]     //NUnit style projects
         [Path(@"([a-zA-Z0-9]+)_Test\\.*\.cs$", false)]      //Verification projects
         [ComplianceType("code")]
-        [ErrorLevel(TestStatus.Error)]
+        [ErrorLevel(TestStatus.Warning)]
         [Output("A span that represents where this error resides or null if there is no error")]
         public static Span HasConstructor(this ClassDeclarationSyntax node)
         {

CodeComplianceTest_Engine/Query/Checks/HasOneConstructor.cs

@@ -44,7 +44,7 @@ public static partial class Query
         [Path(@"([a-zA-Z0-9]+)_Tests\\.*\.cs$", false)]     //NUnit style projects
         [Path(@"([a-zA-Z0-9]+)_Test\\.*\.cs$", false)]      //Verification projects
         [ComplianceType("code")]
-        [ErrorLevel(TestStatus.Error)]
+        [ErrorLevel(TestStatus.Warning)]
         [Output("A span that represents where this error resides or null if there is no error")]
         public static Span HasValidConstructor(this ClassDeclarationSyntax node)
         {

CodeComplianceTest_Engine/Query/Checks/HasSingleClass.cs

@@ -38,7 +38,7 @@ namespace BH.Engine.Test.CodeCompliance.Checks
     public static partial class Query
     {
         [Message("IImutable objects must have at least one property with only a get accessor (no set accessor).", "IsValidIImmutableObject")]
-        [ErrorLevel(TestStatus.Error)]
+        [ErrorLevel(TestStatus.Warning)]
         [Path(@"([a-zA-Z0-9]+)_?oM\\.*\.cs$")]
         [Path(@"([a-zA-Z0-9]+)_Engine\\.*\.cs$", false)]
         [Path(@"([a-zA-Z0-9]+)_Adapter\\.*\.cs$", false)]