From 7239471e1f2aaba493b4ff535cddd32993ee91aa Mon Sep 17 00:00:00 2001
From: Edgar Mengel <menedgnick@gmail.com>
Date: Thu, 21 May 2020 12:43:32 +0200
Subject: [PATCH] Make ready Berlin Clock with Unit tests

---
 .gitignore                                    | 188 ++++++++++++++++++
 BDD/BerlinClockFeatureSteps.cs                |  41 ++--
 BerlinClock.csproj                            |   9 +-
 BerlinClock.sln                               |  13 +-
 .../BerlinClockNUnitTest.csproj               |  19 ++
 .../BerlinClockTimeConverterTests.cs          | 174 ++++++++++++++++
 .../BerlinClockCharViewFormatTests.cs         |  50 +++++
 .../Utils/StringHelperTests.cs                |  99 +++++++++
 Classes/BerlinClockTimeConverter.cs           | 121 +++++++++++
 .../ViewFormats/BerlinClockCharViewFormat.cs  |  63 ++++++
 Classes/ITimeConverter.cs                     |  12 --
 Classes/TimeConverter.cs                      |  15 --
 Classes/Utils/StringHelper.cs                 |  60 ++++++
 Interfaces/Clock/BerlinClockModel.cs          |  53 +++++
 .../ViewFormats/IBerlinClockViewFormat.cs     |  15 ++
 Interfaces/ITimeConverter.cs                  |   7 +
 Readme.md                                     |   1 +
 17 files changed, 888 insertions(+), 52 deletions(-)
 create mode 100644 .gitignore
 create mode 100644 BerlinClockNUnitTest/BerlinClockNUnitTest.csproj
 create mode 100644 BerlinClockNUnitTest/BerlinClockTimeConverterTests.cs
 create mode 100644 BerlinClockNUnitTest/Clock/ViewFormats/BerlinClockCharViewFormatTests.cs
 create mode 100644 BerlinClockNUnitTest/Utils/StringHelperTests.cs
 create mode 100644 Classes/BerlinClockTimeConverter.cs
 create mode 100644 Classes/Clock/ViewFormats/BerlinClockCharViewFormat.cs
 delete mode 100644 Classes/ITimeConverter.cs
 delete mode 100644 Classes/TimeConverter.cs
 create mode 100644 Classes/Utils/StringHelper.cs
 create mode 100644 Interfaces/Clock/BerlinClockModel.cs
 create mode 100644 Interfaces/Clock/ViewFormats/IBerlinClockViewFormat.cs
 create mode 100644 Interfaces/ITimeConverter.cs

diff --git a/.gitignore b/.gitignore
new file mode 100644
index 00000000..7966e092
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1,188 @@
+## Ignore Visual Studio and Rider temporary files, build results, and
+## files generated by popular Visual Studio add-ons.
+
+# User-specific files
+*.suo
+*.user
+*.sln.docstates
+launchSettings.json
+
+# Build results
+
+[Dd]ebug/
+[Rr]elease/
+x64/
+build/
+[Bb]in/
+[Oo]bj/
+.vs/
+tools
+!tools/packages.config
+
+# Enable "build/" folder in the NuGet Packages folder since NuGet packages use it for MSBuild targets
+!packages/*/build/
+
+# MSTest test Results
+[Tt]est[Rr]esult*/
+[Bb]uild[Ll]og.*
+
+*_i.c
+*_p.c
+*.ilk
+*.meta
+*.obj
+*.pch
+*.pdb
+*.pgc
+*.pgd
+*.rsp
+*.sbr
+*.tlb
+*.tli
+*.tlh
+*.tmp
+*.tmp_proj
+*.log
+*.vspscc
+*.vssscc
+.builds
+*.pidb
+*.log
+*.scc
+
+# Visual C++ cache files
+ipch/
+*.aps
+*.ncb
+*.opensdf
+*.sdf
+*.cachefile
+
+# Visual Studio profiler
+*.psess
+*.vsp
+*.vspx
+
+# Guidance Automation Toolkit
+*.gpState
+
+# ReSharper is a .NET coding add-in
+_ReSharper*/
+*.[Rr]e[Ss]harper
+
+# TeamCity is a build add-in
+_TeamCity*
+
+# DotCover is a Code Coverage Tool
+*.dotCover
+
+# NCrunch
+*.ncrunch*
+.*crunch*.local.xml
+
+# Installshield output folder
+[Ee]xpress/
+
+# DocProject is a documentation generator add-in
+DocProject/buildhelp/
+DocProject/Help/*.HxT
+DocProject/Help/*.HxC
+DocProject/Help/*.hhc
+DocProject/Help/*.hhk
+DocProject/Help/*.hhp
+DocProject/Help/Html2
+DocProject/Help/html
+
+# Click-Once directory
+publish/
+
+# Publish Web Output
+*.Publish.xml
+
+# NuGet Packages Directory
+packages/
+
+# Windows Azure Build Output
+csx
+*.build.csdef
+
+# Windows Store app package directory
+AppPackages/
+
+# Rider specific folder
+\.idea/
+
+# Others
+sql/
+*.Cache
+ClientBin/
+[Ss]tyle[Cc]op.*
+~$*
+*~
+*.dbmdl
+*.[Pp]ublish.xml
+*.pfx
+*.publishsettings
+
+# RIA/Silverlight projects
+Generated_Code/
+
+# Backup & report files from converting an old project file to a newer
+# Visual Studio version. Backup files are not needed, because we have git ;-)
+_UpgradeReport_Files/
+Backup*/
+UpgradeLog*.XML
+UpgradeLog*.htm
+
+# SQL Server files
+App_Data/*.mdf
+App_Data/*.ldf
+
+
+#LightSwitch generated files
+GeneratedArtifacts/
+_Pvt_Extensions/
+ModelManifest.xml
+
+# =========================
+# Windows detritus
+# =========================
+
+# Windows image file caches
+Thumbs.db
+ehthumbs.db
+
+# Folder config file
+Desktop.ini
+
+# Recycle Bin used on file shares
+$RECYCLE.BIN/
+
+# Mac desktop service store files
+.DS_Store
+
+# =========================
+# NUnit Specific
+# =========================
+
+.~
+*.userprefs
+*.StyleCop
+*.sdf
+GeneratedAssemblyInfo.cs
+StyleCop.Cache
+local.settings.include
+InternalTrace.txt
+TestResult.xml
+testCaseCollection.xml
+deploy
+lib
+test-results
+package
+images
+MockAssemblyResult.xml
+PortabilityAnalysis*.html
+project.lock.json
+*.project.lock.json
+.dotnet
+.config/dotnet-tools.json
diff --git a/BDD/BerlinClockFeatureSteps.cs b/BDD/BerlinClockFeatureSteps.cs
index 4390f3cb..2b852eba 100644
--- a/BDD/BerlinClockFeatureSteps.cs
+++ b/BDD/BerlinClockFeatureSteps.cs
@@ -1,28 +1,27 @@
-using System;
-using TechTalk.SpecFlow;
+using BerlinClock.Interfaces;
 using Microsoft.VisualStudio.TestTools.UnitTesting;
-using System.Linq;
+using TechTalk.SpecFlow;
 
 namespace BerlinClock
 {
-    [Binding]
-    public class TheBerlinClockSteps
-    {
-        private ITimeConverter berlinClock = new TimeConverter();
-        private String theTime;
+	[Binding]
+	public class TheBerlinClockSteps
+	{
+		private ITimeConverter berlinClock = new BerlinClockTimeConverter();
+		private string theTime;
+
+
+		[When(@"the time is ""(.*)""")]
+		public void WhenTheTimeIs(string time)
+		{
+			this.theTime = time;
+		}
 
-        
-        [When(@"the time is ""(.*)""")]
-        public void WhenTheTimeIs(string time)
-        {
-            theTime = time;
-        }
-        
-        [Then(@"the clock should look like")]
-        public void ThenTheClockShouldLookLike(string theExpectedBerlinClockOutput)
-        {
-            Assert.AreEqual(berlinClock.convertTime(theTime), theExpectedBerlinClockOutput);
-        }
+		[Then(@"the clock should look like")]
+		public void ThenTheClockShouldLookLike(string theExpectedBerlinClockOutput)
+		{
+			Assert.AreEqual(this.berlinClock.ConvertTime(this.theTime), theExpectedBerlinClockOutput);
+		}
 
-    }
+	}
 }
diff --git a/BerlinClock.csproj b/BerlinClock.csproj
index ac8af99d..ad503c7f 100644
--- a/BerlinClock.csproj
+++ b/BerlinClock.csproj
@@ -50,14 +50,18 @@
   </ItemGroup>
   <ItemGroup>
     <Compile Include="BDD\BerlinClockFeatureSteps.cs" />
+    <Compile Include="Classes\Utils\StringHelper.cs" />
+    <Compile Include="Interfaces\Clock\BerlinClockModel.cs" />
+    <Compile Include="Classes\Clock\ViewFormats\BerlinClockCharViewFormat.cs" />
+    <Compile Include="Interfaces\Clock\ViewFormats\IBerlinClockViewFormat.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="BDD\BerlinClockFeatureSteps.feature.cs">
       <AutoGen>True</AutoGen>
       <DesignTime>True</DesignTime>
       <DependentUpon>BerlinClockFeatureSteps.feature</DependentUpon>
     </Compile>
-    <Compile Include="Classes\ITimeConverter.cs" />
-    <Compile Include="Classes\TimeConverter.cs" />
+    <Compile Include="Interfaces\ITimeConverter.cs" />
+    <Compile Include="Classes\BerlinClockTimeConverter.cs" />
   </ItemGroup>
   <ItemGroup>
     <None Include="App.config" />
@@ -70,6 +74,7 @@
   <ItemGroup>
     <Service Include="{82A7F48D-3B50-4B1E-B82E-3ADA8210C358}" />
   </ItemGroup>
+  <ItemGroup />
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
        Other similar extension points exist, see Microsoft.Common.targets.
diff --git a/BerlinClock.sln b/BerlinClock.sln
index 082911a2..2dafea40 100644
--- a/BerlinClock.sln
+++ b/BerlinClock.sln
@@ -1,10 +1,12 @@
 
 Microsoft Visual Studio Solution File, Format Version 12.00
-# Visual Studio 2013
-VisualStudioVersion = 12.0.21005.1
+# Visual Studio Version 16
+VisualStudioVersion = 16.0.29806.167
 MinimumVisualStudioVersion = 10.0.40219.1
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "BerlinClock", "BerlinClock.csproj", "{0451204D-BF86-43E1-B560-FC9AC830B9A9}"
 EndProject
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "BerlinClockNUnitTest", "BerlinClockNUnitTest\BerlinClockNUnitTest.csproj", "{69310066-B9EC-4A1F-85E2-E1F6D8F015ED}"
+EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
 		Debug|Any CPU = Debug|Any CPU
@@ -15,8 +17,15 @@ Global
 		{0451204D-BF86-43E1-B560-FC9AC830B9A9}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{0451204D-BF86-43E1-B560-FC9AC830B9A9}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{0451204D-BF86-43E1-B560-FC9AC830B9A9}.Release|Any CPU.Build.0 = Release|Any CPU
+		{69310066-B9EC-4A1F-85E2-E1F6D8F015ED}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{69310066-B9EC-4A1F-85E2-E1F6D8F015ED}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{69310066-B9EC-4A1F-85E2-E1F6D8F015ED}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{69310066-B9EC-4A1F-85E2-E1F6D8F015ED}.Release|Any CPU.Build.0 = Release|Any CPU
 	EndGlobalSection
 	GlobalSection(SolutionProperties) = preSolution
 		HideSolutionNode = FALSE
 	EndGlobalSection
+	GlobalSection(ExtensibilityGlobals) = postSolution
+		SolutionGuid = {E8033423-7F55-436F-96A2-24BE7E6C8B42}
+	EndGlobalSection
 EndGlobal
diff --git a/BerlinClockNUnitTest/BerlinClockNUnitTest.csproj b/BerlinClockNUnitTest/BerlinClockNUnitTest.csproj
new file mode 100644
index 00000000..592177c0
--- /dev/null
+++ b/BerlinClockNUnitTest/BerlinClockNUnitTest.csproj
@@ -0,0 +1,19 @@
+<Project Sdk="Microsoft.NET.Sdk">
+
+  <PropertyGroup>
+    <TargetFramework>netcoreapp3.1</TargetFramework>
+
+    <IsPackable>false</IsPackable>
+  </PropertyGroup>
+
+  <ItemGroup>
+    <PackageReference Include="nunit" Version="3.12.0" />
+    <PackageReference Include="NUnit3TestAdapter" Version="3.15.1" />
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.4.0" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <ProjectReference Include="..\BerlinClock.csproj" />
+  </ItemGroup>
+
+</Project>
diff --git a/BerlinClockNUnitTest/BerlinClockTimeConverterTests.cs b/BerlinClockNUnitTest/BerlinClockTimeConverterTests.cs
new file mode 100644
index 00000000..e493dfb6
--- /dev/null
+++ b/BerlinClockNUnitTest/BerlinClockTimeConverterTests.cs
@@ -0,0 +1,174 @@
+using BerlinClock;
+using BerlinClock.Classes.Clock.ViewFormats;
+using BerlinClock.Interfaces;
+using BerlinClock.Interfaces.Clock.ViewFormats;
+using NUnit.Framework;
+using System;
+using System.Collections;
+
+namespace BerlinClockNUnitTest
+{
+	[TestFixture]
+	public class BerlinClockTimeConverterTests
+	{
+		private BerlinClockTimeConverter _berlinClockTimeConverter;
+
+		[SetUp]
+		public void SetUp()
+		{
+			this._berlinClockTimeConverter = new BerlinClockTimeConverter();
+		}
+
+		[Test]
+		public void BerlinClockCharViewFormat_CreatedSuccessfully()
+		{
+			// Assert
+			Assert.IsNotNull(this._berlinClockTimeConverter, "Object should be created");
+		}
+
+		[Test]
+		public void BerlinClockCharViewFormat_IsITimeConverter()
+		{
+			// Assert
+			Assert.IsTrue(this._berlinClockTimeConverter is ITimeConverter, $"Object should be type of {typeof(ITimeConverter)}");
+		}
+
+		public static IEnumerable ConvertTimeStringCases
+		{
+			// Arrange
+			get
+			{
+				yield return new TestCaseData("00:00:00").Returns("Y\r\nOOOO\r\nOOOO\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData("24:00:00").Returns("Y\r\nRRRR\r\nRRRR\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData("00:00:01").Returns("O\r\nOOOO\r\nOOOO\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData("11:25:41").Returns("O\r\nRROO\r\nROOO\r\nYYRYYOOOOOO\r\nOOOO");
+				yield return new TestCaseData("17:39:55").Returns("O\r\nRRRO\r\nRROO\r\nYYRYYRYOOOO\r\nYYYY");
+				yield return new TestCaseData("23:59:59").Returns("O\r\nRRRR\r\nRRRO\r\nYYRYYRYYRYY\r\nYYYY");
+			}
+		}
+		[TestCaseSource("ConvertTimeStringCases")]
+		public string ConvertTimeString_GetCorrectResultCases(string timeSpan)
+		{
+			// Act
+			string result = this._berlinClockTimeConverter.ConvertTime(timeSpan);
+
+			// Assert
+			return result;
+		}
+
+		[Test]
+		public void ConvertTimeString_IncorrectCase_TimeSpanNotCorrect()
+		{
+			TestDelegate testDelegate = () =>
+			{
+				// Arrange
+				string timeSpan = "99:99:99f";
+
+				// Act
+				this._berlinClockTimeConverter.ConvertTime(timeSpan);
+			};
+
+			// Assert
+			Assert.Throws<ArgumentException>(testDelegate);
+		}
+
+		public static IEnumerable ConvertTimeStringAndFormatsCases
+		{
+			// Arrange
+			get
+			{
+				yield return new TestCaseData("00:00:00", new BerlinClockCharViewFormat()).Returns("Y\r\nOOOO\r\nOOOO\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData("24:00:00", new BerlinClockCharViewFormat()).Returns("Y\r\nRRRR\r\nRRRR\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData("00:00:01", new BerlinClockCharViewFormat()).Returns("O\r\nOOOO\r\nOOOO\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData("11:25:41", new BerlinClockCharViewFormat()).Returns("O\r\nRROO\r\nROOO\r\nYYRYYOOOOOO\r\nOOOO");
+				yield return new TestCaseData("17:39:55", new BerlinClockCharViewFormat()).Returns("O\r\nRRRO\r\nRROO\r\nYYRYYRYOOOO\r\nYYYY");
+				yield return new TestCaseData("23:59:59", new BerlinClockCharViewFormat()).Returns("O\r\nRRRR\r\nRRRO\r\nYYRYYRYYRYY\r\nYYYY");
+			}
+		}
+		[TestCaseSource("ConvertTimeStringAndFormatsCases")]
+		public string ConvertTimeStringAndFormats_GetCorrectResultCases(
+			string timeSpan, IBerlinClockViewFormat berlinClockViewFormat)
+		{
+			// Act
+			string result = this._berlinClockTimeConverter.ConvertTime(timeSpan, berlinClockViewFormat);
+
+			// Assert
+			return result;
+		}
+
+		[Test]
+		public void ConvertTimeStringAndFormats_IncorrectCase_TimeSpanNotCorrect()
+		{
+			TestDelegate testDelegate = () =>
+			{
+				// Arrange
+				string timeSpan = "99:99:99f";
+				IBerlinClockViewFormat berlinClockViewFormat = new BerlinClockCharViewFormat();
+
+				// Act
+				this._berlinClockTimeConverter.ConvertTime(timeSpan, berlinClockViewFormat);
+			};
+
+			// Assert
+			Assert.Throws<ArgumentException>(testDelegate);
+		}
+
+		[Test]
+		public void ConvertTimeStringAndFormats_IncorrectCase_BerlinClockViewFormatNotCorrect()
+		{
+			TestDelegate testDelegate = () =>
+			{
+				// Arrange
+				string timeSpan = "00:12:34";
+				IBerlinClockViewFormat berlinClockViewFormat = null;
+
+				// Act
+				this._berlinClockTimeConverter.ConvertTime(timeSpan, berlinClockViewFormat);
+			};
+
+			// Assert
+			Assert.Throws<ArgumentNullException>(testDelegate);
+		}
+
+		public static IEnumerable ConvertTimeSpanAndFormatCases
+		{
+			// Arrange
+			get
+			{
+				yield return new TestCaseData(TimeSpan.Parse("00:00:00"), new BerlinClockCharViewFormat()).Returns("Y\r\nOOOO\r\nOOOO\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData(TimeSpan.Parse("24:00:00"), new BerlinClockCharViewFormat()).Returns("Y\r\nRRRR\r\nRRRR\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData(TimeSpan.Parse("00:00:01"), new BerlinClockCharViewFormat()).Returns("O\r\nOOOO\r\nOOOO\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData(TimeSpan.Parse("11:25:41"), new BerlinClockCharViewFormat()).Returns("O\r\nRROO\r\nROOO\r\nYYRYYOOOOOO\r\nOOOO");
+				yield return new TestCaseData(TimeSpan.Parse("17:39:55"), new BerlinClockCharViewFormat()).Returns("O\r\nRRRO\r\nRROO\r\nYYRYYRYOOOO\r\nYYYY");
+				yield return new TestCaseData(TimeSpan.Parse("23:59:59"), new BerlinClockCharViewFormat()).Returns("O\r\nRRRR\r\nRRRO\r\nYYRYYRYYRYY\r\nYYYY");
+			}
+		}
+		[TestCaseSource("ConvertTimeSpanAndFormatCases")]
+		public string ConvertTimeSpanAndFormat_GetCorrectResultCases(
+			TimeSpan timeSpan, IBerlinClockViewFormat berlinClockViewFormat)
+		{
+			// Act
+			string result = this._berlinClockTimeConverter.ConvertTime(timeSpan, berlinClockViewFormat);
+
+			// Assert
+			return result;
+		}
+
+		[Test]
+		public void ConvertTimeSpanAndFormat_IncorrectCase_BerlinClockViewFormatNotCorrect()
+		{
+			TestDelegate testDelegate = () =>
+			{
+				// Arrange
+				TimeSpan timeSpan = new TimeSpan();
+				IBerlinClockViewFormat berlinClockViewFormat = null;
+
+				// Act
+				this._berlinClockTimeConverter.ConvertTime(timeSpan, berlinClockViewFormat);
+			};
+
+			// Assert
+			Assert.Throws<ArgumentNullException>(testDelegate);
+		}
+	}
+}
diff --git a/BerlinClockNUnitTest/Clock/ViewFormats/BerlinClockCharViewFormatTests.cs b/BerlinClockNUnitTest/Clock/ViewFormats/BerlinClockCharViewFormatTests.cs
new file mode 100644
index 00000000..eecd153d
--- /dev/null
+++ b/BerlinClockNUnitTest/Clock/ViewFormats/BerlinClockCharViewFormatTests.cs
@@ -0,0 +1,50 @@
+using BerlinClock.Classes.Clock.ViewFormats;
+using BerlinClock.Interfaces.Clock;
+using NUnit.Framework;
+using System.Collections;
+
+namespace BerlinClockNUnitTest.Clock.ViewFormats
+{
+	[TestFixture]
+	public class BerlinClockCharViewFormatTests
+	{
+		private BerlinClockCharViewFormat _berlinClockCharViewFormat;
+
+		[SetUp]
+		public void SetUp()
+		{
+			this._berlinClockCharViewFormat = new BerlinClockCharViewFormat();
+		}
+
+		[Test]
+		public void BerlinClockCharViewFormat_CreatedSuccessfully()
+		{
+			// Assert
+			Assert.IsNotNull(this._berlinClockCharViewFormat, "Object should be created");
+		}
+
+		public static IEnumerable MakeViewCases
+		{
+			// Arrange
+			get
+			{
+				yield return new TestCaseData(new BerlinClockModel(true, 0, 0, 0, 0)).Returns("Y\r\nOOOO\r\nOOOO\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData(new BerlinClockModel(false, 0, 0, 0, 0)).Returns("O\r\nOOOO\r\nOOOO\r\nOOOOOOOOOOO\r\nOOOO");
+				yield return new TestCaseData(new BerlinClockModel(true, 0, 0, 0, 1)).Returns("Y\r\nOOOO\r\nOOOO\r\nOOOOOOOOOOO\r\nYOOO");
+				yield return new TestCaseData(new BerlinClockModel(true, 1, 2, 3, 4)).Returns("Y\r\nROOO\r\nRROO\r\nYYROOOOOOOO\r\nYYYY");
+				yield return new TestCaseData(new BerlinClockModel(false, 1, 2, 3, 4)).Returns("O\r\nROOO\r\nRROO\r\nYYROOOOOOOO\r\nYYYY");
+				yield return new TestCaseData(new BerlinClockModel(true, 4, 4, 11, 4)).Returns("Y\r\nRRRR\r\nRRRR\r\nYYRYYRYYRYY\r\nYYYY");
+				yield return new TestCaseData(new BerlinClockModel(false, 4, 4, 11, 4)).Returns("O\r\nRRRR\r\nRRRR\r\nYYRYYRYYRYY\r\nYYYY");
+			}
+		}
+		[TestCaseSource("MakeViewCases")]
+		public string MakeView_GetCorrectViewCases(BerlinClockModel model)
+		{
+			// Act
+			string result = this._berlinClockCharViewFormat.MakeView(model);
+
+			// Assert
+			return result;
+		}
+	}
+}
diff --git a/BerlinClockNUnitTest/Utils/StringHelperTests.cs b/BerlinClockNUnitTest/Utils/StringHelperTests.cs
new file mode 100644
index 00000000..dcf4438a
--- /dev/null
+++ b/BerlinClockNUnitTest/Utils/StringHelperTests.cs
@@ -0,0 +1,99 @@
+using BerlinClock.Classes.Utils;
+using NUnit.Framework;
+using System;
+using System.Collections;
+
+namespace BerlinClockNUnitTest.Utils
+{
+	[TestFixture]
+	public class StringHelperTests
+	{
+		public static IEnumerable GetFilledStringByCharactersCases
+		{
+			// Arrange
+			get
+			{
+				yield return new TestCaseData(10, 10, 'x', 'y').Returns("xxxxxxxxxx");
+				yield return new TestCaseData(10, 5, 'x', 'y').Returns("xxxxxyyyyy");
+				yield return new TestCaseData(1, 0, 'x', 'y').Returns("y");
+			}
+		}
+		[TestCaseSource("GetFilledStringByCharactersCases")]
+		public string GetFilledStringByCharacters_CorrectCases(
+			int limitChars, int countToFill, char fillingSymbol, char emptySymbol)
+		{
+			// Act
+			string result = StringHelper.GetFilledStringByCharacters(limitChars, countToFill, fillingSymbol, emptySymbol);
+
+			// Assert
+			return result;
+		}
+
+		[Test]
+		public void GetFilledStringByCharacters_IncorrectCase_LimitCharsSmallerThanCountOfFill()
+		{
+			TestDelegate testDelegate = () =>
+			{
+				// Arrange
+				int limitChars = 0;
+				int countToFill = 1;
+				char fillingSymbol = 'x';
+				char emptySymbol = 'y';
+
+				// Act
+				StringHelper.GetFilledStringByCharacters(limitChars, countToFill, fillingSymbol, emptySymbol);
+			};
+
+			// Assert
+			Assert.Throws<ArgumentOutOfRangeException>(testDelegate);
+		}
+
+		public static IEnumerable GetFilledStringByCharactersWithSeparatorsCases
+		{
+			// Arrange
+			get
+			{
+				yield return new TestCaseData(10, 10, 'x', 2, 's', 'y').Returns("xsxsxsxsxs");
+				yield return new TestCaseData(10, 5, 'x', 3, 's', 'y').Returns("xxsxxyyyyy");
+				yield return new TestCaseData(15, 12, 'x', 4, 's', 'y').Returns("xxxsxxxsxxxsyyy");
+				yield return new TestCaseData(1, 0, 'x', 1, 's', 'y').Returns("y");
+			}
+		}
+		[TestCaseSource("GetFilledStringByCharactersWithSeparatorsCases")]
+		public string GetFilledStringByCharactersWithSeparators_CorrectCases(
+			int limitChars, int countToFill, char fillingSymbol,
+			int numberOfSymbolsBeforeSeparator, char separatorSymbol, char emptySymbol)
+		{
+			// Act
+			string result = StringHelper.GetFilledStringByCharactersWithSeparators(
+				limitChars, countToFill, fillingSymbol,
+				numberOfSymbolsBeforeSeparator, separatorSymbol, emptySymbol);
+
+			// Assert
+			return result;
+		}
+
+		[Test]
+		public void GetFilledStringByCharactersWithSeparators_IncorrectCase_LimitCharsSmallerThanCountOfFill()
+		{
+			TestDelegate testDelegate = () =>
+			{
+				// Arrange
+				int limitChars = 0;
+				int countToFill = 1;
+				char fillingSymbol = 'x';
+				int numberOfSymbolsBeforeSeparator = 2;
+				char separatorSymbol = 's';
+				char emptySymbol = 'y';
+
+				// Act
+				StringHelper.GetFilledStringByCharactersWithSeparators(
+					limitChars, countToFill, fillingSymbol,
+					numberOfSymbolsBeforeSeparator, separatorSymbol, emptySymbol);
+			};
+
+			// Assert
+			Assert.Throws<ArgumentOutOfRangeException>(testDelegate);
+		}
+	}
+}
diff --git a/Classes/BerlinClockTimeConverter.cs b/Classes/BerlinClockTimeConverter.cs
new file mode 100644
index 00000000..88a53c06
--- /dev/null
+++ b/Classes/BerlinClockTimeConverter.cs
@@ -0,0 +1,121 @@
+using BerlinClock.Classes.Clock.ViewFormats;
+using BerlinClock.Interfaces;
+using BerlinClock.Interfaces.Clock;
+using BerlinClock.Interfaces.Clock.ViewFormats;
+using System;
+using System.Globalization;
+
+namespace BerlinClock
+{
+	/// <summary>
+	/// Time converter to Berlin Clock. See remarks to details
+	/// </summary>
+	/// <remarks>
+	/// About the clock you can find info in link below:
+	/// http://www.aquaphoenix.com/misc/settheoryclock/
+	/// 
+	/// Example of their work:
+	/// https://jayasurian123.github.io/berlin-uhr/
+	/// </remarks>
+	public class BerlinClockTimeConverter : ITimeConverter
+	{
+		/// <summary>
+		/// Convert input time to Berlin Clock
+		/// </summary>
+		/// <param name="timeSpan">Time span</param>
+		/// <returns>Converted time to <see cref="BerlinClockCharViewFormat"/> format</returns>
+		public string ConvertTime(string timeSpan)
+		{
+			IBerlinClockViewFormat berlinClockViewFormat = new BerlinClockCharViewFormat();
+
+			return this.ConvertTime(timeSpan, berlinClockViewFormat);
+		}
+
+		/// <summary>
+		/// Convert input time to Berlin Clock
+		/// </summary>
+		/// <param name="timeSpan">Time span</param>
+		/// <param name="berlinClockViewFormat">View of Berlin clock</param>
+		/// <param name="timeFormatProvider">Culture of time span convertation</param>
+		/// <returns>Formatted Berlin Clock as string result</returns>
+		public string ConvertTime(
+			string timeSpan,
+			IBerlinClockViewFormat berlinClockViewFormat,
+			IFormatProvider timeFormatProvider = null)
+		{
+			if (string.IsNullOrEmpty(timeSpan))
+			{
+				throw new ArgumentException($"Argument {timeSpan} can't be null or empty");
+			}
+			if (berlinClockViewFormat == null)
+			{
+				throw new ArgumentNullException($"Argument {berlinClockViewFormat} can't be null");
+			}
+
+			IFormatProvider usedFormatProvider = timeFormatProvider ?? CultureInfo.InvariantCulture;
+			TimeSpan convertedTimeSpan;
+			bool parsed = TimeSpan.TryParse(timeSpan, usedFormatProvider, out convertedTimeSpan);
+
+			if (!parsed)
+			{
+				throw new ArgumentException($"Argument {timeSpan} should be a time span and convertable to {typeof(TimeSpan)}");
+			}
+
+			return this.GenerateTimeToViewFormat(convertedTimeSpan, berlinClockViewFormat);
+		}
+
+		/// <summary>
+		/// Convert input time to Berlin Clock
+		/// </summary>
+		/// <param name="timeSpan">Time span</param>
+		/// <param name="berlinClockViewFormat">View of Berlin clock</param>
+		/// <returns>Formatted Berlin Clock as string result</returns>
+		public string ConvertTime(
+			TimeSpan timeSpan,
+			IBerlinClockViewFormat berlinClockViewFormat)
+		{
+			if (berlinClockViewFormat == null)
+			{
+				throw new ArgumentNullException($"Argument {berlinClockViewFormat} can't be null");
+			}
+
+			return this.GenerateTimeToViewFormat(timeSpan, berlinClockViewFormat);
+		}
+
+
+		private string GenerateTimeToViewFormat(
+			TimeSpan timeSpan,
+			IBerlinClockViewFormat berlinClockViewFormat)
+		{
+			int hour = this.CalculateHour(timeSpan);
+			int minutes = timeSpan.Minutes;
+			int seconds = timeSpan.Seconds;
+
+			BerlinClockModel berlinClockModel = new BerlinClockModel(
+				evenSeconds: seconds % 2 == 0,
+				countOf5Hours: hour / 5,
+				countOfModHours: hour % 5,
+				countOf5Minutes: minutes / 5,
+				countOfModMinutes: minutes % 5
+			);
+
+			return berlinClockViewFormat.MakeView(berlinClockModel);
+		}
+
+		private int CalculateHour(TimeSpan timeSpan)
+		{
+			// Check case when clock should show another time in midnight 24:00:00 than 00:00:00
+			// Original TimeSpan.Parse didn't abillity to convert correctly
+			// https://docs.microsoft.com/en-us/dotnet/api/system.timespan.parse?view=netcore-3.1#System_TimeSpan_Parse_System_String_
+			if (timeSpan.Days == 24
+				&& timeSpan.Hours == 0
+				&& timeSpan.Minutes == 0
+				&& timeSpan.Seconds == 0)
+			{
+				return 24;
+			}
+
+			return timeSpan.Hours;
+		}
+	}
+}
diff --git a/Classes/Clock/ViewFormats/BerlinClockCharViewFormat.cs b/Classes/Clock/ViewFormats/BerlinClockCharViewFormat.cs
new file mode 100644
index 00000000..b84022c3
--- /dev/null
+++ b/Classes/Clock/ViewFormats/BerlinClockCharViewFormat.cs
@@ -0,0 +1,63 @@
+using BerlinClock.Classes.Utils;
+using BerlinClock.Interfaces.Clock;
+using BerlinClock.Interfaces.Clock.ViewFormats;
+
+namespace BerlinClock.Classes.Clock.ViewFormats
+{
+	public class BerlinClockCharViewFormat : IBerlinClockViewFormat
+	{
+		private const char c_offSymbol = 'O';
+		private const char c_secondSymbol = 'Y';
+		private const char c_hourSymbol = 'R';
+		private const char c_minuteSymbol = 'Y';
+		private const char c_quarterSymbol = 'R';
+
+		private const int c_quaterSeparateNumber = 3;
+
+
+		/// <summary>
+		/// Creates view of <see cref="BerlinClockModel"/> as characters. See remarks for examples
+		/// </summary>
+		/// <param name="model">Berlin clock model</param>
+		/// <returns><see cref="BerlinClockModel"/> as characters</returns>
+		/// <remarks>
+		/// Some examples:
+		/// 1. For "23:59:59"
+		/// O
+		/// RRRR
+		/// RRRO
+		/// YYRYYRYYRYY
+		/// YYYY
+		/// 
+		/// 2. For "00:00:00"
+		/// Y
+		/// OOOO
+		/// OOOO
+		/// OOOOOOOOOOO
+		/// OOOO
+		/// </remarks>
+		public string MakeView(BerlinClockModel model)
+		{
+			char secondsSymbol = model.EvenSeconds
+				? c_secondSymbol
+				: c_offSymbol;
+			string secondsRow = secondsSymbol.ToString();
+
+			string filling5HoursRow = StringHelper.GetFilledStringByCharacters(
+				BerlinClockModel.c_limit5HoursChars, model.CountOf5Hours, c_hourSymbol, c_offSymbol);
+
+			string fillingModHoursRow = StringHelper.GetFilledStringByCharacters(
+				BerlinClockModel.c_limitModHoursChars, model.CountOfModHours, c_hourSymbol, c_offSymbol);
+
+			string filling5MinutesRow = StringHelper.GetFilledStringByCharactersWithSeparators(
+				BerlinClockModel.c_limit5MinutesChars, model.CountOf5Minutes, c_minuteSymbol, c_quaterSeparateNumber, c_quarterSymbol, c_offSymbol);
+
+			string fillingModMinutesRow = StringHelper.GetFilledStringByCharacters(
+				BerlinClockModel.c_limitModMinutesChars, model.CountOfModMinutes, c_minuteSymbol, c_offSymbol);
+
+			string resultView = $"{secondsRow}\r\n{filling5HoursRow}\r\n{fillingModHoursRow}\r\n{filling5MinutesRow}\r\n{fillingModMinutesRow}";
+
+			return resultView;
+		}
+	}
+}
diff --git a/Classes/ITimeConverter.cs b/Classes/ITimeConverter.cs
deleted file mode 100644
index 1d9e4c27..00000000
--- a/Classes/ITimeConverter.cs
+++ /dev/null
@@ -1,12 +0,0 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-
-namespace BerlinClock
-{
-    public interface ITimeConverter
-    {
-        String convertTime(String aTime);
-    }
-}
diff --git a/Classes/TimeConverter.cs b/Classes/TimeConverter.cs
deleted file mode 100644
index dd5bf4e0..00000000
--- a/Classes/TimeConverter.cs
+++ /dev/null
@@ -1,15 +0,0 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-
-namespace BerlinClock
-{
-    public class TimeConverter : ITimeConverter
-    {
-        public string convertTime(string aTime)
-        {
-            throw new NotImplementedException();
-        }
-    }
-}
diff --git a/Classes/Utils/StringHelper.cs b/Classes/Utils/StringHelper.cs
new file mode 100644
index 00000000..9f3f3f05
--- /dev/null
+++ b/Classes/Utils/StringHelper.cs
@@ -0,0 +1,60 @@
+using System;
+
+namespace BerlinClock.Classes.Utils
+{
+	public static class StringHelper
+	{
+		public static string GetFilledStringByCharacters(
+			int limitChars,
+			int countToFill,
+			char fillingSymbol,
+			char emptySymbol)
+		{
+			if (limitChars < countToFill)
+			{
+				throw new ArgumentOutOfRangeException(nameof(limitChars), $"{nameof(limitChars)} should be more than {nameof(countToFill)}");
+			}
+
+			char[] chars = new char[limitChars];
+			for (int i = 0; i < chars.Length; i++)
+			{
+				chars[i] = i < countToFill
+					? fillingSymbol
+					: emptySymbol;
+			}
+
+			return new string(chars);
+		}
+
+		public static string GetFilledStringByCharactersWithSeparators(
+			int limitChars,
+			int countToFill,
+			char fillingSymbol,
+			int numberOfSymbolsBeforeSeparator,
+			char separatorSymbol,
+			char emptySymbol)
+		{
+			if (limitChars < countToFill)
+			{
+				throw new ArgumentOutOfRangeException($"{nameof(limitChars)} should be more than {nameof(countToFill)}");
+			}
+
+			char[] chars = new char[limitChars];
+			for (int i = 0; i < chars.Length; i++)
+			{
+				if (i < countToFill)
+				{
+					chars[i] = (i + 1) % numberOfSymbolsBeforeSeparator == 0
+						? separatorSymbol
+						: fillingSymbol;
+				}
+				else
+				{
+					chars[i] = emptySymbol;
+				}
+			}
+
+			return new string(chars);
+		}
+	}
+}
diff --git a/Interfaces/Clock/BerlinClockModel.cs b/Interfaces/Clock/BerlinClockModel.cs
new file mode 100644
index 00000000..5670abf8
--- /dev/null
+++ b/Interfaces/Clock/BerlinClockModel.cs
@@ -0,0 +1,53 @@
+using System;
+
+namespace BerlinClock.Interfaces.Clock
+{
+	/// <summary>
+	/// Berlin clock container
+	/// </summary>
+	public class BerlinClockModel
+	{
+		public const int c_limit5HoursChars = 4;
+		public const int c_limitModHoursChars = 4;
+		public const int c_limit5MinutesChars = 11;
+		public const int c_limitModMinutesChars = 4;
+
+		public bool EvenSeconds { get; set; }
+		public int CountOf5Hours { get; set; }
+		public int CountOfModHours { get; set; }
+		public int CountOf5Minutes { get; set; }
+		public int CountOfModMinutes { get; set; }
+
+
+		public BerlinClockModel(
+			bool evenSeconds,
+			int countOf5Hours,
+			int countOfModHours,
+			int countOf5Minutes,
+			int countOfModMinutes)
+		{
+			if (0 > countOf5Hours || countOf5Hours > c_limit5HoursChars)
+			{
+				throw new ArgumentOutOfRangeException($"Argument {countOf5Hours} out of required range | limit: {c_limit5HoursChars}");
+			}
+			if (0 > countOfModHours || countOfModHours > c_limitModHoursChars)
+			{
+				throw new ArgumentOutOfRangeException($"Argument {countOfModHours} out of required range | limit: {c_limitModHoursChars}");
+			}
+			if (0 > countOf5Minutes || countOf5Minutes > c_limit5MinutesChars)
+			{
+				throw new ArgumentOutOfRangeException($"Argument {countOf5Minutes} out of required range | limit: {c_limit5MinutesChars}");
+			}
+			if (0 > countOfModMinutes || countOfModMinutes > c_limitModMinutesChars)
+			{
+				throw new ArgumentOutOfRangeException($"Argument {countOfModMinutes} out of required range | limit: {c_limitModMinutesChars}");
+			}
+
+			this.EvenSeconds = evenSeconds;
+			this.CountOf5Hours = countOf5Hours;
+			this.CountOfModHours = countOfModHours;
+			this.CountOf5Minutes = countOf5Minutes;
+			this.CountOfModMinutes = countOfModMinutes;
+		}
+	}
+}
diff --git a/Interfaces/Clock/ViewFormats/IBerlinClockViewFormat.cs b/Interfaces/Clock/ViewFormats/IBerlinClockViewFormat.cs
new file mode 100644
index 00000000..d36a035f
--- /dev/null
+++ b/Interfaces/Clock/ViewFormats/IBerlinClockViewFormat.cs
@@ -0,0 +1,15 @@
+namespace BerlinClock.Interfaces.Clock.ViewFormats
+{
+	/// <summary>
+	/// Contract to decorate outgoing <see cref="BerlinClockModel"/>
+	/// </summary>
+	public interface IBerlinClockViewFormat
+	{
+		/// <summary>
+		/// Creates view of <see cref="BerlinClockModel"/>
+		/// </summary>
+		/// <param name="berlinClockModel">Berlin clock model</param>
+		/// <returns>View as string result</returns>
+		string MakeView(BerlinClockModel berlinClockModel);
+	}
+}
\ No newline at end of file
diff --git a/Interfaces/ITimeConverter.cs b/Interfaces/ITimeConverter.cs
new file mode 100644
index 00000000..9ea96b85
--- /dev/null
+++ b/Interfaces/ITimeConverter.cs
@@ -0,0 +1,7 @@
+namespace BerlinClock.Interfaces
+{
+	public interface ITimeConverter
+	{
+		string ConvertTime(string timeSpan);
+	}
+}
diff --git a/Readme.md b/Readme.md
index 2fc6f6bb..b53fad48 100644
--- a/Readme.md
+++ b/Readme.md
@@ -1,4 +1,5 @@
 # The Berlin Clock
+# READY SOLUTION!
 
 The Berlin Uhr (Clock) is a rather strange way to show the time. On the top of the clock there is a yellow lamp that
 blinks on/off every two seconds. The time is calculated by adding rectangular lamps.