From c335015ba60012d2cbe565dc7dc0da3339b4574b Mon Sep 17 00:00:00 2001
From: James Lu <james@overdrivenetworks.com>
Date: Sun, 23 Jun 2019 12:53:44 -0700
Subject: [PATCH] Update Travis-CI settings

- Skip FML tests - they seem to be very unreliable
- Exclude AQI, NuWeather when running tests on pull requests
- Move SYSDNS_HOST_COMMAND into 'script' to clean up env settings
---
 .travis.yml | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index f069d9c..f1312c3 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -19,20 +19,20 @@ install:
 # Some plugins use external APIs, and can cause tests to fail due to them being slow/down.
 # --no-network disables these, and we can still allow tests that have network access to fail.
 env:
-    - SYSDNS_HOST_COMMAND="/usr/bin/unbound-host" TEST_ARGS="--no-network"
-    - SYSDNS_HOST_COMMAND="/usr/bin/unbound-host" TEST_ARGS=""
+    - TEST_ARGS="--no-network"
+    - TEST_ARGS=""
 
 script:
-    - 'if [ "$TRAVIS_PULL_REQUEST" != "false" ]; then echo "Skipping tests that require secret API keys (Weather, LastFM)"; export SKIP_PLUGINS="--exclude=./LastFM"; fi'
-    # Don't test Weather, it hits my API limits too quickly
-    - supybot-test -c --plugins-dir=. $TEST_ARGS $SKIP_PLUGINS --exclude=./Weather
+    - 'if [ "$TRAVIS_PULL_REQUEST" != "false" ]; then echo "Skipping tests that require secret API keys"; export SKIP_PLUGINS=("--exclude=./LastFM" "--exclude=./NuWeather" "--exclude=./AQI"); fi'
+    - export SYSDNS_HOST_COMMAND="/usr/bin/unbound-host"
+    - supybot-test -c --plugins-dir=. $TEST_ARGS ${SKIP_PLUGINS[@]} --exclude=./Weather --exclude=./FML
 
 notifications:
     email: false
 
 matrix:
     allow_failures:
-        - env: SYSDNS_HOST_COMMAND="/usr/bin/unbound-host" TEST_ARGS=""
+        - env: TEST_ARGS=""
 
 branches:
   except: