appveyor: Fix a switched condition for cargotest

[alexcrichton]

It was intended that EXCLUDE_CARGO doesn't run cargotest!

[alexcrichton]

@bors: r+ p=50

[bors]

📌 Commit 893e499 has been approved by alexcrichton

[bors]

⌛ Testing commit 893e499 with merge cdcca78...

[bors]

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing cdcca78 to master...

[kennytm]

Follow up on #48809 (comment).

The timings in this build are:

Job	Duration
check x86_64-pc-windows-msvc	2h 16m
check i686-pc-windows-msvc (1)	2h 29m
check i686-pc-windows-msvc (2)	1h 41m
check-aux EXCLUDE_CARGO	1h 33m
cargotest	2h 02m
tools	1h 29m
check i686-pc-windows-gnu (1)	2h 10m
check i686-pc-windows-gnu (2)	1h 39m
check x86_64-pc-windows-gnu	2h 17m
dist x86_64-pc-windows-msvc	1h 50m
dist i686-pc-windows-msvc	2h 01m
dist i686-pc-windows-gnu	2h 03m
dist x86_64-pc-windows-gnu	1h 56m
dist-alt	1h 53m

To further balance the time taken we could

• Exclude 24 minutes of tests in i686-pc-windows-msvc, move to the new job
• Exclude 15 minutes of tests in i686-pc-windows-gnu, move to the new job
• Exclude 22 minutes of tests in x86_64-pc-windows-msvc, move to check-aux
• Note sure what to do with x86_64-pc-windows-gnu.

Some other reshuffling options regarding the 64-bit jobs:

1.  

   • Combine check-aux and tools into a single job (note: affects Linux as well)
   • Move 8 minutes from x86_64-pc-windows-msvc to cargotest
   • Split x86_64-pc-windows-gnu

2.  

   • Move the src/tools/cargo test of cargotest into tools
   • Move the cargotest test of cargotest into check-aux (note: these two affect Linux as well)
   • Split either x86_64-pc-windows-msvc or x86_64-pc-windows-gnu

(We may need more accurate metrics for this bin packing problem 😄)

[alexcrichton]

Nice analysis @kennytm! I've opened up #48844 to track the rebalancing act here