[GitHub] [tvm] alanmacd commented on a diff in pull request #11362: [microTVM][ARM][Zephyr] Add CMSIS dependencies in Zephyr project build

[GitBox <gi...@apache.org>]

alanmacd commented on code in PR #11362:
URL: https://github.com/apache/tvm/pull/11362#discussion_r887235029


##########
tests/micro/zephyr/conftest.py:
##########
@@ -76,4 +76,21 @@ def temp_dir(board):
     if not os.path.exists(board_workspace.parent):
         os.makedirs(board_workspace.parent)
 
-    return tempdir(board_workspace)
+    keep_for_debug = tvm_debug if tvm_debug else None
+    test_temp_dir = tempdir(custom_path=board_workspace, keep_for_debug=keep_for_debug)
+    return test_temp_dir
+
+
+@pytest.fixture(autouse=True)
+def skip_by_board(request, board):
+    """Skip test if board is in the list."""
+    if request.node.get_closest_marker("skip_boards"):
+        if board in request.node.get_closest_marker("skip_boards").args[0]:
+            pytest.skip("skipped on this board: {}".format(board))
+
+
+def pytest_configure(config):
+    config.addinivalue_line(
+        "markers",
+        "skip_by_board(board): skip test for the given board",

Review Comment:
   @mehrdadh does this need to be _skip_boards_?
   `"skip_boards(board): skip test for the given board",`



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org