.less parsing errors now result in a ParserException

src/FubuMVC.Less.Tests/DefaultServicesTester.cs

@@ -27,6 +27,13 @@ public void less_engine()
                 .Value.ShouldBeOfType<LessEngine>();
         }
 
+		[Test]
+		public void engine_should_use_exception_logger() {
+			_services.DefaultServiceFor<ILessEngine>()
+				.Value.As<LessEngine>().Logger
+				.ShouldBeOfType<ExceptionLogger>();
+		}
+
         [Test]
         public void less_compiler()
         {

src/FubuMVC.Less.Tests/LessCompilerTester.cs

@@ -1,8 +1,10 @@
+using System;
 using FubuTestingSupport;
 using NUnit.Framework;
 using Rhino.Mocks;
-using dotless.Core;
-
+using dotless.Core;
+using dotless.Core.Parser;
+
 namespace FubuMVC.Less.Tests
 {
     [TestFixture]
@@ -14,5 +16,14 @@ public void should_use_less_engine()
             ClassUnderTest.Compile(".a{}");
             MockFor<ILessEngine>().AssertWasCalled(x => x.TransformToCss(".a{}", null));
         }
+
+		[Test]
+		public void should_throw_when_invalid_source() {
+			var engine = MockFor<LessEngine>();
+			engine.Parser = new Parser();
+			engine.Logger = new ExceptionLogger();
+			Services.Inject<ILessEngine>(engine);
+			Assert.Throws<ParserException>(() => ClassUnderTest.Compile("bomb()"));
+		}
     }
 }

src/FubuMVC.Less/ExceptionLogger.cs

@@ -0,0 +1,14 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using dotless.Core.Loggers;
+
+namespace FubuMVC.Less {
+	public class ExceptionLogger : Logger {
+		public ExceptionLogger() : base(LogLevel.Error) {}
+		protected override void Log(string message) {
+			throw new ParserException(message);
+		}
+	}
+}

src/FubuMVC.Less/FubuMVC.Less.csproj

@@ -67,10 +67,12 @@
     <Compile Include="..\CommonAssemblyInfo.cs">
       <Link>Properties\CommonAssemblyInfo.cs</Link>
     </Compile>
+    <Compile Include="ExceptionLogger.cs" />
     <Compile Include="LessCompiler.cs" />
     <Compile Include="LessExtension.cs" />
     <Compile Include="LessTransformer.cs" />
     <Compile Include="LessTransformerPolicy.cs" />
+    <Compile Include="ParserException.cs" />
   </ItemGroup>
   <ItemGroup>
     <None Include=".package-manifest" />

src/FubuMVC.Less/LessExtension.cs

@@ -18,7 +18,7 @@ public void Configure(FubuRegistry registry)
 
         private static void registerDefaultServices(IServiceRegistry s)
         {
-            s.SetServiceIfNone<ILessEngine>(new LessEngine());
+            s.SetServiceIfNone<ILessEngine>(new LessEngine(){Logger = new ExceptionLogger()});
             s.SetServiceIfNone<ILessCompiler, LessCompiler>();
             s.AddService<ITransformerPolicy, LessTransformerPolicy>();
         }

src/FubuMVC.Less/ParserException.cs

@@ -0,0 +1,26 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Runtime.Serialization;
+using System.Text;
+
+namespace FubuMVC.Less {
+	[Serializable]
+	public class ParserException : Exception {
+		//
+		// For guidelines regarding the creation of new exception types, see
+		//    http://msdn.microsoft.com/library/default.asp?url=/library/en-us/cpgenref/html/cpconerrorraisinghandlingguidelines.asp
+		// and
+		//    http://msdn.microsoft.com/library/default.asp?url=/library/en-us/dncscol/html/csharp07192001.asp
+		//
+
+		public ParserException() {}
+		public ParserException(string message) : base(message) {}
+		public ParserException(string message, Exception inner) : base(message, inner) {}
+
+		protected ParserException(
+			SerializationInfo info,
+			StreamingContext context) : base(info, context) {}
+	}
+
+}