convert test id from plan to string (via allure-framework#425)

sseliverstov · web-flow · commit 606b80e343d2 · 2019-09-08T20:34:27.000+03:00
diff --git a/allure-pytest/src/plugin.py b/allure-pytest/src/plugin.py
@@ -153,7 +153,7 @@ def select_by_testcase(items):
     if file_path:
         with open(file_path, 'r') as file:
             plan = json.load(file)
-            ids = set([item.get("id") for item in plan])
+            ids = set([str(item.get("id")) for item in plan])
     return filter(lambda item: ids & set(allure_label(item, LabelType.ID)) if ids else True, items)
 
 
diff --git a/allure-pytest/test/integration/allure_ee/select_by_testcase_id_test.py b/allure-pytest/test/integration/allure_ee/select_by_testcase_id_test.py
@@ -8,11 +8,11 @@
     ["ids", "expected_tests"],
     [
         (
-            [{"id": "1"}, {"id": "2"}],
+            [{"id": 1}, {"id": 2}],
             ["test_number_one", "test_number_two"]
         ),
         (
-            [{"id": "1"}, {"id": "3"}, {"id": "3+"}],
+            [{"id": 1}, {"id": 3}, {"id": 4}],
             ["test_number_one", "test_number_three"]
         ),
         (
@@ -34,7 +34,7 @@ def test_select_by_testcase_id_test(ids, expected_tests, allured_testdir):
     ...     pass
 
     >>> @allure.id("3")
-    ... @allure.id("3+")
+    ... @allure.id("4")
     ... def test_number_three():
     ...     pass
 

Original file line number	Diff line number	Diff line change
`@@ -8,11 +8,11 @@`
`8`	`8`	`["ids", "expected_tests"],`
`9`	`9`	`[`
`10`	`10`	`(`
`11`		`- [{"id": "1"}, {"id": "2"}],`
	`11`	`+ [{"id": 1}, {"id": 2}],`
`12`	`12`	`["test_number_one", "test_number_two"]`
`13`	`13`	`),`
`14`	`14`	`(`
`15`		`- [{"id": "1"}, {"id": "3"}, {"id": "3+"}],`
	`15`	`+ [{"id": 1}, {"id": 3}, {"id": 4}],`
`16`	`16`	`["test_number_one", "test_number_three"]`
`17`	`17`	`),`
`18`	`18`	`(`
`@@ -34,7 +34,7 @@ def test_select_by_testcase_id_test(ids, expected_tests, allured_testdir):`
`34`	`34`	`... pass`
`35`	`35`
`36`	`36`	`>>> @allure.id("3")`
`37`		`- ... @allure.id("3+")`
	`37`	`+ ... @allure.id("4")`
`38`	`38`	`... def test_number_three():`
`39`	`39`	`... pass`
`40`	`40`