added first tests

filipw · filipw · commit 2d0a88b02490 · 2015-03-28T23:06:56.000+01:00
diff --git a/src/ScriptCs.Contracts/ScriptPackSession.cs b/src/ScriptCs.Contracts/ScriptPackSession.cs
@@ -37,7 +37,7 @@ private void AddScriptContextNamespace()
             }
         }
 
-        public IEnumerable<IScriptPackContext> Contexts
+        public virtual IEnumerable<IScriptPackContext> Contexts
         {
             get { return _contexts; }
         }
diff --git a/src/ScriptCs.Core/ReplCommands/ScriptPacksCommand.cs b/src/ScriptCs.Core/ReplCommands/ScriptPacksCommand.cs
@@ -30,20 +30,22 @@ public string CommandName
         public object Execute(IRepl repl, object[] args)
         {
             var packContexts = repl.ScriptPackSession.Contexts;
+            var originalColor = _console.ForegroundColor;
+            _console.ForegroundColor = ConsoleColor.Yellow;
+
             if (packContexts.IsNullOrEmpty())
             {
                 _console.WriteLine("There are no script packs available in this REPL session");
+                _console.ForegroundColor = originalColor;
                 return null;
             }
 
             var importedNamespaces = repl.Namespaces.Union(repl.ScriptPackSession.Namespaces).ToArray();
-            var originalColor = _console.ForegroundColor;
 
             foreach (var packContext in packContexts)
             {
                 var contextType = packContext.GetType();
-
-                _console.ForegroundColor = ConsoleColor.Yellow;
+                
                 _console.WriteLine(contextType.ToString());
                 _console.ForegroundColor = originalColor;
 
diff --git a/test/ScriptCs.Core.Tests/ReplCommands/ScriptPacksCommandTests.cs b/test/ScriptCs.Core.Tests/ReplCommands/ScriptPacksCommandTests.cs
@@ -0,0 +1,48 @@
+﻿using System.Collections.Generic;
+using System.Linq;
+using Moq;
+using ScriptCs.Contracts;
+using ScriptCs.ReplCommands;
+using Xunit;
+
+namespace ScriptCs.Tests.ReplCommands
+{
+    public class ScriptPacksCommandTests
+    {
+        public class CommandNameProperty
+        {
+            [Fact]
+            public void ReturnsScriptPacks()
+            {
+                // act
+                var cmd = new ScriptPacksCommand(new Mock<IConsole>().Object);
+
+                // assert
+                Assert.Equal("scriptpacks", cmd.CommandName);
+            }
+        }
+
+        public class ExecuteMethod
+        {
+            [Fact]
+            public void ShouldExitIfThereAreNoScriptPacks()
+            {
+                // arrange
+                var console = new Mock<IConsole>();
+                var repl = new Mock<IRepl>();
+                var scriptPackSession = new Mock<ScriptPackSession>(Enumerable.Empty<IScriptPack>(), new string[0]);
+
+                scriptPackSession.Setup(x => x.Contexts).Returns((IEnumerable<IScriptPackContext>) null);
+                repl.Setup(x => x.ScriptPackSession).Returns(scriptPackSession.Object);
+
+                var cmd = new ScriptPacksCommand(console.Object);
+
+                // act
+                cmd.Execute(repl.Object, null);
+
+                // assert
+                console.Verify(x => x.WriteLine("There are no script packs available in this REPL session"));
+            }
+        }
+    }
+}
diff --git a/test/ScriptCs.Core.Tests/ScriptCs.Core.Tests.csproj b/test/ScriptCs.Core.Tests/ScriptCs.Core.Tests.csproj
@@ -56,6 +56,7 @@
     <Compile Include="FileSystemMigratorTests.cs" />
     <Compile Include="PackageReferenceTests.cs" />
     <Compile Include="ReplCommands\HelpCommandTests.cs" />
+    <Compile Include="ReplCommands\ScriptPacksCommandTests.cs" />
     <Compile Include="ScriptLibraryComposerTests.cs" />
     <Compile Include="ReferenceLineProcessorTests.cs" />
     <Compile Include="LoadLineProcessorTests.cs" />

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ private void AddScriptContextNamespace()`
`37`	`37`	`}`
`38`	`38`	`}`
`39`	`39`
`40`		`- public IEnumerable<IScriptPackContext> Contexts`
	`40`	`+ public virtual IEnumerable<IScriptPackContext> Contexts`
`41`	`41`	`{`
`42`	`42`	`get { return _contexts; }`
`43`	`43`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,20 +30,22 @@ public string CommandName`
`30`	`30`	`public object Execute(IRepl repl, object[] args)`
`31`	`31`	`{`
`32`	`32`	`var packContexts = repl.ScriptPackSession.Contexts;`
	`33`	`+ var originalColor = _console.ForegroundColor;`
	`34`	`+ _console.ForegroundColor = ConsoleColor.Yellow;`
	`35`	`+`
`33`	`36`	`if (packContexts.IsNullOrEmpty())`
`34`	`37`	`{`
`35`	`38`	`_console.WriteLine("There are no script packs available in this REPL session");`
	`39`	`+ _console.ForegroundColor = originalColor;`
`36`	`40`	`return null;`
`37`	`41`	`}`
`38`	`42`
`39`	`43`	`var importedNamespaces = repl.Namespaces.Union(repl.ScriptPackSession.Namespaces).ToArray();`
`40`		`- var originalColor = _console.ForegroundColor;`
`41`	`44`
`42`	`45`	`foreach (var packContext in packContexts)`
`43`	`46`	`{`
`44`	`47`	`var contextType = packContext.GetType();`
`45`		`-`
`46`		`- _console.ForegroundColor = ConsoleColor.Yellow;`
	`48`	`+`
`47`	`49`	`_console.WriteLine(contextType.ToString());`
`48`	`50`	`_console.ForegroundColor = originalColor;`
`49`	`51`