update blues branch with xiatechs branch

.github/workflows/test.yml

@@ -1,26 +1,22 @@
-on: [push]
+on:
+  push:
+    branches: '*'
+  pull_request:
+    branches: '*'
+
 name: Test
 jobs:
   test:
-    strategy:
-      matrix:
-        go-version: [1.16.x]
-        os: [ubuntu-latest]
-    runs-on: ${{ matrix.os }}
+    name: Unit Tests
+    runs-on: ubuntu-latest
     steps:
-    - name: Install Go
-      uses: actions/setup-go@v2
-      with:
-        go-version: ${{ matrix.go-version }}
-    - name: Checkout code
-      uses: actions/checkout@v2
-    - name: Install dependencies
-      run: |
-        go get -u honnef.co/go/tools/cmd/staticcheck@latest
-        go get -u golang.org/x/tools/cmd/goimports
-    - name: Run staticcheck
-      run: staticcheck ./...
-    - name: Check code formatting
-      run: test -z $(goimports -l .)
-    - name: Run Test
-      run: go test ./...
+      - name: Install Go
+        uses: actions/setup-go@v4
+        with:
+          go-version: '1.20.1'
+
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Unit tests
+        run: go test ./... -race

.gitignore

@@ -4,6 +4,8 @@
 *.dll
 *.so
 *.dylib
+.idea
+.idea/*
 
 # Test binary, built with `go test -c`
 *.test
@@ -13,3 +15,7 @@
 
 # Dependency directories (remove the comment below to include it)
 # vendor/
+
+# specific files
+jlib/timeparse/outputdata.json
+jlib/timeparse/outputdata_lite.json

CONTRIBUTING.md

@@ -1,4 +1,4 @@
-# Contributing to blues/jsonata-go
+# Contributing to blues/jsonata-go [xiatechs fork]
 
 We love pull requests from everyone. By participating in this project, you
 agree to abide by the Blues Inc [code of conduct].
@@ -18,7 +18,7 @@ clean up inconsistent whitespace )
 * by closing [issues][]
 * by reviewing patches
 
-[issues]: https://github.com/blues/jsonata-go/issues
+[issues]: https://github.com/xiatechs/jsonata-go/issues
 
 ## Submitting an Issue
 
@@ -55,7 +55,7 @@ clean up inconsistent whitespace )
 * If you don't know how to add tests, please put in a PR and leave a comment asking for help. 
 We love helping!
 
-[repo]: https://github.com/blues/jsonata-go/tree/master
+[repo]: https://github.com/xiatechs/jsonata-go/tree/master
 [fork]: https://help.github.com/articles/fork-a-repo/
 [branch]:
 https://help.github.com/articles/creating-and-deleting-branches-within-your-repository/

README.md

@@ -7,17 +7,17 @@ It currently has feature parity with jsonata-js 1.5.4. As well as a most of the
 
 ## Install
 
-    go get github.com/blues/jsonata-go
+    go get github.com/xiatechs/jsonata-go
 
 ## Usage
 
 ```Go
 import (
-	"encoding/json"
+	"github.com/goccy/go-json"
 	"fmt"
 	"log"
 
-	jsonata "github.com/blues/jsonata-go"
+	jsonata "github.com/xiatechs/jsonata-go"
 )
 
 const jsonString = `
@@ -56,7 +56,7 @@ func main() {
 
 ## JSONata Server
 A locally hosted version of [JSONata Exerciser](http://try.jsonata.org/)
-for testing is [available here](https://github.com/blues/jsonata-go/jsonata-server).
+for testing is [available here](https://github.com/xiatechs/jsonata-go/jsonata-server).
 
 ## JSONata tests
 A CLI tool for running jsonata-go against the [JSONata test suite](https://github.com/jsonata-js/jsonata/tree/master/test/test-suite) is [available here](./jsonata-test).

callable.go

@@ -5,18 +5,20 @@
 package jsonata
 
 import (
-	"encoding/json"
 	"fmt"
+	"github.com/goccy/go-json"
 	"reflect"
 	"regexp"
 	"strings"
+	"sync"
 
-	"github.com/blues/jsonata-go/jlib"
-	"github.com/blues/jsonata-go/jparse"
-	"github.com/blues/jsonata-go/jtypes"
+	"github.com/xiatechs/jsonata-go/jlib"
+	"github.com/xiatechs/jsonata-go/jparse"
+	"github.com/xiatechs/jsonata-go/jtypes"
 )
 
 type callableName struct {
+	mu   sync.Mutex
 	name string
 }
 
@@ -25,6 +27,8 @@ func (n callableName) Name() string {
 }
 
 func (n *callableName) SetName(s string) {
+	n.mu.Lock()
+	defer n.mu.Unlock()
 	n.name = s
 }
 
@@ -76,6 +80,7 @@ func newGoCallableParam(typ reflect.Type) goCallableParam {
 // A goCallable represents a built-in or third party Go function.
 // It implements the Callable interface.
 type goCallable struct {
+	mu sync.Mutex
 	callableName
 	callableMarshaler
 	fn               reflect.Value
@@ -205,6 +210,8 @@ func makeGoCallableParams(typ reflect.Type) []goCallableParam {
 }
 
 func (c *goCallable) SetContext(context reflect.Value) {
+	c.mu.Lock()
+	defer c.mu.Unlock()
 	c.context = context
 }
 
@@ -682,7 +689,7 @@ func (f *transformationCallable) Call(argv []reflect.Value) (reflect.Value, erro
 
 	obj, err := f.clone(argv[0])
 	if err != nil {
-		return undefined, newEvalError(ErrClone, nil, nil)
+		return undefined, newEvalError(ErrClone, nil, nil, 0)
 	}
 
 	if obj == undefined {
@@ -739,7 +746,7 @@ func (f *transformationCallable) updateEntries(item reflect.Value) error {
 	}
 
 	if !jtypes.IsMap(updates) {
-		return newEvalError(ErrIllegalUpdate, f.updates, nil)
+		return newEvalError(ErrIllegalUpdate, f.updates, nil, 0)
 	}
 
 	for _, key := range updates.MapKeys() {
@@ -759,7 +766,7 @@ func (f *transformationCallable) deleteEntries(item reflect.Value) error {
 	deletes = arrayify(deletes)
 
 	if !jtypes.IsArrayOf(deletes, jtypes.IsString) {
-		return newEvalError(ErrIllegalDelete, f.deletes, nil)
+		return newEvalError(ErrIllegalDelete, f.deletes, nil, 0)
 	}
 
 	for i := 0; i < deletes.Len(); i++ {

callable_test.go

@@ -11,10 +11,13 @@ import (
 	"regexp"
 	"sort"
 	"strings"
+	"sync"
 	"testing"
 
-	"github.com/blues/jsonata-go/jparse"
-	"github.com/blues/jsonata-go/jtypes"
+	"github.com/stretchr/testify/assert"
+
+	"github.com/xiatechs/jsonata-go/jparse"
+	"github.com/xiatechs/jsonata-go/jtypes"
 )
 
 var (
@@ -2328,8 +2331,8 @@ func testTransformationCallable(t *testing.T, tests []transformationCallableTest
 			}
 		}
 
-		if !reflect.DeepEqual(err, test.Error) {
-			t.Errorf("transform %d: expected error %v, got %v", i+1, test.Error, err)
+		if err != nil && test.Error != nil {
+			assert.EqualError(t, err, test.Error.Error())
 		}
 	}
 }
@@ -2371,6 +2374,7 @@ func TestRegexCallable(t *testing.T) {
 				"groups": []string{},
 				"next": &matchCallable{
 					callableName: callableName{
+						sync.Mutex{},
 						"next",
 					},
 					match:  "ac",
@@ -2379,6 +2383,7 @@ func TestRegexCallable(t *testing.T) {
 					groups: []string{},
 					next: &matchCallable{
 						callableName: callableName{
+							sync.Mutex{},
 							"next",
 						},
 						match:  "ad",
@@ -2387,6 +2392,7 @@ func TestRegexCallable(t *testing.T) {
 						groups: []string{},
 						next: &matchCallable{
 							callableName: callableName{
+								sync.Mutex{},
 								"next",
 							},
 							match:  "ab",
@@ -2395,6 +2401,7 @@ func TestRegexCallable(t *testing.T) {
 							groups: []string{},
 							next: &matchCallable{
 								callableName: callableName{
+									sync.Mutex{},
 									"next",
 								},
 								match:  "a",
@@ -2425,6 +2432,7 @@ func TestRegexCallable(t *testing.T) {
 				},
 				"next": &matchCallable{
 					callableName: callableName{
+						sync.Mutex{},
 						"next",
 					},
 					match: "ac",
@@ -2435,6 +2443,7 @@ func TestRegexCallable(t *testing.T) {
 					},
 					next: &matchCallable{
 						callableName: callableName{
+							sync.Mutex{},
 							"next",
 						},
 						match: "ad",
@@ -2445,6 +2454,7 @@ func TestRegexCallable(t *testing.T) {
 						},
 						next: &matchCallable{
 							callableName: callableName{
+								sync.Mutex{},
 								"next",
 							},
 							match: "ab",
@@ -2455,6 +2465,7 @@ func TestRegexCallable(t *testing.T) {
 							},
 							next: &matchCallable{
 								callableName: callableName{
+									sync.Mutex{},
 									"next",
 								},
 								match: "a",
@@ -2491,6 +2502,7 @@ func TestRegexCallable(t *testing.T) {
 				},
 				"next": &matchCallable{
 					callableName: callableName{
+						sync.Mutex{},
 						"next",
 					},
 					match: "ac",
@@ -2502,6 +2514,7 @@ func TestRegexCallable(t *testing.T) {
 					},
 					next: &matchCallable{
 						callableName: callableName{
+							sync.Mutex{},
 							"next",
 						},
 						match: "ad",
@@ -2513,6 +2526,7 @@ func TestRegexCallable(t *testing.T) {
 						},
 						next: &matchCallable{
 							callableName: callableName{
+								sync.Mutex{},
 								"next",
 							},
 							match: "ab",
@@ -2524,6 +2538,7 @@ func TestRegexCallable(t *testing.T) {
 							},
 							next: &matchCallable{
 								callableName: callableName{
+									sync.Mutex{},
 									"next",
 								},
 								match: "a",

config/config.go

@@ -0,0 +1,7 @@
+package config
+
+var defaultDivisionPrecision int32 = 8
+
+func GetDivisionPrecision() int32 {
+	return defaultDivisionPrecision
+}