From 305c95b69fec32cbde60d59e9a538be49c857a83 Mon Sep 17 00:00:00 2001
From: Todd Gamblin <tgamblin@llnl.gov>
Date: Mon, 23 Jul 2018 01:38:09 -0700
Subject: [PATCH] tests: move doc tests to the unit test stage

- Frequently, the documentation build will fail mysteriously in some
  spack command.

- The cause is some new bug introduced by the PR, but this is not
  apparent because the unit tests haven't run and the doc tests aren't
  targeted at code bugs.

- Users end up puzzled by doc failures when they're really code failures.

- Move the doc tests parallel with the code tests, so that we can more
  easily see bugs like this.
---
 .travis.yml | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index 538aad5133..73cf39adea 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -14,17 +14,12 @@ branches:
 jobs:
   fast_finish: true
   include:
-    - stage: 'flake8 + documentation'
+    - stage: 'style checks'
       python: '2.7'
       os: linux
       language: python
       env: TEST_SUITE=flake8
-    - stage: 'flake8 + documentation'
-      python: '3.6'
-      os: linux
-      language: python
-      env: TEST_SUITE=doc
-    - stage: 'unit tests'
+    - stage: 'unit tests + documentation'
       python: '2.6'
       os: linux
       language: python
@@ -49,6 +44,10 @@ jobs:
       os: linux
       language: python
       env: [ TEST_SUITE=unit, COVERAGE=true ]
+    - python: '3.6'
+      os: linux
+      language: python
+      env: TEST_SUITE=doc
     - stage: 'unit tests - osx'
       os: osx
       language: generic
@@ -96,8 +95,8 @@ jobs:
       env: [ TEST_SUITE=build, COVERAGE=true, 'SPEC=mpich' ]
 
 stages:
-  - 'flake8 + documentation'
-  - 'unit tests'
+  - 'style checks'
+  - 'unit tests + documentation'
   - 'build tests'
   - name: 'unit tests - osx'
     if: type IN (cron)
-- 
GitLab