fix #601: Do not treat booleans as numbers in List::toString()

Author: adazem009

include/scratchcpp/list.h

@@ -139,7 +139,7 @@ class LIBSCRATCHCPP_EXPORT List : public Entity
                 strings.push_back(std::string());
                 value_toString(item, &strings.back());
 
-                if (value_isValidNumber(item) && !strings.back().empty()) {
+                if (value_isValidNumber(item) && !value_isBool(item) && !strings.back().empty()) {
                     double doubleNum = value_toDouble(item);
                     long num = value_toLong(item);
 

test/scratch_classes/list_test.cpp

@@ -278,6 +278,20 @@ TEST(ListTest, ToString)
     list.toString(s);
     ASSERT_EQ(s, "098");
     ASSERT_EQ(list.toString(), "098");
+
+    list.clear();
+    list.append("true");
+    list.append("false");
+    list.toString(s);
+    ASSERT_EQ(s, "true false");
+    ASSERT_EQ(list.toString(), "true false");
+
+    list.clear();
+    list.append(true);
+    list.append(false);
+    list.toString(s);
+    ASSERT_EQ(s, "true false");
+    ASSERT_EQ(list.toString(), "true false");
 }
 
 TEST(ListTest, Clone)