vp9_spatial_svc_encoder.sh: Disable existing tests, add a test that works.

- vp9_spatial_svc_encoder.c no longer supports the -m parameter that
  has been used in the example test. Tests using -m have been disabled.
- Added a basic test that appears to work as of commit
  3249f26ff85e2bfe148167ce80df53643a89a2d2.
- Minor style clean up.

Change-Id: Ic1402fcbbe28e33982c5ea12d1e3349f4069a5bf

1 files changed, 31 insertions, 20 deletions

diff --git a/test/vp9_spatial_svc_encoder.sh b/test/vp9_spatial_svc_encoder.sh
index b48b9e1d2..7a964a920 100755
--- a/test/vp9_spatial_svc_encoder.sh
+++ b/test/vp9_spatial_svc_encoder.sh
@@ -25,12 +25,13 @@ vp9_spatial_svc_encoder_verify_environment() {
 
 # Runs vp9_spatial_svc_encoder. $1 is the test name.
 vp9_spatial_svc_encoder() {
-  local encoder="${LIBVPX_BIN_PATH}/vp9_spatial_svc_encoder"
-  encoder="${encoder}${VPX_TEST_EXE_SUFFIX}"
-  local test_name="$1"
-  local output_file="${VPX_TEST_OUTPUT_DIR}/vp9_ssvc_encoder${test_name}.ivf"
-  local frames_to_encode="10"
-  local max_kf="9999"
+  local readonly \
+    encoder="${LIBVPX_BIN_PATH}/vp9_spatial_svc_encoder${VPX_TEST_EXE_SUFFIX}"
+  local readonly test_name="$1"
+  local readonly \
+    output_file="${VPX_TEST_OUTPUT_DIR}/vp9_ssvc_encoder${test_name}.ivf"
+  local readonly frames_to_encode=10
+  local readonly max_kf=9999
 
   shift
 
@@ -40,8 +41,8 @@ vp9_spatial_svc_encoder() {
   fi
 
   eval "${VPX_TEST_PREFIX}" "${encoder}" -w "${YUV_RAW_INPUT_WIDTH}" \
-      -h "${YUV_RAW_INPUT_HEIGHT}" -k "${max_kf}" -f "${frames_to_encode}" \
-      "$@" "${YUV_RAW_INPUT}" "${output_file}" ${devnull}
+    -h "${YUV_RAW_INPUT_HEIGHT}" -k "${max_kf}" -f "${frames_to_encode}" \
+    "$@" "${YUV_RAW_INPUT}" "${output_file}" ${devnull}
 
   [ -e "${output_file}" ] || return 1
 }
@@ -49,44 +50,54 @@ vp9_spatial_svc_encoder() {
 # Each mode is run with layer count 1-$vp9_ssvc_test_layers.
 vp9_ssvc_test_layers=5
 
-vp9_spatial_svc_mode_i() {
+DISABLED_vp9_spatial_svc_mode_i() {
   if [ "$(vp9_encode_available)" = "yes" ]; then
-    local test_name="${FUNCNAME}"
+    local readonly test_name="DISABLED_vp9_spatial_svc_mode_i"
     for layers in $(seq 1 ${vp9_ssvc_test_layers}); do
       vp9_spatial_svc_encoder "${test_name}" -m i -l ${layers}
     done
   fi
 }
 
-vp9_spatial_svc_mode_altip() {
+DISABLED_vp9_spatial_svc_mode_altip() {
   if [ "$(vp9_encode_available)" = "yes" ]; then
-    local test_name="${FUNCNAME}"
+    local readonly test_name="DISABLED_vp9_spatial_svc_mode_altip"
     for layers in $(seq 1 ${vp9_ssvc_test_layers}); do
       vp9_spatial_svc_encoder "${test_name}" -m "alt-ip" -l ${layers}
     done
   fi
 }
 
-vp9_spatial_svc_mode_ip() {
+DISABLED_vp9_spatial_svc_mode_ip() {
   if [ "$(vp9_encode_available)" = "yes" ]; then
-    local test_name="${FUNCNAME}"
+    local readonly test_name="DISABLED_vp9_spatial_svc_mode_ip"
     vp9_spatial_svc_encoder "${test_name}" -m ip -l 1
   fi
 }
 
-vp9_spatial_svc_mode_gf() {
+DISABLED_vp9_spatial_svc_mode_gf() {
   if [ "$(vp9_encode_available)" = "yes" ]; then
-    local test_name="${FUNCNAME}"
+    local readonly test_name="DISABLED_vp9_spatial_svc_mode_gf"
     for layers in $(seq 1 ${vp9_ssvc_test_layers}); do
       vp9_spatial_svc_encoder "${test_name}" -m gf -l ${layers}
     done
   fi
 }
 
-vp9_spatial_svc_tests="vp9_spatial_svc_mode_i
-                       vp9_spatial_svc_mode_altip
-                       vp9_spatial_svc_mode_ip
-                       vp9_spatial_svc_mode_gf"
+vp9_spatial_svc() {
+  if [ "$(vp9_encode_available)" = "yes" ]; then
+    local readonly test_name="vp9_spatial_svc"
+    for layers in $(seq 1 ${vp9_ssvc_test_layers}); do
+      vp9_spatial_svc_encoder "${test_name}" -l ${layers}
+    done
+  fi
+}
+
+readonly vp9_spatial_svc_tests="DISABLED_vp9_spatial_svc_mode_i
+                                DISABLED_vp9_spatial_svc_mode_altip
+                                DISABLED_vp9_spatial_svc_mode_ip
+                                DISABLED_vp9_spatial_svc_mode_gf
+                                vp9_spatial_svc"
 
 if [ "$(vpx_config_option_enabled CONFIG_SPATIAL_SVC)" = "yes" ]; then
   run_tests \