Skip to content

Commit ad7bf70

Browse files
committed
fix broken tests
1 parent 82cbcfc commit ad7bf70

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

lib/gateway/gateway_test.go

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import (
77
"github.com/stackup-app/stackup/lib/gateway"
88
"github.com/stackup-app/stackup/lib/scripting"
99
"github.com/stackup-app/stackup/lib/settings"
10-
"github.com/stackup-app/stackup/lib/types"
1110
"github.com/stretchr/testify/assert"
1211
)
1312

@@ -40,9 +39,9 @@ func TestGatewayInitialize(t *testing.T) {
4039
g := gateway.New(nil)
4140
findTask := func(id string) (any, error) { return nil, nil }
4241
engine := scripting.CreateNewJavascriptEngine(&sync.Map{}, g, findTask, func() string { return "." })
43-
var engineIntf interface{} = engine
44-
var engineContract types.JavaScriptEngineContract = engineIntf.(types.JavaScriptEngineContract)
45-
g.Initialize(s, engineContract, nil)
42+
// var engineIntf interface{} =
43+
// var engineContract types.JavaScriptEngineContract = engineIntf.(types.JavaScriptEngineContract)
44+
g.Initialize(s, engine.AsContract(), nil)
4645

4746
assert.True(t, g.Enabled, "gateway should be enabled")
4847
assert.Equal(t, 3, len(g.AllowedDomains), "gateway should have 3 allowed domains")

lib/scripting/scriptEngine.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,9 @@ func (e *JavaScriptEngine) Initialize(appVars *sync.Map, environ []string) {
7979
e.CreateEnvironmentVariables(environ)
8080
}
8181

82-
func (e *JavaScriptEngine) GetFindTaskById(id string) (any, error) {
83-
return e.FindTaskById(id)
82+
func (e JavaScriptEngine) GetFindTaskById(id string) (any, bool) { //GetFindTaskById(id string) (any, error) {
83+
result, err := e.FindTaskById(id)
84+
return result, err == nil
8485
}
8586

8687
func (e *JavaScriptEngine) initializeExtensions() {

0 commit comments

Comments
 (0)