Better replacement strategy for test id

Author: BeyondEvil

pytest_html/plugin.py

@@ -15,6 +15,7 @@
 import time
 import bisect
 import warnings
+import re
 
 try:
     from ansi2html import Ansi2HTMLConverter, style
@@ -168,7 +169,7 @@ def create_asset(
         ):
             # 255 is the common max filename length on various filesystems
             asset_file_name = "{}_{}_{}.{}".format(
-                self.test_id.replace(":", "_"),
+                re.sub(r"[^\w\.]", "_", self.test_id),
                 str(extra_index),
                 str(test_index),
                 file_extension,

tox.ini

@@ -7,12 +7,13 @@
 envlist = py{27,36,37,py,py3}{,-ansi2html}, flake8, black
 
 [testenv]
-commands = pytest -v -r a {posargs}
+setenv = PYTHONDONTWRITEBYTECODE=1
 deps =
     pytest-xdist
     pytest-rerunfailures
     pytest-mock
     py{27,36,py,py3}-ansi2html: ansi2html
+commands = pytest -v -r a {posargs}
 
 [testenv:flake8]
 skip_install = true