add IBuildLight and refactor Program to use appropriate type

Readify · Apr 18, 2013 · 8180549 · 8180549
1 parent 8ec7f7e
commit 8180549
Show file tree

Hide file tree

Showing 5 changed files with 97 additions and 61 deletions.
diff --git a/TeamFlash/BuildLight.cs b/TeamFlash/BuildLight.cs
@@ -0,0 +1,35 @@
+namespace TeamFlash
+{
+    class BuildLight : IBuildLight
+    {
+        private readonly Monitor monitor = new Monitor();
+
+        public void Success()
+        {
+            monitor.SetLed(DelcomBuildIndicator.REDLED, false, false);
+            monitor.SetLed(DelcomBuildIndicator.GREENLED, true, false);
+            monitor.SetLed(DelcomBuildIndicator.BLUELED, false, false);
+        }
+
+        public void Warning()
+        {
+            monitor.SetLed(DelcomBuildIndicator.REDLED, false, false);
+            monitor.SetLed(DelcomBuildIndicator.GREENLED, false, false);
+            monitor.SetLed(DelcomBuildIndicator.BLUELED, true, false);
+        }
+
+        public void Fail()
+        {
+            monitor.SetLed(DelcomBuildIndicator.REDLED, true, false);
+            monitor.SetLed(DelcomBuildIndicator.GREENLED, false, false);
+            monitor.SetLed(DelcomBuildIndicator.BLUELED, false, false);
+        }
+
+        public void Off()
+        {
+            monitor.SetLed(DelcomBuildIndicator.REDLED, false, false);
+            monitor.SetLed(DelcomBuildIndicator.GREENLED, false, false);
+            monitor.SetLed(DelcomBuildIndicator.BLUELED, false, false);
+        }
+    }
+}
diff --git a/TeamFlash/IBuildLight.cs b/TeamFlash/IBuildLight.cs
@@ -0,0 +1,15 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+
+namespace TeamFlash
+{
+    interface IBuildLight
+    {
+        void Success();
+        void Warning();
+        void Fail();
+        void Off();
+    }
+}
diff --git a/TeamFlash/Linux/BuildLight.cs b/TeamFlash/Linux/BuildLight.cs
@@ -0,0 +1,32 @@
+namespace TeamFlash.Linux
+{
+    class BuildLight : IBuildLight
+    {
+        static void SetLightColor(int color)
+        {
+            DelcomBuildIndicator.OpenDevice();
+            DelcomBuildIndicator.SetColor(color);
+            DelcomBuildIndicator.CloseDevice();
+        }
+
+        public void Success()
+        {
+            SetLightColor(DelcomBuildIndicator.Green);
+        }
+
+        public void Warning()
+        {
+            SetLightColor(DelcomBuildIndicator.Blue);
+        }
+
+        public void Fail()
+        {
+            SetLightColor(DelcomBuildIndicator.Red);
+        }
+
+        public void Off()
+        {
+            SetLightColor(DelcomBuildIndicator.Off);
+        }
+    }
+}
diff --git a/TeamFlash/Program.cs b/TeamFlash/Program.cs
@@ -26,8 +26,13 @@ static void Main()
                 ? new string[0]
                 : teamFlashConfig.BuildTypeIds.Split(new[] { ',' }, StringSplitOptions.RemoveEmptyEntries).ToArray();
 
-            var monitor = new Monitor();
-            TurnOffLights(monitor);
+#if __MonoCS__
+            var buildLight = new Linux.BuildLight();
+#else
+            var buildLight = new BuildLight();
+#endif
+
+            buildLight.Off();
 
             while (!Console.KeyAvailable)
             {
@@ -39,28 +44,27 @@ static void Main()
                 switch (lastBuildStatus)
                 {
                     case BuildStatus.Unavailable:
-                        TurnOffLights(monitor);
+                        buildLight.Off();
                         Console.WriteLine(DateTime.Now.ToShortTimeString() + " Server unavailable");
                         break;
                     case BuildStatus.Passed:
-                        TurnOnSuccessLight(monitor);
+                        buildLight.Success();
                         Console.WriteLine(DateTime.Now.ToShortTimeString() + " Passed");
                         break;
                     case BuildStatus.Investigating:
-                        TurnOnWarningLight(monitor);
+                        buildLight.Warning();
                         Console.WriteLine(DateTime.Now.ToShortTimeString() + " Investigating");
                         break;
                     case BuildStatus.Failed:
-                        TurnOnFailLight(monitor);
+                        buildLight.Fail();
                         Console.WriteLine(DateTime.Now.ToShortTimeString() + " Failed");
                         break;
                 }
 
                 Wait();
             }
 
-            TurnOffLights(monitor);
-
+            buildLight.Off();
         }
 
         static TeamFlashConfig LoadConfig()
@@ -133,59 +137,6 @@ static void Wait()
             }
         }
 
-#if __MonoCS__
-        static void SetLightColor(int color)
-        {
-            Linux.DelcomBuildIndicator.OpenDevice();
-            Linux.DelcomBuildIndicator.SetColor(color);
-            Linux.DelcomBuildIndicator.CloseDevice();
-        }
-#endif
-
-        static void TurnOnSuccessLight(Monitor monitor)
-        {
-#if __MonoCS__
-            SetLightColor(Linux.DelcomBuildIndicator.Green);
-#else
-            monitor.SetLed(DelcomBuildIndicator.REDLED, false, false);
-            monitor.SetLed(DelcomBuildIndicator.GREENLED, true, false);
-            monitor.SetLed(DelcomBuildIndicator.BLUELED, false, false);
-#endif
-        }
-
-        static void TurnOnWarningLight(Monitor monitor)
-        {
-#if __MonoCS__
-            SetLightColor(Linux.DelcomBuildIndicator.Blue);
-#else
-            monitor.SetLed(DelcomBuildIndicator.REDLED, false, false);
-            monitor.SetLed(DelcomBuildIndicator.GREENLED, false, false);
-            monitor.SetLed(DelcomBuildIndicator.BLUELED, true, false);
-#endif
-        }
-
-        static void TurnOnFailLight(Monitor monitor)
-        {
-#if __MonoCS__
-            SetLightColor(Linux.DelcomBuildIndicator.Red);
-#else
-            monitor.SetLed(DelcomBuildIndicator.REDLED, true, false);
-            monitor.SetLed(DelcomBuildIndicator.GREENLED, false, false);
-            monitor.SetLed(DelcomBuildIndicator.BLUELED, false, false);
-#endif
-        }
-
-        static void TurnOffLights(Monitor monitor)
-        {
-#if __MonoCS__
-            SetLightColor(Linux.DelcomBuildIndicator.Off);
-#else
-            monitor.SetLed(DelcomBuildIndicator.REDLED, false, false);
-            monitor.SetLed(DelcomBuildIndicator.GREENLED, false, false);
-            monitor.SetLed(DelcomBuildIndicator.BLUELED, false, false);
-#endif
-        }
-
         static BuildStatus RetrieveBuildStatus(
             string serverUrl, string username, string password,
             IEnumerable<string> buildTypeIds)

diff --git a/TeamFlash/TeamFlash.csproj b/TeamFlash/TeamFlash.csproj
@@ -53,8 +53,11 @@
     <Reference Include="System.Xml" />
   </ItemGroup>
   <ItemGroup>
+    <Compile Include="BuildLight.cs" />
     <Compile Include="BuildStatus.cs" />
     <Compile Include="DelcomBuildIndicator.cs" />
+    <Compile Include="IBuildLight.cs" />
+    <Compile Include="Linux\BuildLight.cs" />
     <Compile Include="Linux\DelcomBuildIndicator.cs" />
     <Compile Include="Monitor.cs" />
     <Compile Include="Program.cs" />