From d465ceb04503832cffca266619d6b863abcde77e Mon Sep 17 00:00:00 2001
From: Dan Callahan <dan.callahan@gmail.com>
Date: Mon, 2 Nov 2015 17:36:20 -0600
Subject: [PATCH] Resolve SC2154: 'variable is referenced but not assigned'

In this case, Bats sets $status, $output, and $lines, but ShellCheck
doesn't statically know that. Therefore, we ignore SC2154 when those
variables are first introduced in ./tests/unit/test_helper.bash

https://github.com/koalaman/shellcheck/wiki/SC2154
---
 tests/unit/test_helper.bash | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/tests/unit/test_helper.bash b/tests/unit/test_helper.bash
index 32cba7a27..f1f6b7174 100644
--- a/tests/unit/test_helper.bash
+++ b/tests/unit/test_helper.bash
@@ -19,6 +19,8 @@ flunk() {
   return 1
 }
 
+# ShellCheck doesn't know about $status from Bats
+# shellcheck disable=SC2154
 assert_success() {
   if [[ "$status" -ne 0 ]]; then
     flunk "command failed with exit status $status"
@@ -43,6 +45,8 @@ assert_equal() {
   fi
 }
 
+# ShellCheck doesn't know about $output from Bats
+# shellcheck disable=SC2154
 assert_output() {
   local expected
   if [[ $# -eq 0 ]]; then
@@ -53,6 +57,8 @@ assert_output() {
   assert_equal "$expected" "$output"
 }
 
+# ShellCheck doesn't know about $lines from Bats
+# shellcheck disable=SC2154
 assert_line() {
   if [[ "$1" -ge 0 ]] 2>/dev/null; then
     assert_equal "$2" "${lines[$1]}"