Better standardize CI testing

- Fix appveyor configuration to test using --no-default-features
- Change --no-default-features from a matrix option to a different
  command within the matrix for travis. This will reduce load by
  not having to spin up a whole new VM just for testing a different
  cargo command.
- Remove obsolete comment from appveyor.yml

Author: daboross

Diff for: .travis.yml

@@ -5,12 +5,11 @@ rust:
 - stable
 - beta
 - nightly
-env:
-- COMPILE_ARGS=("--no-default-features")
-- COMPILE_ARGS=()
 os:
 - linux
 - osx
 script:
-- cargo build --verbose "${COMPILE_ARGS[@]}"
-- cargo test --verbose "${COMPILE_ARGS[@]}" -- --skip auth
+- cargo build --verbose
+- cargo test --verbose -- --skip auth
+- cargo build --verbose --no-default-features
+- cargo test --verbose --no-default-features -- --skip auth

Diff for: appveyor.yml

@@ -29,11 +29,9 @@ install:
 
 test_script:
 - cargo build --verbose --target %TARGET%
-# TODO: this doesn't test authenticated access, only parsing and non-auth calls.
-# We're probably OK not testing logging in on windows since it is tested on linux,
-# and we do test at least making HTTPS calls to the server with this, but it might
-# still be nice for completeness.
 - cargo test --verbose --target %TARGET% -- --skip auth
+- cargo build --verbose --target %TARGET% --no-default-features
+- cargo test --verbose --target %TARGET% --no-default-features -- --skip auth
 
 cache:
 - target