[FIXED #48]

w8tcha · May 6, 2024 · 97ea82f · 97ea82f
1 parent 954f27b
commit 97ea82f
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 15 deletions.
diff --git a/BootstrapEmailNet.Tests/BootstrapEmailTests.cs b/BootstrapEmailNet.Tests/BootstrapEmailTests.cs
@@ -24,12 +24,12 @@ public BootstrapEmailTests(ITestOutputHelper testOutputHelper)
     [Fact]
     public void ParallelTests()
     {
-	    var compiler = new BootstrapEmail();
+		var compiler = new BootstrapEmail();
 
-	    var tasks = Enumerable.Range(0, 50)
-		    .Select(_ => (Action)(() =>
+		var tasks = Enumerable.Range(0, 50)
+			.Select(_ => (Action)(() =>
 			{
-				var html =  compiler.Compile(
+				var html = compiler.Compile(
 					"""<a href="#" class="btn btn-primary">A button</a> <a href="#" class="btn btn-secondary">B button</a>""",
 					string.Empty,
 					InputType.String);
@@ -84,8 +84,8 @@ public void ParallelTests()
 
 				Assert.Equal(expected, html, ignoreLineEndingDifferences: true, ignoreAllWhiteSpace: true);
 			}))
-		    .ToArray();
-	    Parallel.Invoke(tasks);
+			.ToArray();
+		Parallel.Invoke(tasks);
 	}
 
 	/// <summary>

diff --git a/BootstrapEmailNet/Compiler.cs b/BootstrapEmailNet/Compiler.cs
@@ -44,12 +44,17 @@ public class Compiler
     public string InputHtml { get; set; }
 
     /// <summary>
-    /// Initializes a new instance of the <see cref="Compiler"/> class.
+    /// The lock
     /// </summary>
-    /// <param name="input">The input.</param>
-    /// <param name="config">The configuration.</param>
-    /// <param name="type">The type.</param>
-    public Compiler(string input, ConfigStore config, InputType type = InputType.String)
+    private static readonly ReaderWriterLockSlim Lock = new();
+
+	/// <summary>
+	/// Initializes a new instance of the <see cref="Compiler"/> class.
+	/// </summary>
+	/// <param name="input">The input.</param>
+	/// <param name="config">The configuration.</param>
+	/// <param name="type">The type.</param>
+	public Compiler(string input, ConfigStore config, InputType type = InputType.String)
     {
 	    this.Config = new Config(config);
 	    this.Type = type;
@@ -266,8 +271,16 @@ private static void WriteResourceToFile(Assembly assembly, string resourceName,
 	        Directory.CreateDirectory(dirName);
 		}
 
-	    using var file = new FileStream(fileName, FileMode.Create, FileAccess.Write);
+	    Lock.EnterWriteLock();
 
-	    resource?.CopyTo(file);
+	    try
+	    {
+		    using var fs = new FileStream(fileName, FileMode.OpenOrCreate, FileAccess.ReadWrite);
+			resource?.CopyTo(fs);
+		}
+	    finally
+	    {
+		    Lock.ExitWriteLock();
+	    }
     }
 }
diff --git a/BootstrapEmailNet/SassCache.cs b/BootstrapEmailNet/SassCache.cs
@@ -3,6 +3,7 @@
 namespace BootstrapEmail.Net;
 
 using System;
+using System.IO;
 using System.Security.Cryptography;
 
 /// <summary>
@@ -68,7 +69,9 @@ public string Compile()
 			        return ReadFile(filePath);
 		        }
 
-		        break;
+		        throw new FileNotFoundException(
+			        $"The Email Path does not exist: {this.config.ConfigStore.CssEmailPath}",
+			        this.config.ConfigStore.CssEmailPath);
 	        }
 	        case SassTypes.Head when !string.IsNullOrEmpty(this.config.ConfigStore.CssHeadPath):
 	        {
@@ -79,7 +82,9 @@ public string Compile()
 			        return ReadFile(filePath);
 		        }
 
-		        break;
+		        throw new FileNotFoundException(
+			        $"The Email Path does not exist: {this.config.ConfigStore.CssHeadPath}",
+			        this.config.ConfigStore.CssEmailPath);
 	        }
         }