add pytest markers to linter tests

ros2 · Jun 1, 2018 · e93ce3d · e93ce3d
1 parent 690fab5
commit e93ce3d
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 0 deletions.
diff --git a/launch/package.xml b/launch/package.xml
@@ -9,6 +9,7 @@
 
   <test_depend>ament_flake8</test_depend>
   <test_depend>ament_pep257</test_depend>
+  <test_depend>python3-pytest</test_depend>
 
   <export>
     <build_type>ament_python</build_type>

diff --git a/launch/test/test_flake8.py b/launch/test/test_flake8.py
@@ -13,8 +13,11 @@
 # limitations under the License.
 
 from ament_flake8.main import main
+import pytest
 
 
+@pytest.mark.flake8
+@pytest.mark.linter
 def test_flake8():
     rc = main(argv=[])
     assert rc == 0, 'Found errors'
diff --git a/launch/test/test_pep257.py b/launch/test/test_pep257.py
@@ -13,8 +13,11 @@
 # limitations under the License.
 
 from ament_pep257.main import main
+import pytest
 
 
+@pytest.mark.linter
+@pytest.mark.pep257
 def test_pep257():
     rc = main(argv=[])
     assert rc == 0, 'Found code style errors / warnings'
diff --git a/launch_testing/package.xml b/launch_testing/package.xml
@@ -14,6 +14,7 @@
   <test_depend>ament_flake8</test_depend>
   <test_depend>ament_pep257</test_depend>
   <test_depend>launch</test_depend>
+  <test_depend>python3-pytest</test_depend>
 
   <export>
     <build_type>ament_python</build_type>

diff --git a/launch_testing/test/test_flake8.py b/launch_testing/test/test_flake8.py
@@ -13,8 +13,11 @@
 # limitations under the License.
 
 from ament_flake8.main import main
+import pytest
 
 
+@pytest.mark.flake8
+@pytest.mark.linter
 def test_flake8():
     rc = main(argv=[])
     assert rc == 0, 'Found errors'
diff --git a/launch_testing/test/test_pep257.py b/launch_testing/test/test_pep257.py
@@ -13,8 +13,11 @@
 # limitations under the License.
 
 from ament_pep257.main import main
+import pytest
 
 
+@pytest.mark.linter
+@pytest.mark.pep257
 def test_pep257():
     rc = main(argv=[])
     assert rc == 0, 'Found code style errors / warnings'