Fix scenario ID computation

Author: npilon

planterbox/feature.py

@@ -75,8 +75,10 @@ def __init__(self, feature_path, scenarios_to_run=None, feature_text=None,
     def id(self):
         if self.scenarios_to_run:
             scenario_string = StringIO()
-            csv.writer(scenario_string).writerow(self.scenarios_to_run)
-            return self.feature_id() + ':' + scenario_string.getvalue()
+            csv.writer(
+                scenario_string, quoting=csv.QUOTE_NONNUMERIC
+            ).writerow(list(self.scenarios_to_run))
+            return self.feature_id() + ':' + scenario_string.getvalue().strip()
         else:
             return self.feature_id()
 

planterbox/tests/test_plugin.py

@@ -59,6 +59,12 @@ def test_single_feature_index_scenario(self):
             {('example.package', 'test.feature'): {0}},
         )
 
+    def test_single_feature_name_scenario(self):
+        self.assertEqual(
+            normalize_names(['example.package:test.feature:A Test Scenario']),
+            {('example.package', 'test.feature'): {'A Test Scenario'}},
+        )
+
     def test_single_feature_index_many_scenarios(self):
         self.assertEqual(
             normalize_names(['example.package:test.feature:3,0,5']),