Remove empty step, minor polish

.github/actions/on_device_tests/action.yaml

@@ -112,7 +112,7 @@ runs:
           gsutil cp "${GCS_RESULTS_PATH}/${COBALT_XMLS_FILENAME}" . || true
 
           # Break if the files were downloaded.
-          if [[ -f "${COBALT_ERROR_LOG}" && -f "${COBALT_XMLS_FILENAME}" ]]; then
+          if [[ -f "${COBALT_XMLS_FILENAME}" && -f "${COBALT_ERROR_LOG}" && && -f "${COBALT_INFO_LOG}" ]]; then
             break
           fi
 
@@ -124,6 +124,7 @@ runs:
         # Prepare unit test results for DataDog upload.
         mkdir -p ${UNIT_TEST_RESULT_PATH}/${{ matrix.platform }}/${{ matrix.shard }}/
 
+        # Set tags for test differentiation.
         tags="platform:${{ matrix.platform }}"
         echo $tags > ${UNIT_TEST_RESULT_PATH}/${{ matrix.platform }}/TAGS
 
@@ -134,11 +135,6 @@ runs:
         echo "COBALT_ERROR_LOG=${COBALT_ERROR_LOG}" >> $GITHUB_ENV
         echo "COBALT_INFO_LOG=${COBALT_INFO_LOG}" >> $GITHUB_ENV
       shell: bash
-    - name: Populate TAGS for unit test report
-      if: always() && env.TEST_TYPE == 'unit_test'
-      shell: bash
-      run: |
-        # Set tags for test differentiation.
     - name: Archive Unit Test Logs
       uses: actions/upload-artifact@v3
       if: always() && env.TEST_TYPE == 'unit_test'