Quick Save

Author: rsdoiel

cmds/jsonrange/jsonrange.go

@@ -268,7 +268,7 @@ func main() {
 	app.BoolVar(&showLast, "last", false, "return the index of the last element in list (e.g. length - 1)")
 	app.BoolVar(&showValues, "values", false, "return the values instead of the keys")
 	app.StringVar(&delimiter, "d,delimiter", "", "set delimiter for range output")
-	app.IntVar(&limit, "limit", 0, "limit the number of items output")
+	app.IntVar(&limit, "limit", -1, "limit the number of items output")
 
 	// Parse options and environment
 	app.Parse()

demos/jsonrange/demo.bash

@@ -8,11 +8,11 @@ cat person.json
 echo ''
 
 echo ' running: cat person.json | jsonrange -i -'
-cat person.json | jsonrange -i - > result1.txt
+cat person.json | jsonrange -i - | sort > result1.txt
 cat <<EOF > expected1.txt
-name
-email
 age
+email
+name
 EOF
 echo -n 'expected: '
 cat expected1.txt
@@ -21,11 +21,11 @@ cat result1.txt
 echo ''
 
 echo ' running: jsonrange -i person.json -values'
-jsonrange -i person.json -values | result2.txt
+jsonrange -i person.json -values | sort > result2.txt
 cat <<EOF > expected2.txt
+42
 "Doe, Jane"
 "[email protected]"
-42
 EOF
 echo -n 'expected: '
 cat expected2.txt

demos/jsonrange/expected1.txt

@@ -1,3 +1,3 @@
-name
-email
 age
+email
+name

demos/jsonrange/expected8.txt

@@ -1,2 +1,2 @@
+0
 1
-2

test_cmds.bash

@@ -606,6 +606,58 @@ function test_jsonmunge(){
 
 function test_jsonrange(){
     if [ -f temp.txt ]; then rm temp.txt; fi
+    cat demos/jsonrange/person.json | bin/jsonrange -i - | sort > temp.txt
+    assert_exists "test_jsonrange (result1)" temp.txt
+    R=$(cmp demos/jsonrange/expected1.txt temp.txt)
+    assert_empty "test_jsonrange (result1)" "$R"
+
+    if [ -f temp.txt ]; then rm temp.txt; fi
+    bin/jsonrange -i demos/jsonrange/person.json -values | sort > temp.txt
+    assert_exists "test_jsonrange (expected2)" temp.txt
+    R=$(cmp demos/jsonrange/expected2.txt temp.txt)
+    assert_empty "test_jsonrange (expected2)" "$R"
+
+    if [ -f temp.txt ]; then rm temp.txt; fi
+    bin/jsonrange -i demos/jsonrange/array1.json -o temp.txt
+    assert_exists "test_jsonrange (expected3)" temp.txt
+    R=$(cmp demos/jsonrange/expected3.txt temp.txt)
+    assert_empty "test_jsonrange (expected3)" "$R"
+
+    if [ -f temp.txt ]; then rm temp.txt; fi
+    bin/jsonrange -i demos/jsonrange/array1.json -values -o temp.txt
+    assert_exists "test_jsonrange (expected4)" temp.txt
+    R=$(cmp demos/jsonrange/expected4.txt temp.txt)
+    assert_empty "test_jsonrange (expected4)" "$R"
+
+    if [ -f temp.txt ]; then rm temp.txt; fi
+    bin/jsonrange -i demos/jsonrange/array2.json -length -o temp.txt
+    assert_exists "test_jsonrange (expected5)" temp.txt
+    R=$(cmp demos/jsonrange/expected5.txt temp.txt)
+    assert_empty "test_jsonrange (expected5)" "$R"
+
+    if [ -f temp.txt ]; then rm temp.txt; fi
+    bin/jsonrange -i demos/jsonrange/array2.json -last -o temp.txt
+    assert_exists "test_jsonrange (expected6)" temp.txt
+    R=$(cmp demos/jsonrange/expected6.txt temp.txt)
+    assert_empty "test_jsonrange (expected6)" "$R"
+
+    if [ -f temp.txt ]; then rm temp.txt; fi
+    bin/jsonrange -nl -i demos/jsonrange/array2.json -values -last -o temp.txt
+    assert_exists "test_jsonrange (expected7)" temp.txt
+    R=$(cmp demos/jsonrange/expected7.txt temp.txt)
+    assert_empty "test_jsonrange (expected7)" "$R"
+
+    if [ -f temp.txt ]; then rm temp.txt; fi
+    bin/jsonrange -i demos/jsonrange/array3.json -limit 2 -o temp.txt
+    assert_exists "test_jsonrange (expected8)" temp.txt
+    R=$(cmp demos/jsonrange/expected8.txt temp.txt)
+    assert_empty "test_jsonrange (expected8)" "$R"
+
+    if [ -f temp.txt ]; then rm temp.txt; fi
+    bin/jsonrange -i demos/jsonrange/array3.json -values -limit 2 -o temp.txt
+    assert_exists "test_jsonrange (expected9)" temp.txt
+    R=$(cmp demos/jsonrange/expected9.txt temp.txt)
+    assert_empty "test_jsonrange (expected9)" "$R"
 
     if [ -f temp.txt ]; then rm temp.txt; fi
     echo "test_jsonrange OK";