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

@@ -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'
+    - stage: 'unit tests + documentation'
-      python: '3.6'
-      os: linux
-      language: python
-      env: TEST_SUITE=doc
-    - stage: 'unit tests'
       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'
+  - 'style checks'
-  - 'unit tests'
+  - 'unit tests + documentation'
   - 'build tests'
   - name: 'unit tests - osx'
     if: type IN (cron)