test: Rename and better categorize (move) tests

Author: hyperupcall

README.md

@@ -47,7 +47,7 @@ assert [ "$REPLY" = rho ]
 
 ## Installation
 
-STATUS: ALPHA
+STATUS: BETA
 
 ```sh
 echo "dependencies = [ 'hyperupcall/bash-object' ]" > 'bpm.toml'

tests/get-errors.bats

@@ -1,3 +1,15 @@
-#!/usr/bin/env bash
+#!/usr/bin/env bats
 
 load './util/init.sh'
+
+@test "Error if getting more than a string" {
+	declare -A OBJECT=()
+	str='golf'
+
+	bobject set-string --ref OBJECT '.foxtrot' str
+	run bobject get-string --ref OBJECT '.foxtrot.omega'
+
+	assert_failure
+	assert_line -p 'ERROR_NOT_FOUND'
+	assert_line -p "The passed querytree implies that 'foxtrot' accesses an object or array, but a string with a value of 'golf' was found instead"
+}

tests/get.bats renamed to tests/get-old.bats

@@ -0,0 +1,15 @@
+#!/usr/bin/env bats
+
+load './util/init.sh'
+
+@test "Correctly gets object" {
+	declare -A inner_object=([cool]='Wolf 359')
+	declare -A OBJ=([stars]=$'\x1C\x1Dtype=object;&inner_object')
+
+	bobject set-object --value
+	bobject get-object --value 'OBJ' '.stars'
+	assert [ "${REPLY[cool]}" = 'Wolf 359' ]
+
+	bobject get-string --value 'OBJ' '.stars.cool'
+	assert [ "$REPLY" = 'Wolf 359' ]
+}

tests/get-value.bats

@@ -2,6 +2,47 @@
 
 load './util/init.sh'
 
+# object
+@test "Correctly sets object" {
+	declare -A obj=([omicron]=pi [rho]=sigma [tau]=upsilon)
+	declare -A OBJECT=()
+
+	bobject set-object --ref 'OBJECT' '.obj' obj
+
+	bobject get-object --value 'OBJECT' '.obj'
+	assert [ ${#REPLY[@]} -eq 3 ]
+	assert [ "${REPLY[omicron]}" = pi ]
+	assert [ "${REPLY[rho]}" = sigma ]
+	assert [ "${REPLY[tau]}" = upsilon ]
+
+	bobject get-string --value 'OBJECT' '.obj.rho'
+	assert [ "$REPLY" = sigma ]
+}
+
+# array
+@test "Correctly sets array" {
+	declare -a arr=(omicron pi rho)
+	declare -A OBJECT=()
+
+	bobject set-array --ref 'OBJECT' '.arr' arr
+
+	bobject get-array --value 'OBJECT' '.arr'
+	assert [ ${#REPLY[@]} -eq 3 ]
+	assert [ "${REPLY[0]}" = omicron ]
+	assert [ "${REPLY[1]}" = pi ]
+	assert [ "${REPLY[2]}" = rho ]
+
+	bobject get-string --value 'OBJECT' '.["arr"].[0]'
+	assert [ "$REPLY" = omicron ]
+
+	bobject get-string --value 'OBJECT' '.["arr"].[1]'
+	assert [ "$REPLY" = pi ]
+
+	bobject get-string --value 'OBJECT' '.["arr"].[2]'
+	assert [ "$REPLY" = rho ]
+}
+
+# string
 @test "Correctly sets string at root" {
 	declare -A OBJECT=()
 	str='my_value'

tests/parse-filter-advanced.bats renamed to tests/querytree-advanced.bats

@@ -1,7 +1,19 @@
-# shellcheck shell=bash
+#!/usr/bin/env bats
 
 load './util/init.sh'
 
+@test "Error if setting more than a string" {
+	declare -A OBJECT=()
+	str='golf'
+
+	bobject set-string --ref OBJECT '.foxtrot' str
+	run bobject set-string --ref OBJECT '.foxtrot.omega' str
+
+	assert_failure
+	assert_line -p 'ERROR_NOT_FOUND'
+	assert_line -p "The passed querytree implies that 'foxtrot' accesses an object or array, but a string with a value of 'golf' was found instead"
+}
+
 @test "Error if random variable already exists for set-object" {
 	declare -A OBJECT=()
 	declare -A obj=()
@@ -33,28 +45,3 @@ load './util/init.sh'
 	assert_output -p 'ERROR_INTERNAL'
 	assert_output -p "Variable 'some_other_var' exists, but it shouldn't"
 }
-
-@test "Error if setting more than a string" {
-	declare -A OBJECT=()
-	str='golf'
-
-	bobject set-string --ref OBJECT '.foxtrot' str
-	run bobject set-string --ref OBJECT '.foxtrot.omega' str
-
-	assert_failure
-	assert_line -p 'ERROR_NOT_FOUND'
-	assert_line -p "The passed querytree implies that 'foxtrot' accesses an object or array, but a string with a value of 'golf' was found instead"
-}
-
-# TODO: move
-@test "Error if getting more than a string" {
-	declare -A OBJECT=()
-	str='golf'
-
-	bobject set-string --ref OBJECT '.foxtrot' str
-	run bobject get-string --ref OBJECT '.foxtrot.omega'
-
-	assert_failure
-	assert_line -p 'ERROR_NOT_FOUND'
-	assert_line -p "The passed querytree implies that 'foxtrot' accesses an object or array, but a string with a value of 'golf' was found instead"
-}