Run packaging integration tests by default (#231)

* Run integration tests on packaged SDK by default.

(Check for blank value because when running on merge, all input params are blank.)

* Log if skip integration tests is enabled.

* Reorder logs.

* Make logical test consistent.

* Make conditional single line.

Author: jonsimantov

.github/workflows/cpp-packaging.yml

@@ -11,13 +11,13 @@ on:
       preserveIntermediateArtifacts:
         description: 'preserve intermediate artifacts?'
         default: 0
+      skipIntegrationTests:
+        description: 'skip integration tests?'
+        default: 0
       downloadPublicVersion:
         description: 'public version # to test against'
       downloadPreviousRun:
         description: 'previous run # to test against'
-      runIntegrationTests:
-        description: 'run integration tests?'
-        default: 0
 
 env:
   # Packaging prerequisites
@@ -49,6 +49,10 @@ jobs:
             fi
           fi
 
+      - name: log if skipping integration tests
+        if: !(github.event.inputs.skipIntegrationTests == 0 || github.event.inputs.skipIntegrationTests == '')
+        run: echo "::warning ::Skipping integration tests."
+
       - name: log if preserving intermediate artifacts
         if: |
           github.event.inputs.preserveIntermediateArtifacts != 0 && github.event.inputs.preserveIntermediateArtifacts != '' &&
@@ -615,7 +619,7 @@ jobs:
 
   tests:
     needs: [merge_packages, download_sdk_package]
-    if: github.event.inputs.runIntegrationTests != 0 && !cancelled()
+    if: (github.event.inputs.skipIntegrationTests == 0 || github.event.inputs.skipIntegrationTests == '') && !cancelled()
     runs-on: ${{ matrix.os }}
     strategy:
       matrix: