Add Skyline.DataMiner.CICD.Validators.Common.Testing to the Internals…

…VisibleTo (#26)

* Small tweak for the validators

* Enable again for DIS (still needed there)

* Removed InternalsVisibleTo for DIS & Validator and removed unnecessary ctor.

Protocol/Models/Edit/Protocol/Protocol.cs

@@ -9,17 +9,11 @@ public partial class Protocol
             "Name", "Description", "Version","Provider", "Vendor", "VendorOID", "DeviceOID", "ElementType", "Type", "Display", "SNMP",
             "DVEs", "ExportRules", "Params", "QActions", "Groups", "Triggers", "Actions", "Timers", "PortSettings"
         };
-
-        // Used by DIS > SLDisUnitTestsShared > ProtocolTestsHelper.cs
+
         internal Protocol(Read.IProtocol read, EditXml.XmlElement editNode) : base(read, null, editNode)
         {
         }
-
-        // Used by DIS > SLDisUnitTestsShared > ProtocolTestsHelper.cs
-        internal Protocol(Read.IProtocol read) : base(read, null, CreateEditNode(read))
-        {
-        }
-
+
         public override string[] ElementOrder => elementOrder;
 
         protected override void OnCreated()

Protocol/Models/Edit/ProtocolDocumentEdit.cs

@@ -37,15 +37,8 @@ public ProtocolDocumentEdit(IProtocolModel protocolModel, XmlDocument document)
 
         public Protocol Protocol
         {
-            get
-            {
-                return protocol;
-            }
-
-            set
-            {
-                AssignProtocol(value);
-            }
+            get => protocol;
+            set => AssignProtocol(value);
         }
 
         #endregion

Protocol/Protocol.csproj

@@ -47,9 +47,6 @@
 
     <ItemGroup>
         <InternalsVisibleTo Include="$(MSBuildProjectName)Tests" />
-
-        <!-- Needed for DIS -->
-        <InternalsVisibleTo Include="SLDisUnitTestsShared" />
     </ItemGroup>
 
     <ItemGroup>