Added unit tests

Author: jamescrosswell

src/Sentry.Android.AssemblyReader/V2/AssemblyStoreReader.cs

@@ -45,7 +45,7 @@ protected AssemblyStoreReader(Stream store, string path, DebugLogger? logger)
 
     protected BinaryReader CreateReader() => new BinaryReader(StoreStream, ReaderEncoding, leaveOpen: true);
 
-    protected abstract bool IsSupported();
+    protected internal abstract bool IsSupported();
     protected abstract void Prepare();
     protected abstract ulong GetStoreStartDataOffset();
 

src/Sentry.Android.AssemblyReader/V2/StoreReader.Classes.cs

@@ -31,7 +31,7 @@ public Header(uint magic, uint version, uint entry_count, uint index_entry_count
         }
     }
 
-    private sealed class IndexEntry
+    internal sealed class IndexEntry
     {
         public readonly ulong name_hash;
         public readonly uint descriptor_index;
@@ -45,7 +45,7 @@ public IndexEntry(ulong name_hash, uint descriptor_index, bool ignore)
         }
     }
 
-    private sealed class EntryDescriptor
+    internal sealed class EntryDescriptor
     {
         public uint mapping_index;
 
@@ -59,7 +59,7 @@ private sealed class EntryDescriptor
         public uint config_data_size;
     }
 
-    private sealed class StoreItemV2 : AssemblyStoreItem
+    internal sealed class StoreItemV2 : AssemblyStoreItem
     {
         public StoreItemV2(AndroidTargetArch targetArch, string name, bool is64Bit, List<IndexEntry> indexEntries,
             EntryDescriptor descriptor, bool ignore)

src/Sentry.Android.AssemblyReader/V2/StoreReader.cs

@@ -91,7 +91,7 @@ public StoreReader(Stream store, string path, DebugLogger? logger)
 
     protected override ulong GetStoreStartDataOffset() => elfOffset;
 
-    protected override bool IsSupported()
+    protected internal override bool IsSupported()
     {
         lock (StreamLock)
         {

test/Sentry.Android.AssemblyReader.Tests/StoreReaderTests.cs

@@ -0,0 +1,50 @@
+using Sentry.Android.AssemblyReader.V2;
+
+namespace Sentry.Android.AssemblyReader.Tests;
+
+public class StoreReaderTests
+{
+    [Fact]
+    public void IsSupported_Concurrent_IsThreadSafe()
+    {
+        // Arrange
+        var buffer = new byte[1024*1024];
+        var memoryStream = new MemoryStream(buffer);
+        var storeReader = new StoreReader(memoryStream, "testStore", null);
+
+        // Act
+        Parallel.For(0, 10, _ => storeReader.IsSupported());
+
+        // No Assert - test passes if no exceptions are thrown
+    }
+
+    [Fact]
+    public void ReadEntryImageData_Concurrent_IsThreadSafe()
+    {
+        // Arrange
+        var buffer = new byte[1024*1024];
+        var memoryStream = new MemoryStream(buffer);
+        var storeReader = new StoreReader(memoryStream, "testStore", null);
+        var entry = new StoreReader.StoreItemV2(
+            AndroidTargetArch.Arm64,
+            "testAssembly.dll",
+            is64Bit: true,
+            new List<StoreReader.IndexEntry>(),
+            new StoreReader.EntryDescriptor
+            {
+                data_offset = 0,
+                data_size = 0,
+                debug_data_offset = 0,
+                debug_data_size = 0,
+                config_data_offset = 0,
+                config_data_size = 0,
+                mapping_index = 0
+            },
+            ignore: false);
+
+        // Act
+        Parallel.For(0, 10, _ => storeReader.ReadEntryImageData(entry));
+
+        // No Assert - test passes if no exceptions are thrown
+    }
+}