Unit test: added new test methods to check if saving diagnostic data works fine.

Author: peetery

DataTest/DataTest.cs

@@ -40,5 +40,20 @@ public void getTableDataTest()
             Assert.AreEqual (width, tableData.Width);
             Assert.AreEqual(height , tableData.Height);
         }
+
+        [Test]
+        public async Task LogDiagnosticDataAsyncTest()
+        {
+            var mockDataAPI = new MockDataAPI();
+            string testData = "{\"Event\":\"Test\"}";
+
+            await mockDataAPI.LogDiagnosticDataAsync(testData);
+
+            List<string> loggedData = mockDataAPI.GetLoggedData();
+
+            Assert.IsNotNull(loggedData);
+            Assert.AreEqual(1, loggedData.Count);
+            Assert.AreEqual(testData, loggedData.First());
+        }
     }
 }

DataTest/MockDataAPI.cs

@@ -0,0 +1,30 @@
+using Data;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace DataTest
+{
+    public class MockDataAPI : DataAPI
+    {
+        private readonly List<string> _loggedData = new List<string>();
+        private readonly SemaphoreSlim _fileSemaphore = new SemaphoreSlim(1, 1);
+
+        public override async Task LogDiagnosticDataAsync(string data)
+        {
+            await _fileSemaphore.WaitAsync();
+            try
+            {
+                _loggedData.Add(data);
+            }
+            finally
+            {
+                _fileSemaphore.Release();
+            }
+        }
+
+        public List<string> GetLoggedData() => _loggedData;
+    }
+}

Logic/LogicAPI.cs

@@ -16,11 +16,13 @@ public class LogicAPI : LogicAbstractAPI
         public override Table table { get; }
         public override ObservableCollection<BallLogic> balls { get; } = new ObservableCollection<BallLogic>();
 
-        public LogicAPI()
+        public LogicAPI() : this(DataAPI.createDataAPI()) { }
+
+        public LogicAPI(DataAPI dataAPI)
         {
-           _dataAPI = DataAPI.createDataAPI();
+            _dataAPI = dataAPI;
             table = _dataAPI.getTableData(500, 500);
-           BallLogic.SetTable(table);
+            BallLogic.SetTable(table);
         }
 
         public override void RunSimulation()

LogicTest/LogicTest.cs

@@ -2,6 +2,7 @@
 using Data;
 using System.Collections.ObjectModel;
 using System.Numerics;
+using DataTest;
 
 namespace LogicTest
 {
@@ -86,5 +87,21 @@ public void createDeleteBallsTest()
             _logicAPI.deleteBalls();
             Assert.AreEqual(0, _logicAPI.balls.Count);
         }
+
+        [Test]
+        public async Task RunSimulation_LogDataTest()
+        {
+            var mockDataAPI = new MockDataAPI();
+            var logicAPI = new LogicAPI(mockDataAPI);
+            logicAPI.createBalls(2, 10);
+
+            logicAPI.RunSimulation();
+            await Task.Delay(100);
+            logicAPI.StopSimulation();
+
+            List<string> loggedData = mockDataAPI.GetLoggedData();
+
+            Assert.IsNotNull(loggedData);
+        }
     }
 }

LogicTest/LogicTest.csproj

@@ -18,6 +18,7 @@
   </ItemGroup>
 
   <ItemGroup>
+    <ProjectReference Include="..\DataTest\DataTest.csproj" />
     <ProjectReference Include="..\Data\Data.csproj" />
     <ProjectReference Include="..\Logic\Logic.csproj" />
   </ItemGroup>