refs #1777: fix testing. first time testing always failed it means testing on travis.ci will never be succeeded.

tests/roots/test-theming/conf.py

@@ -1,5 +1,5 @@
 # -*- coding: utf-8 -*-
 
 html_theme = 'test-theme'
-master = 'index'
+master_doc = 'index'
 

tests/test_theming.py

@@ -13,7 +13,6 @@ import os
 import zipfile
 import sys
 import subprocess
-from functools import wraps
 import tempfile
 from path import path
 
@@ -23,34 +22,6 @@ from sphinx.util.osutil import cd
 from util import with_app, raises, TestApp, rootdir
 
 
-def with_theme_setup(root, *args, **kwds):
-    """
-    Run `setup.py build_sphinx` with args and kwargs,
-    pass it to the test and clean up properly.
-    """
-    def generator(func):
-        @wraps(func)
-        def deco(*args2, **kwargs2):
-            tempdir = path(tempfile.mkdtemp())
-            testrootdir = rootdir / 'roots' / ('test-' + root)
-            pkgdir = tempdir / root
-            testrootdir.copytree(pkgdir)
-            with cd(pkgdir):
-                command = [sys.executable, 'setup.py', 'install']
-                command.extend(args)
-                try:
-                    proc = subprocess.Popen(
-                        command,
-                        env=os.environ,
-                        stdout=subprocess.PIPE,
-                        stderr=subprocess.PIPE)
-                    func(pkgdir, proc)
-                finally:
-                    tempdir.rmtree(ignore_errors=True)
-        return deco
-    return generator
-
-
 @with_app(confoverrides={'html_theme': 'ziptheme',
                          'html_theme_options.testopt': 'foo'})
 def test_theme_api(app, status, warning):
@@ -115,14 +86,36 @@ def test_js_source(app, status, warning):
     assert 'Underscore.js {v}'.format(v=v) in underscore_src, msg
 
 
-@with_theme_setup('theming')
+def test_theme_plugin():
-def test_theme_plugin(pkgroot, proc):
+    tempdir = path(tempfile.mkdtemp())
-    out, err = proc.communicate()
+    testrootdir = rootdir / 'roots' / ('test-theming')
-    print(out)
+    pkgdir = tempdir / 'theming'
-    print(err)
+    testrootdir.copytree(pkgdir)
-    assert proc.returncode == 0, 'expect zero status for setup.py'
+    with cd(pkgdir):
-    app = TestApp(testroot='theming')
+        command = [sys.executable, 'setup.py', 'install']
-    try:
+        try:
-        assert 'test-theme' in Theme.themes
+            proc = subprocess.Popen(
-    finally:
+                command,
-        app.cleanup()
+                env=os.environ,
+                stdout=subprocess.PIPE,
+                stderr=subprocess.PIPE)
+            out, err = proc.communicate()
+            print(out)
+            print(err)
+            assert proc.returncode == 0, 'expect zero status for setup.py'
+        finally:
+            pass
+
+        command = ['sphinx-build', '.', '_build/html']
+        try:
+            proc = subprocess.Popen(
+                command,
+                env=os.environ,
+                stdout=subprocess.PIPE,
+                stderr=subprocess.PIPE)
+            out, err = proc.communicate()
+            print(out)
+            print(err)
+            assert proc.returncode == 0, 'expect zero status for setup.py'
+        finally:
+            tempdir.rmtree(ignore_errors=True)