Brian Curtin avatar Brian Curtin committed 80314fe

Change the double enable test since that's now allowable

Comments (0)

Files changed (1)

tests/test_options.py

 import sys
 import os
 import time
+import re
 
 from minidumper import (MiniDumpWithFullMemoryInfo,
                         MiniDumpWithThreadInfo) # Randomly chosen
         self.assertNotEqual(get_size(contents[0]), get_size(contents[1]))
 
     def test_double_enable(self):
-        # Make sure a double initialization raises RuntimeError
-        # There might be a better way that I'm not aware of, but we can
-        # at least make sure a RuntimeError comes through stderr.
+        # Double initialization should work fine.
+        # This used to not be the case, so it raised a RuntimeError...
         proc = subprocess.Popen([sys.executable, "-c",
                           "import minidumper, tester;"
                           "minidumper.enable(dir=r'{0}');"
                                 stderr=subprocess.PIPE)
         stderr = proc.stderr.read().decode(sys.getfilesystemencoding())
         self.addCleanup(proc.stderr.close)
-        self.assertIn("RuntimeError", stderr)
+        text = re.sub("^\[\d* refs\]\\n$", "", stderr)
+        self.assertEqual("", text)
 
 
 if __name__ == "__main__":
     unittest.main()
+
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.