Add book-store exercise (#308)

Add book-store exercise test generator

Author: ErikSchierboom

exercises/book-store/BookStore.cs

@@ -3,7 +3,7 @@
 
 public static class BookStore
 {
-    public static double CalculateTotalCost(List<int> books)
+    public static double Total(IEnumerable<int> books)
     {
         throw new NotImplementedException("You need to implement this function.");
     }

exercises/book-store/BookStoreTest.cs

@@ -1,83 +1,97 @@
-using System.Collections.Generic;
-using System.Linq;
+// This file was auto-generated based on version 1.0.1 of the canonical data.
+
 using Xunit;
 
 public class BookStoreTest
 {
     [Fact]
-    public void Basket_with_single_book()
+    public void Only_a_single_book()
     {
-        Assert.Equal(8, BookStore.CalculateTotalCost(MakeList(1)));
+        var input = new[] { 1 };
+        Assert.Equal(8, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
-    public void Basket_with_two_of_same_book()
+    public void Two_of_the_same_book()
     {
-        Assert.Equal(16, BookStore.CalculateTotalCost(MakeList(2, 2)));
+        var input = new[] { 2, 2 };
+        Assert.Equal(16, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
     public void Empty_basket()
     {
-        Assert.Equal(0, BookStore.CalculateTotalCost(MakeList()));
+        var input = new int[0];
+        Assert.Equal(0, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
-    public void Basket_with_two_different_books()
+    public void Two_different_books()
     {
-        Assert.Equal(15.2, BookStore.CalculateTotalCost(MakeList(1, 2)));
+        var input = new[] { 1, 2 };
+        Assert.Equal(15.2, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
-    public void Basket_with_three_different_books()
+    public void Three_different_books()
     {
-        Assert.Equal(21.6, BookStore.CalculateTotalCost(MakeList(1, 2, 3)));
+        var input = new[] { 1, 2, 3 };
+        Assert.Equal(21.6, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
-    public void Basket_with_four_different_books()
+    public void Four_different_books()
     {
-        Assert.Equal(25.6, BookStore.CalculateTotalCost(MakeList(1, 2, 3, 4)));
+        var input = new[] { 1, 2, 3, 4 };
+        Assert.Equal(25.6, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
-    public void Basket_with_five_different_books()
+    public void Five_different_books()
     {
-        Assert.Equal(30, BookStore.CalculateTotalCost(MakeList(1, 2, 3, 4, 5)));
+        var input = new[] { 1, 2, 3, 4, 5 };
+        Assert.Equal(30, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
-    public void Basket_with_eight_books()
+    public void Two_groups_of_four_is_cheaper_than_group_of_five_plus_group_of_three()
     {
-        Assert.Equal(51.20, BookStore.CalculateTotalCost(MakeList(1, 1, 2, 2, 3, 3, 4, 5)));
+        var input = new[] { 1, 1, 2, 2, 3, 3, 4, 5 };
+        Assert.Equal(51.2, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
-    public void Basket_with_nine_books()
+    public void Group_of_four_plus_group_of_two_is_cheaper_than_two_groups_of_three()
     {
-        Assert.Equal(55.60, BookStore.CalculateTotalCost(MakeList(1, 1, 2, 2, 3, 3, 4, 4, 5)));
+        var input = new[] { 1, 1, 2, 2, 3, 4 };
+        Assert.Equal(40.8, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
-    public void Basket_with_ten_books()
+    public void Two_each_of_first_4_books_and_1_copy_each_of_rest()
     {
-        Assert.Equal(60, BookStore.CalculateTotalCost(MakeList(1, 1, 2, 2, 3, 3, 4, 4, 5, 5)));
+        var input = new[] { 1, 1, 2, 2, 3, 3, 4, 4, 5 };
+        Assert.Equal(55.6, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
-    public void Basket_with_eleven_books()
+    public void Two_copies_of_each_book()
     {
-        Assert.Equal(68, BookStore.CalculateTotalCost(MakeList(1, 1, 2, 2, 3, 3, 4, 4, 5, 5, 1)));
+        var input = new[] { 1, 1, 2, 2, 3, 3, 4, 4, 5, 5 };
+        Assert.Equal(60, BookStore.Total(input));
     }
 
     [Fact(Skip = "Remove to run test")]
-    public void Basket_with_twelve_books()
+    public void Three_copies_of_first_book_and_2_each_of_remaining()
     {
-        Assert.Equal(75.20, BookStore.CalculateTotalCost(MakeList(1, 1, 2, 2, 3, 3, 4, 4, 5, 5, 1, 2)));
+        var input = new[] { 1, 1, 2, 2, 3, 3, 4, 4, 5, 5, 1 };
+        Assert.Equal(68, BookStore.Total(input));
     }
 
-    private static List<int> MakeList(params int[] values)
+    [Fact(Skip = "Remove to run test")]
+    public void Three_each_of_first_2_books_and_2_each_of_remaining_books()
     {
-        return values.ToList();
+        var input = new[] { 1, 1, 2, 2, 3, 3, 4, 4, 5, 5, 1, 2 };
+        Assert.Equal(75.2, BookStore.Total(input));
     }
-}
+}

exercises/book-store/Example.cs

@@ -4,14 +4,14 @@
 
 public static class BookStore
 {
-    public static double CalculateTotalCost(List<int> books)
+    public static double Total(IEnumerable<int> books)
     {
-        return CalculateTotalCost(books, 0);
+        return Total(books, 0);
     }
 
-    private static double CalculateTotalCost(List<int> books, double priceSoFar)
+    private static double Total(IEnumerable<int> books, double priceSoFar)
     {
-        if (books.Count == 0)
+        if (!books.Any())
         {
             return priceSoFar;
         }
@@ -33,7 +33,7 @@ private static double CalculateTotalCost(List<int> books, double priceSoFar)
                 remaining.Remove(item);
             }
 
-            var price = CalculateTotalCost(remaining.ToList(), priceSoFar + CostPerGroup(i));
+            var price = Total(remaining.ToList(), priceSoFar + CostPerGroup(i));
             minPrice = Math.Min(minPrice, price);
         }
 

exercises/transpose/TransposeTest.cs

@@ -15,8 +15,7 @@ public void Empty_string()
     [Fact(Skip = "Remove to run test")]
     public void Two_characters_in_a_row()
     {
-        var input = 
-            "A1";
+        var input = "A1";
         var expected = 
             "A\n" +
             "1";
@@ -29,8 +28,7 @@ public void Two_characters_in_a_column()
         var input = 
             "A\n" +
             "1";
-        var expected = 
-            "A1";
+        var expected = "A1";
         Assert.Equal(expected, Transpose.String(input));
     }
 
@@ -50,8 +48,7 @@ public void Simple()
     [Fact(Skip = "Remove to run test")]
     public void Single_line()
     {
-        var input = 
-            "Single line.";
+        var input = "Single line.";
         var expected = 
             "S\n" +
             "i\n" +

generators/Exercise.cs

@@ -9,12 +9,10 @@ protected Exercise()
         {
             Name = GetType().ToExerciseName();
             CanonicalData = CanonicalDataParser.Parse(Name);
-            Configuration = new ExerciseConfiguration();
         }
 
         public string Name { get; }
         public CanonicalData CanonicalData { get; }
-        public ExerciseConfiguration Configuration { get; }
 
         public void Generate() => TestClassFile.Write(this, Render());
 

generators/ExerciseConfiguration.cs

@@ -1,13 +1,11 @@
-using Generators.Output;
-
-namespace Generators.Exercises
+namespace Generators.Exercises
 {
     public class BeerSong : Exercise
     {
         public BeerSong()
         {
             foreach (var canonicalDataCase in CanonicalData.Cases)
-                canonicalDataCase.Expected = new MultiLineString(canonicalDataCase.Expected.ToString());
+                canonicalDataCase.UseExpectedParameter = true;
         }
     }
 }

generators/Exercises/BeerSong.cs

@@ -0,0 +1,16 @@
+using Newtonsoft.Json.Linq;
+
+namespace Generators.Exercises
+{
+    public class BookStore : Exercise
+    {
+        public BookStore()
+        {
+            foreach (var canonicalDataCase in CanonicalData.Cases)
+            {
+                canonicalDataCase.Input = ((JArray)canonicalDataCase.Properties["basket"]).Values<int>();
+                canonicalDataCase.UseInputParameters = true;
+            }
+        }
+    }
+}

generators/Exercises/BookStore.cs

@@ -1,5 +1,4 @@
-using Generators.Output;
-using Newtonsoft.Json.Linq;
+using Newtonsoft.Json.Linq;
 
 namespace Generators.Exercises
 {
@@ -8,7 +7,10 @@ public class FoodChain : Exercise
         public FoodChain()
         {
             foreach (var canonicalDataCase in CanonicalData.Cases)
-                canonicalDataCase.Expected = new MultiLineString((JArray)canonicalDataCase.Expected);
+            {
+                canonicalDataCase.Expected = string.Join("\n", (JArray)canonicalDataCase.Expected);
+                canonicalDataCase.UseExpectedParameter = true;
+            }
         }
     }
 }

generators/Exercises/FoodChain.cs

@@ -6,8 +6,8 @@ public class NthPrime : Exercise
     {
         public NthPrime()
         {
-            Configuration.ExceptionType = typeof(ArgumentOutOfRangeException);
-            Configuration.ThrowExceptionWhenExpectedValueEquals = x => x is bool;
+            foreach (var canonicalDataCase in CanonicalData.Cases)
+                canonicalDataCase.ExceptionThrown = canonicalDataCase.Expected is bool ? typeof(ArgumentOutOfRangeException) : null;
         }
     }
 }