fix: bpm-load prints better errors

Author: hyperupcall

pkg/lib/cmd/bpm-load.sh

@@ -1,7 +1,9 @@
 #!/usr/bin/env bash
 
-# TODO: print error if sourcing and passing arguments
-# TODO: print error if not being sourced and no arguments are passed
+if [[ "${BASH_SOURCE[0]}" != "${0}" && $# -ne 0 ]]; then
+	printf '%s\n' "bpm-load: Error: Incorrect usage. See documentation"
+	return 1
+fi
 
 # @description Source Bash packages to initialize any functions
 # that they may want to provide in the global scope

tests/bpm-load.bats

@@ -21,6 +21,58 @@ load 'util/init.sh'
 	assert_output "it works :)"
 }
 
-@test "properly restores options" {
-	:
+@test "works with file argument" {
+	local site='github.com'
+	local pkg="user/project2"
+
+	BPM_REPO_SOURCE="$BPM_TEST_REPO_ROOT/../source"
+	BPM_CELLAR="$BPM_TEST_DIR/cellar"
+
+	test_util.setup_pkg "$pkg"; {
+		echo "printf '%s\n' 'it works :)'" > 'file.bash'
+	}; test_util.finish_pkg
+	test_util.mock_add "$pkg"
+
+	source bpm-load
+	run bpm-load --global "$pkg" 'file.bash'
+
+	assert_success
+	assert_output "it works :)"
+}
+
+@test "errors if used incorrectly (soucing with arguments passed)" {
+	local site='github.com'
+	local pkg="user/project2"
+
+	BPM_REPO_SOURCE="$BPM_TEST_REPO_ROOT/../source"
+	BPM_CELLAR="$BPM_TEST_DIR/cellar"
+
+	test_util.setup_pkg "$pkg"; {
+		echo "printf '%s\n' 'it works :)'" > 'load.bash'
+	}; test_util.finish_pkg
+	test_util.mock_add "$pkg"
+
+	run source bpm-load --global "$pkg"
+
+	assert_failure
+	assert_line -p "Incorrect usage. See documentation"
+}
+
+@test "errors if used incorrectly (running function with no arguments passed)" {
+	local site='github.com'
+	local pkg="user/project2"
+
+	BPM_REPO_SOURCE="$BPM_TEST_REPO_ROOT/../source"
+	BPM_CELLAR="$BPM_TEST_DIR/cellar"
+
+	test_util.setup_pkg "$pkg"; {
+		echo "printf '%s\n' 'it works :)'" > 'load.bash'
+	}; test_util.finish_pkg
+	test_util.mock_add "$pkg"
+
+	source bpm-load
+	run bpm-load
+
+	assert_failure
+	assert_line -p "Error: Must pass in package name as first parameter"
 }