fixup! Fix reload & setContent to use navigation timeout

grafana · Sep 18, 2023 · c35d40f · c35d40f
1 parent 499d0d0
commit c35d40f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/common/frame.go b/common/frame.go
@@ -1429,7 +1429,7 @@ func (f *Frame) SetContent(html string, opts goja.Value) {
 	f.log.Debugf("Frame:SetContent", "fid:%s furl:%q", f.ID(), f.URL())
 
 	parsedOpts := NewFrameSetContentOptions(
-		time.Duration(f.manager.timeoutSettings.navigationTimeout()) * time.Second,
+		f.manager.timeoutSettings.navigationTimeout(),
 	)
 	if err := parsedOpts.Parse(f.ctx, opts); err != nil {
 		k6ext.Panic(f.ctx, "parsing set content options: %w", err)

diff --git a/common/page.go b/common/page.go
@@ -862,7 +862,7 @@ func (p *Page) Reload(opts goja.Value) api.Response {
 
 	parsedOpts := NewPageReloadOptions(
 		LifecycleEventLoad,
-		time.Duration(p.timeoutSettings.navigationTimeout())*time.Second,
+		p.timeoutSettings.navigationTimeout(),
 	)
 	if err := parsedOpts.Parse(p.ctx, opts); err != nil {
 		k6ext.Panic(p.ctx, "parsing reload options: %w", err)