🎨 Changed wrapping only empty with quotes to wrapping String types with quotes

Author: Guldem

lib/src/test_options/value_with_test_options.dart

@@ -8,7 +8,7 @@ class ValueWithTestOptions extends Iterable<dynamic> {
   final TestOptions testOptions;
 
   String get description =>
-      '[ ${value.map((e) => e.toString().trim().isEmpty ? '\'$e\'' : e.toString()).join(', ')} ]';
+      '[ ${value.map((e) => e is String ? '\'$e\'' : e.toString()).join(', ')} ]';
 
   @override
   Iterator<dynamic> get iterator => value.iterator;

test/test_options/value_with_test_options_test.dart

@@ -0,0 +1,33 @@
+import 'package:parameterized_test/src/test_options/test_options.dart';
+import 'package:parameterized_test/src/test_options/value_with_test_options.dart';
+import 'package:test/test.dart';
+
+void main() {
+  final TestOptions testOptions = TestOptions();
+
+  group('description tests', () {
+    test('ValueWithTestOptions with only String values are wrapped with \'quotes\'', () {
+      final result = ValueWithTestOptions(
+        ['a', 'b', '', '   '],
+        testOptions,
+      ).description;
+
+      expect(result, "[ 'a', 'b', '', '   ' ]");
+    });
+
+    test('ValueWithTestOptions with mixed types only String values are wrapped with \'quotes\'', () {
+      final result = ValueWithTestOptions(
+        ['a', 'b', '', '   ', 1, 1.5, true, _ClassWithToString()],
+        testOptions,
+      ).description;
+
+      expect(result, "[ 'a', 'b', '', '   ', 1, 1.5, true, class with toString ]");
+    });
+
+  });
+}
+
+class _ClassWithToString {
+  @override
+  String toString() => 'class with toString';
+}