Edit pull request feedback

Author: holdenout

exercises/kindergarten-garden/canonical-data.json

@@ -3,6 +3,7 @@
   "version": "1.0.0",
   "comments": [
     "Garden can be created with just a diagram or a diagram and students.",
+    "Students' gardens are assigned in alphabetical order.",
     "The default students are:",
     "  Alice, Bob, Charlie, David, Eve, Fred, Ginny, Harriet, Ileana, Joseph, Kincaid, and Larry."
   ],
@@ -11,40 +12,40 @@
       "description": "partial garden tests",
       "cases": [
         {
-          "description": "test Alice's garden",
+          "description": "test garden with single student",
           "property": "plants",
           "diagram": "RC\nGG",
           "student": "Alice",
           "expected": ["radishes", "clover", "grass", "grass"]
         },
         {
-          "description": "test different garden for Alice",
+          "description": "test different garden with single student",
           "property": "plants",
           "diagram": "VC\nRC",
           "student": "Alice",
           "expected": ["violets", "clover", "radishes", "clover"]
         },
         {
-          "description": "test Bob's garden",
+          "description": "test garden with two students",
           "property": "plants",
           "diagram": "VVCG\nVVRC",
           "student": "Bob",
           "expected": ["clover", "grass", "radishes", "clover"]
         },
         {
-          "description": "test Bob and Charlie's garden",
+          "description": "test multiple students for the same garden with three students",
           "cases": [
             {
-              "description": "test Bob's garden",
-              "diagram": "VVCCGG\nVVCCGG",
+              "description": "test second student's garden",
               "property": "plants",
+              "diagram": "VVCCGG\nVVCCGG",
               "student": "Bob",
               "expected": ["clover", "clover", "clover", "clover"]
             },
             {
-              "description": "test Charlie's garden",
-              "diagram": "VVCCGG\nVVCCGG",
+              "description": "test third student's garden",
               "property": "plants",
+              "diagram": "VVCCGG\nVVCCGG",
               "student": "Charlie",
               "expected": ["grass", "grass", "grass", "grass"]
             }
@@ -56,67 +57,67 @@
       "description": "full garden tests",
       "cases": [
         {
-          "description": "test Alice's garden",
+          "description": "test first student's garden",
           "property": "plants",
           "diagram": "VRCGVVRVCGGCCGVRGCVCGCGV\nVRCCCGCRRGVCGCRVVCVGCGCV",
           "student": "Alice",
           "expected": ["violets", "radishes", "violets", "radishes"]
         },
         {
-          "description": "test Bob's garden",
-          "diagram": "VRCGVVRVCGGCCGVRGCVCGCGV\nVRCCCGCRRGVCGCRVVCVGCGCV",
+          "description": "test second student's garden",
           "property": "plants",
+          "diagram": "VRCGVVRVCGGCCGVRGCVCGCGV\nVRCCCGCRRGVCGCRVVCVGCGCV",
           "student": "Bob",
           "expected": ["clover", "grass", "clover", "clover"]
         },
         {
-          "description": "test Kincaid's garden",
-          "diagram": "VRCGVVRVCGGCCGVRGCVCGCGV\nVRCCCGCRRGVCGCRVVCVGCGCV",
+          "description": "test second to last student's garden",
           "property": "plants",
+          "diagram": "VRCGVVRVCGGCCGVRGCVCGCGV\nVRCCCGCRRGVCGCRVVCVGCGCV",
           "student": "Kincaid",
           "expected": ["grass", "clover", "clover", "grass"]
         },
         {
-          "description": "test Larry's garden",
-          "diagram": "VRCGVVRVCGGCCGVRGCVCGCGV\nVRCCCGCRRGVCGCRVVCVGCGCV",
+          "description": "test last student's garden",
           "property": "plants",
+          "diagram": "VRCGVVRVCGGCCGVRGCVCGCGV\nVRCCCGCRRGVCGCRVVCVGCGCV",
           "student": "Larry",
           "expected": ["grass", "violets", "clover", "violets"]
         }
       ]
     },
     {
-      "description": "disordered tests",
+      "description": "test with non-alphabetical student list",
       "cases": [
         {
-          "description": "test Patricia's garden",
+          "description": "test first student's garden",
           "property": "plants",
           "students": ["Samantha", "Patricia", "Xander", "Roger"],
-          "diagram": ["VCRRGVRG\nRVGCCGCV"],
+          "diagram": "VCRRGVRG\nRVGCCGCV",
           "student": "Patricia",
           "expected": ["violets", "clover", "radishes", "violets"]
         },
         {
-          "description": "test Roger's garden",
+          "description": "test second student's garden",
           "property": "plants",
           "students": ["Samantha", "Patricia", "Xander", "Roger"],
-          "diagram": ["VCRRGVRG\nRVGCCGCV"],
+          "diagram": "VCRRGVRG\nRVGCCGCV",
           "student": "Roger",
           "expected": ["radishes", "radishes", "grass", "clover"]
         },
         {
-          "description": "test Samantha's garden",
+          "description": "test third student's garden",
           "property": "plants",
           "students": ["Samantha", "Patricia", "Xander", "Roger"],
-          "diagram": ["VCRRGVRG\nRVGCCGCV"],
+          "diagram": "VCRRGVRG\nRVGCCGCV",
           "student": "Samantha",
           "expected": ["grass", "violets", "clover", "grass"]
         },
         {
-          "description": "test Xander's garden",
+          "description": "test fourth (last) student's garden",
           "property": "plants",
           "students": ["Samantha", "Patricia", "Xander", "Roger"],
-          "diagram": ["VCRRGVRG\nRVGCCGCV"],
+          "diagram": "VCRRGVRG\nRVGCCGCV",
           "student": "Xander",
           "expected": ["radishes", "grass", "clover", "violets"]
         }