Fix code review remarks

Author: comandeo-mongo

.evergreen/config/common.yml.erb

@@ -502,14 +502,14 @@ functions:
           ${PREPARE_SHELL}
           SERVERLESS=1 SSL=ssl RVM_RUBY="${RVM_RUBY}" SINGLE_MONGOS="${SINGLE_MONGOS}" MULTI_ATLASPROXY_SERVERLESS_URI="${MULTI_ATLASPROXY_SERVERLESS_URI}" SINGLE_ATLASPROXY_SERVERLESS_URI="${SINGLE_ATLASPROXY_SERVERLESS_URI}" .evergreen/run-tests-serverless.sh
 
-  "stat kmip server":
+  "start kmip server":
     - command: shell.exec
       params:
         background: true
         shell: bash
         working_dir: "src"
         script: |
-          source .evergreen/csfle/activate_venv.sh
+          . .evergreen/csfle/activate_venv.sh
           python -u .evergreen/csfle/kms_http_server.py --ca_file .evergreen/x509gen/ca.pem --cert_file .evergreen/x509gen/server.pem --port 7999 &
           python -u .evergreen/csfle/kms_http_server.py --ca_file .evergreen/x509gen/ca.pem --cert_file .evergreen/x509gen/expired.pem --port 8000 &
           python -u .evergreen/csfle/kms_http_server.py --ca_file .evergreen/x509gen/ca.pem --cert_file .evergreen/x509gen/wrong-host.pem --port 8001 &
@@ -561,7 +561,7 @@ tasks:
   - name: "test-fle"
     commands:
       - func: "export FLE credentials"
-      - func: "stat kmip server"
+      - func: "start kmip server"
       - func: "run tests"
   - name: "test-aws-auth"
     commands:

.evergreen/run-tests.sh

@@ -143,8 +143,7 @@ if test -n "$FLE"; then
   echo "Waiting for mock KMS servers to start..."
    wait_for_kms_server() {
       for i in $(seq 60); do
-         # Exit code 7: "Failed to connect to host".
-         if curl -s "localhost:$1"; test $? -ne 7; then
+         if curl -s "localhost:$1"; test $? -eq 0; then
             return 0
          else
             sleep 1