I think I got saving down

It's not really all that hard, if you think about it

Author: tkellogg

Jump.Location.Specs/FileStoreProviderSpec.cs

@@ -6,25 +6,104 @@
 
 namespace Jump.Location.Specs
 {
-    public class FileStoreProviderSpec : IDisposable
+    public class FileStoreProviderSpec
     {
-        private readonly string path = Path.Combine(Path.GetTempPath(), Path.GetRandomFileName());
-
-        public void Dispose()
+        public class DescribeRevive : IDisposable
         {
-            try { File.Delete(path); }
-            catch { }
+            private readonly string path = Path.Combine(Path.GetTempPath(), Path.GetRandomFileName());
+
+            public void Dispose()
+            {
+                try
+                {
+                    File.Delete(path);
+                }
+                catch
+                {
+                }
+            }
+
+            [Fact]
+            public void It_can_Revive_a_single_record_of_correctly_formatted_text()
+            {
+                var lines = new[] {"FS::C:\\blah\t8.5"};
+                File.WriteAllLines(path, lines);
+                var provider = new FileStoreProvider(path);
+
+                var db = provider.Revive();
+                db.Records.Count().ShouldEqual(1);
+                db.Records.First().Weight.ShouldEqual(8.5M);
+            }
+
+            [Fact]
+            public void It_can_Revive_multiple_records_of_correctly_formatted_text()
+            {
+                var lines = new[]
+                    {
+                        "FS::C:\\blah\t8.5",
+                        "FS::C:\\blah\t8.5",
+                        "FS::C:\\blah\t8.5",
+                    };
+                File.WriteAllLines(path, lines);
+                var provider = new FileStoreProvider(path);
+
+                var db = provider.Revive();
+                db.Records.Count().ShouldEqual(3);
+            }
+
+            [Fact]
+            public void It_skips_blank_and_empty_lines()
+            {
+                var lines = new[]
+                    {
+                        "",
+                        "FS::C:\\blah\t8.5",
+                        "    ",
+                    };
+                File.WriteAllLines(path, lines);
+                var provider = new FileStoreProvider(path);
+
+                var db = provider.Revive();
+                db.Records.Count().ShouldEqual(1);
+            }
+
+            [Fact]
+            public void It_throws_InvalidOperationException_when_row_doesnt_have_2_columns()
+            {
+                var lines = new[] {"FS::C:\\blah    8.5"};
+                File.WriteAllLines(path, lines);
+                var provider = new FileStoreProvider(path);
+
+                Assert.Throws<InvalidOperationException>(() => provider.Revive());
+            }
         }
 
-        [Fact]
-        public void It_can_Revive_a_single_record_of_correctly_formatted_text()
+        public class DescribeSave : IDisposable
         {
-            var lines = new[] { "FS::C:\\blah\t8.5" };
-            File.WriteAllLines(path, lines);
-            var provider = new FileStoreProvider(path);
+            private readonly string path = Path.Combine(Path.GetTempPath(), Path.GetRandomFileName());
+
+            public void Dispose()
+            {
+                try
+                {
+                    File.Delete(path);
+                }
+                catch
+                {
+                }
+            }
+
+            [Fact]
+            public void It_can_save_a_single_record()
+            {
+                var db = new Database();
+                db.Add(new Record("FS::C:\\data", 42M));
+                var provider = new FileStoreProvider(path);
+                provider.Save(db);
 
-            var db = provider.Revive();
-            db.Records.Count().ShouldEqual(1);
+                var contents = File.ReadAllText(path);
+                contents.ShouldEqual("FS::C:\\data\t42\r\n");
+            }
         }
     }
 }

Jump.Location.Specs/Jump.Location.Specs.csproj

@@ -9,10 +9,11 @@
     <AppDesignerFolder>Properties</AppDesignerFolder>
     <RootNamespace>Jump.Location.Specs</RootNamespace>
     <AssemblyName>Jump.Location.Specs</AssemblyName>
-    <TargetFrameworkVersion>v4.5</TargetFrameworkVersion>
+    <TargetFrameworkVersion>v4.0</TargetFrameworkVersion>
     <FileAlignment>512</FileAlignment>
     <SolutionDir Condition="$(SolutionDir) == '' Or $(SolutionDir) == '*Undefined*'">..\</SolutionDir>
     <RestorePackages>true</RestorePackages>
+    <TargetFrameworkProfile />
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">
     <DebugSymbols>true</DebugSymbols>

Jump.Location/App.config

@@ -2,5 +2,5 @@
 <configuration>
     <startup> 
 
-    <supportedRuntime version="v2.0.50727"/></startup>
+    <supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.0"/></startup>
 </configuration>

Jump.Location/FileStoreProvider.cs

@@ -1,4 +1,5 @@
 using System;
+using System.Collections.Generic;
 using System.IO;
 using System.Linq;
 
@@ -21,14 +22,17 @@ public FileStoreProvider(string path)
 
         public void Save(IDatabase database)
         {
-            
+            var lines = database.Records.Select(record => 
+                string.Format("{0}\t{1}", record.FullName, record.Weight));
+            File.WriteAllLines(path, lines);
         }
 
         public IDatabase Revive()
         {
             var db = new Database();
             var allLines = File.ReadAllLines(path);
-            foreach (var columns in allLines.Select(line => line.Split('\t')))
+            var nonBlankLines = allLines.Where(line => !string.IsNullOrWhiteSpace(line));
+            foreach (var columns in nonBlankLines.Select(line => line.Split('\t')))
             {
                 if (columns == null || columns.Length != 2)
                     throw new InvalidOperationException("Row of file didn't have 2 columns separated by a tab");

Jump.Location/Jump.Location.csproj

@@ -9,7 +9,7 @@
     <AppDesignerFolder>Properties</AppDesignerFolder>
     <RootNamespace>Jump.Location</RootNamespace>
     <AssemblyName>Jump.Location</AssemblyName>
-    <TargetFrameworkVersion>v3.5</TargetFrameworkVersion>
+    <TargetFrameworkVersion>v4.0</TargetFrameworkVersion>
     <FileAlignment>512</FileAlignment>
     <TargetFrameworkProfile />
   </PropertyGroup>
@@ -22,6 +22,7 @@
     <DefineConstants>DEBUG;TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <Prefer32Bit>false</Prefer32Bit>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|AnyCPU' ">
     <PlatformTarget>AnyCPU</PlatformTarget>
@@ -31,6 +32,7 @@
     <DefineConstants>TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <Prefer32Bit>false</Prefer32Bit>
   </PropertyGroup>
   <PropertyGroup>
     <StartupObject />