tests: Replace Bats submodule with a shallow clone

The theory is that this will work better when trying to compose projects
using the framework and its plugins, since it won't be a web of
submodules all the way down. By creating shallow clones on demand when
the `./go` script for the framework, a plugin, or an application is
executed, we may avoid creating circular dependencies and generally
reduce the amount of bandwidth and disk space consumed by each build.

Author: mbland

.gitignore

@@ -1,4 +1,5 @@
 .*.swp
 .DS_Store
+tests/bats/
 tests/coverage/
 tests/kcov/

.gitmodules

@@ -1,4 +0,0 @@
-[submodule "test/bats"]
-	path = tests/bats
-	url = https://github.com/sstephenson/bats
-	shallow = true

scripts/test

@@ -23,18 +23,13 @@
 # This command script can serve as a template for your own project's test
 # script. Copy it into your project's script directory and customize as needed.
 
-declare _GO_BATS_COVERAGE_INCLUDE
-_GO_BATS_COVERAGE_INCLUDE=('go' 'go-core.bash' 'lib/' 'libexec/' 'scripts/')
-declare _GO_COVERALLS_URL='https://coveralls.io/github/mbland/go-script-bash'
-
 # Passes all arguments through to `@go.bats_main` from `lib/bats-main`.
 _test_main() {
-  . "$_GO_USE_MODULES" 'bats-main'
-
-  if [[ ! -d "$_GO_BATS_DIR" ]]; then
-    git submodule update --init "$_GO_BATS_DIR"
-  fi
+  local _GO_BATS_COVERAGE_INCLUDE
+  _GO_BATS_COVERAGE_INCLUDE=('go' 'go-core.bash' 'lib/' 'libexec/' 'scripts/')
+  local _GO_COVERALLS_URL='https://coveralls.io/github/mbland/go-script-bash'
 
+  . "$_GO_USE_MODULES" 'bats-main'
   # Tab completions
   @go.bats_main "$@"
 }

tests/bats

@@ -88,14 +88,12 @@ _trim_expected() {
 
   stub_program_in_path 'git' 'echo "GIT ARGV: $*"'
 
-  # This will fail because we didn't create the tests/ directory, but git should
-  # have been called correctly.
+  # Note that rather than running our own `./go` script, we're running
+  # `TEST_GO_SCRIPT` instead. This will fail because we didn't create the tests/
+  # directory, but `git` should have been called correctly.
   run "$TEST_GO_SCRIPT" test --list test
-
-  # Since `test` will try to clone the submodule, but the `git` stub doesn't
-  # actually do anything, `@go.bats_main` will then call `@go.bats_clone`.
   assert_failure
-  assert_lines_match '^GIT ARGV: submodule update --init tests/bats$' \
+  assert_lines_match \
     "^GIT ARGV: clone .* -b $_GO_BATS_VERSION $_GO_BATS_URL $_GO_BATS_DIR\$" \
     "^Successfully cloned \"$_GO_BATS_URL\" .* \"$_GO_BATS_DIR\"" \
     '^Root directory argument tests is not a directory\.$'