updated JsonDeserializer to handle Dictionarys<String,List<T>> by just calling BuildList method which fixed confusion around list of lists.

Also updated the sample data to more clearly show lists vs list of lists

Author: davidgrupp

RestSharp.Tests/JsonTests.cs

@@ -620,7 +620,7 @@ public void Can_Deserialize_Nullable_DateTimeOffset_With_Null()
 		[Fact]
 		public void Can_Deserialize_To_Dictionary_String_String()
 		{
-			var doc = CreateJsonStringDictionary();
+            var doc = CreateJsonStringDictionary();
 			var d = new JsonDeserializer();
 			var response = new RestResponse { Content = doc };
 			var bd = d.Deserialize<Dictionary<string,string>>(response);

RestSharp.Tests/SampleClasses/EmployeeTracker.cs

@@ -14,9 +14,9 @@ public class EmployeeTracker
         public Dictionary<String, List<String>> EmployeesMail { get; set; }
         /// <summary>
         /// Key:    Employee name.
-        /// Value:  Hours worked this week.
+        /// Value:  Hours worked this each week.
         /// </summary>
-        public Dictionary<String, List<Int32>> EmployeesTime { get; set; }
+        public Dictionary<String, List<List<Int32>>> EmployeesTime { get; set; }
 
         /// <summary>
         /// Key:    Employee name.

RestSharp.Tests/SampleData/jsondictionary.txt

@@ -1,21 +1,21 @@
 {
 	"EmployeesPay" : 
 	{
-		"John": [[{"Type":"BiWeekly","Amount":3000},{"Type":"Bonus","Amount":5000}]],
-		"David": [[{"Type":"Monthly","Amount":5000},{"Type":"Bonus","Amount":2500}]],
-		"Mary": [[{"Type":"BiWeekly","Amount":2000}]]
+		"John": [{"Type":"BiWeekly","Amount":3000},{"Type":"Bonus","Amount":5000}],
+		"David": [{"Type":"Monthly","Amount":5000},{"Type":"Bonus","Amount":2500}],
+		"Mary": [{"Type":"BiWeekly","Amount":2000}]
 	},	
 	"EmployeesMail" : 
 	{
-		"John": [["Welcome to Restsharp", "Meetings at 4pm", "Meeting Cancled"]],
-		"David": [["Project deadline is Monday", "Good work"]],
-		"Mary": [["Is there any documentation on Product A", "I'm leaving early today"]]
+		"John": ["Welcome to Restsharp", "Meetings at 4pm", "Meeting Cancled"],
+		"David": ["Project deadline is Monday", "Good work"],
+		"Mary": ["Is there any documentation on Product A", "I'm leaving early today"]
 	},
 
 	"EmployeesTime" : 
 	{
-		"John": [[8, 7, 8, 8, 8],[1,2,3]],
-		"David": [4, 12, 6, 4],
+		"John": [[8, 7, 8, 8, 8], [1, 2, 3]],
+		"David": [[4, 12, 6, 4],[4, 12, 6, 4]],
 		"Mary": [[]]
 	}
 }

RestSharp/Deserializers/JsonDeserializer.cs

@@ -101,20 +101,13 @@ private IDictionary BuildDictionary(Type type, object parent)
 			{
 				var key = child.Key;
 				object item = null;
-				if(valueType.GetGenericTypeDefinition() != typeof(List<>))
+                if (valueType.IsGenericType && valueType.GetGenericTypeDefinition() == typeof(List<>))
 				{
-					item = ConvertValue(valueType, child.Value);
+                    item = BuildList(valueType, child.Value);
 				}
 				else
 				{
-                    item = (IList)Activator.CreateInstance(valueType); 
-                    foreach (var jArray in (JsonArray)child.Value)
-					{
-                        foreach (var element in (IList)ConvertValue(valueType, jArray))
-                        {
-                            ((IList)item).Add(element);
-                        }
-					}
+                    item = ConvertValue(valueType, child.Value); 
 				}
 				dict.Add(key, item);
 			}