Moved the tests for copy ctors to separate classes.

Signed-off-by: Dimitar Dobrev <dpldobrev@yahoo.com>

src/Generator/Passes/CheckStaticClass.cs

@@ -1,5 +1,4 @@
 using System.Linq;
-using System.Threading;
 using CppSharp.AST;
 
 namespace CppSharp.Passes

tests/Basic/Basic.Tests.cs

@@ -202,10 +202,10 @@ public class BasicTests : GeneratorTestFixture
         Assert.That(foo.A, Is.EqualTo(copyFoo.A));
         Assert.That(foo.B, Is.EqualTo(copyFoo.B));
 
-        Bar bar = new Bar { A = 10, B = 5 };
+        var testCopyConstructorRef = new TestCopyConstructorRef { A = 10, B = 5 };
-        var copyBar = new Bar(bar);
+        var copyBar = new TestCopyConstructorRef(testCopyConstructorRef);
-        Assert.That(bar.A, Is.EqualTo(copyBar.A));
+        Assert.That(testCopyConstructorRef.A, Is.EqualTo(copyBar.A));
-        Assert.That(bar.B, Is.EqualTo(copyBar.B));
+        Assert.That(testCopyConstructorRef.B, Is.EqualTo(copyBar.B));
     }
 }
  

tests/Basic/Basic.cpp

@@ -27,12 +27,6 @@ Bar::Bar()
 {
 }
 
-Bar::Bar(const Bar& bar)
-{
-    A = bar.A;
-    B = bar.B;
-}
-
 Bar::Item Bar::RetItem1()
 {
     return Bar::Item1;
@@ -223,3 +217,13 @@ Bar::Item operator |(Bar::Item left, Bar::Item right)
 {
     return left | right;
 }
+
+TestCopyConstructorRef::TestCopyConstructorRef()
+{
+}
+
+TestCopyConstructorRef::TestCopyConstructorRef(const TestCopyConstructorRef& other)
+{
+    A = other.A;
+    B = other.B;
+}

tests/Basic/Basic.h

@@ -28,7 +28,6 @@ struct DLL_API Bar
     };
 
     Bar();
-    Bar(const Bar& bar);
     Item RetItem1();
     int A;
     float B;
@@ -279,3 +278,12 @@ class DependentTypeWithNestedIndependent
         long l;
     };
 };
+
+class DLL_API TestCopyConstructorRef
+{
+public:
+    TestCopyConstructorRef();
+    TestCopyConstructorRef(const TestCopyConstructorRef& other);
+    int A;
+    float B;
+};

tests/CSharpTemp/CSharpTemp.Tests.cs

@@ -107,4 +107,13 @@ public class CSharpTempTests : GeneratorTestFixture
         bar.ArrayOfPrimitivePointers = array;
         Assert.That(i, Is.EqualTo(*(int*) bar.ArrayOfPrimitivePointers[0]));
     }
+
+    [Test]
+    public void TestCopyConstructorValue()
+    {
+        var testCopyConstructorVal = new TestCopyConstructorVal { A = 10, B = 5 };
+        var copyBar = new TestCopyConstructorVal(testCopyConstructorVal);
+        Assert.That(testCopyConstructorVal.A, Is.EqualTo(copyBar.A));
+        Assert.That(testCopyConstructorVal.B, Is.EqualTo(copyBar.B));
+    }
 }

tests/CSharpTemp/CSharpTemp.cpp

@@ -190,4 +190,14 @@ void P::setIsBool(bool value)
 
 }
 
-int TestDestructors::Marker = 0;
+int TestDestructors::Marker = 0;
+
+TestCopyConstructorVal::TestCopyConstructorVal()
+{
+}
+
+TestCopyConstructorVal::TestCopyConstructorVal(const TestCopyConstructorVal& other)
+{
+    A = other.A;
+    B = other.B;
+}

tests/CSharpTemp/CSharpTemp.cs

@@ -64,6 +64,11 @@ namespace CppSharp.Tests
             driver.TranslationUnitPasses.AddPass(new TestAttributesPass());
         }
 
+        public override void Preprocess(Driver driver, ASTContext ctx)
+        {
+            ctx.SetClassAsValueType("TestCopyConstructorVal");
+        }
+
         public override void Postprocess(Driver driver, ASTContext lib)
         {
             new CaseRenamePass(

tests/CSharpTemp/CSharpTemp.h

@@ -138,3 +138,12 @@ struct DLL_API TestDestructors
     TestDestructors() { Marker = 0xf00d; }
     ~TestDestructors() { Marker = 0xcafe; }
 };
+
+class DLL_API TestCopyConstructorVal
+{
+public:
+    TestCopyConstructorVal();
+    TestCopyConstructorVal(const TestCopyConstructorVal& other);
+    int A;
+    float B;
+};