Remove parallel test execution

inancgumus · inancgumus · commit 23b236dfa502 · 2021-12-15T14:00:11.000+03:00
diff --git a/tests/browser_context_options_test.go b/tests/browser_context_options_test.go
@@ -56,8 +56,6 @@ func TestBrowserContextOptionsDefaultValues(t *testing.T) {
 }
 
 func TestBrowserContextOptionsDefaultViewport(t *testing.T) {
-	t.Parallel()
-
 	p := newTestBrowser(t).NewPage(nil)
 
 	viewportSize := p.ViewportSize()
@@ -66,8 +64,6 @@ func TestBrowserContextOptionsDefaultViewport(t *testing.T) {
 }
 
 func TestBrowserContextOptionsSetViewport(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	bctx := tb.NewContext(tb.rt.ToValue(struct {
 		Viewport common.Viewport `js:"viewport"`
@@ -86,8 +82,6 @@ func TestBrowserContextOptionsSetViewport(t *testing.T) {
 }
 
 func TestBrowserContextOptionsExtraHTTPHeaders(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t, withHTTPServer())
 	bctx := tb.NewContext(tb.rt.ToValue(struct {
 		ExtraHTTPHeaders map[string]string `js:"extraHTTPHeaders"`
diff --git a/tests/browser_test.go b/tests/browser_test.go
@@ -29,8 +29,6 @@ import (
 )
 
 func TestBrowserNewPage(t *testing.T) {
-	t.Parallel()
-
 	b := newTestBrowser(t)
 	p := b.NewPage(nil)
 	l := len(b.Contexts())
@@ -50,8 +48,6 @@ func TestBrowserNewPage(t *testing.T) {
 
 // This only works for Chrome!
 func TestBrowserVersion(t *testing.T) {
-	t.Parallel()
-
 	const re = `^\d+\.\d+\.\d+\.\d+$`
 	r, _ := regexp.Compile(re)
 	ver := newTestBrowser(t).Version()
@@ -62,8 +58,6 @@ func TestBrowserVersion(t *testing.T) {
 // TODO: Improve this test, see:
 // https://github.com/grafana/xk6-browser/pull/51#discussion_r742696736
 func TestBrowserUserAgent(t *testing.T) {
-	t.Parallel()
-
 	b := newTestBrowser(t)
 
 	// testBrowserVersion() tests the version already
diff --git a/tests/element_handle_bounding_box_test.go b/tests/element_handle_bounding_box_test.go
@@ -29,8 +29,6 @@ import (
 )
 
 func TestElementHandleBoundingBoxInvisibleElement(t *testing.T) {
-	t.Parallel()
-
 	p := newTestBrowser(t).NewPage(nil)
 
 	p.SetContent(`<div style="display:none">hello</div>`, nil)
@@ -40,8 +38,6 @@ func TestElementHandleBoundingBoxInvisibleElement(t *testing.T) {
 }
 
 func TestElementHandleBoundingBoxSVG(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	p := tb.NewPage(nil)
 
diff --git a/tests/element_handle_click_test.go b/tests/element_handle_click_test.go
@@ -68,8 +68,6 @@ var htmlInputButton = fmt.Sprintf(`
 `, mouseHelperScriptSource)
 
 func TestElementHandleClick(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	p := tb.NewPage(nil)
 
@@ -92,8 +90,6 @@ func TestElementHandleClick(t *testing.T) {
 }
 
 func TestElementHandleClickWithNodeRemoved(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	p := tb.NewPage(nil)
 
@@ -119,8 +115,6 @@ func TestElementHandleClickWithNodeRemoved(t *testing.T) {
 }
 
 func TestElementHandleClickWithDetachedNode(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	p := tb.NewPage(nil)
 
diff --git a/tests/element_handle_getattribute_test.go b/tests/element_handle_getattribute_test.go
@@ -28,8 +28,6 @@ import (
 )
 
 func TestElementHandleGetAttribute(t *testing.T) {
-	t.Parallel()
-
 	const want = "https://somewhere"
 
 	p := newTestBrowser(t).NewPage(nil)
diff --git a/tests/element_handle_input_value_test.go b/tests/element_handle_input_value_test.go
@@ -28,8 +28,6 @@ import (
 )
 
 func TestElementHandleInputValue(t *testing.T) {
-	t.Parallel()
-
 	p := newTestBrowser(t).NewPage(nil)
 
 	p.SetContent(`
diff --git a/tests/element_handle_is_checked_test.go b/tests/element_handle_is_checked_test.go
@@ -28,8 +28,6 @@ import (
 )
 
 func TestElementHandleIsChecked(t *testing.T) {
-	t.Parallel()
-
 	p := newTestBrowser(t).NewPage(nil)
 
 	p.SetContent(`<input type="checkbox" checked>`, nil)
diff --git a/tests/element_handle_screenshot_test.go b/tests/element_handle_screenshot_test.go
@@ -30,8 +30,6 @@ import (
 )
 
 func TestElementHandleScreenshot(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	p := tb.NewPage(nil)
 
@@ -40,20 +38,22 @@ func TestElementHandleScreenshot(t *testing.T) {
 		Height float64 `js:"height"`
 	}{Width: 800, Height: 600}))
 	p.Evaluate(tb.rt.ToValue(`
-         () => {
-             document.body.style.margin = '0';
-             document.body.style.padding = '0';
-             document.documentElement.style.margin = '0';
-             document.documentElement.style.padding = '0';
-             const div = document.createElement('div');
-             div.style.marginTop = '400px';
-             div.style.marginLeft = '100px';
-             div.style.width = '100px';
-             div.style.height = '100px';
-             div.style.background = 'red';
-             document.body.appendChild(div);
-         }
-    `))
+		() => {
+			document.body.style.margin = '0';
+			document.body.style.padding = '0';
+			document.documentElement.style.margin = '0';
+			document.documentElement.style.padding = '0';
+
+			const div = document.createElement('div');
+			div.style.marginTop = '400px';
+			div.style.marginLeft = '100px';
+			div.style.width = '100px';
+			div.style.height = '100px';
+			div.style.background = 'red';
+			
+			document.body.appendChild(div);
+		}
+    	`))
 
 	elem := p.Query("div")
 	buf := elem.Screenshot(nil)
diff --git a/tests/element_handle_waitforselector_test.go b/tests/element_handle_waitforselector_test.go
@@ -28,8 +28,6 @@ import (
 )
 
 func TestElementHandleWaitForSelector(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	p := tb.NewPage(nil)
 	p.SetContent(`<div class="root"></div>`, nil)
diff --git a/tests/js_handle_get_properties_test.go b/tests/js_handle_get_properties_test.go
@@ -28,8 +28,6 @@ import (
 )
 
 func TestJSHandleGetProperties(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	p := tb.NewPage(nil)
 
diff --git a/tests/network_manager_test.go b/tests/network_manager_test.go
@@ -27,8 +27,6 @@ import (
 )
 
 func TestDataURLSkipRequest(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	p := tb.NewPage(nil)
 
diff --git a/tests/page_emulate_media_test.go b/tests/page_emulate_media_test.go
@@ -36,8 +36,6 @@ type emulateMediaOpts struct {
 }
 
 func TestPageEmulateMedia(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	p := tb.NewPage(nil)
 
diff --git a/tests/page_input_value_test.go b/tests/page_input_value_test.go
@@ -28,8 +28,6 @@ import (
 )
 
 func TestPageInputValue(t *testing.T) {
-	t.Parallel()
-
 	p := newTestBrowser(t).NewPage(nil)
 
 	p.SetContent(`
@@ -50,8 +48,6 @@ func TestPageInputValue(t *testing.T) {
 
 // test for: https://github.com/grafana/xk6-browser/issues/132
 func TestPageInputSpecialCharacters(t *testing.T) {
-	t.Parallel()
-
 	p := newTestBrowser(t).NewPage(nil)
 
 	p.SetContent(`<input id="special">`, nil)
diff --git a/tests/page_is_checked_test.go b/tests/page_is_checked_test.go
@@ -28,8 +28,6 @@ import (
 )
 
 func TestPageIsChecked(t *testing.T) {
-	t.Parallel()
-
 	p := newTestBrowser(t).NewPage(nil)
 
 	p.SetContent(`<input type="checkbox" checked>`, nil)
diff --git a/tests/page_screenshot_test.go b/tests/page_screenshot_test.go
@@ -30,8 +30,6 @@ import (
 )
 
 func TestPageScreenshotFullpage(t *testing.T) {
-	t.Parallel()
-
 	tb := newTestBrowser(t)
 	p := tb.NewPage(nil)
 
@@ -40,18 +38,20 @@ func TestPageScreenshotFullpage(t *testing.T) {
 		Height float64 `js:"height"`
 	}{Width: 1280, Height: 800}))
 	p.Evaluate(tb.rt.ToValue(`
-        () => {
-            document.body.style.margin = '0';
-            document.body.style.padding = '0';
-            document.documentElement.style.margin = '0';
-            document.documentElement.style.padding = '0';
-            const div = document.createElement('div');
-            div.style.width = '1280px';
-            div.style.height = '8000px';
-            div.style.background = 'linear-gradient(red, blue)';
-            document.body.appendChild(div);
-        }
-    `))
+	() => {
+		document.body.style.margin = '0';
+		document.body.style.padding = '0';
+		document.documentElement.style.margin = '0';
+		document.documentElement.style.padding = '0';
+
+		const div = document.createElement('div');
+		div.style.width = '1280px';
+		div.style.height = '8000px';
+		div.style.background = 'linear-gradient(red, blue)';
+		
+		document.body.appendChild(div);
+	}
+    	`))
 
 	buf := p.Screenshot(tb.rt.ToValue(struct {
 		FullPage bool `js:"fullPage"`
diff --git a/tests/page_test.go b/tests/page_test.go
@@ -31,8 +31,6 @@ import (
 )
 
 func TestPageGoto(t *testing.T) {
-	t.Parallel()
-
 	b := newTestBrowser(t, withFileServer())
 
 	p := b.NewPage(nil)
@@ -44,8 +42,6 @@ func TestPageGoto(t *testing.T) {
 }
 
 func TestPageGotoDataURI(t *testing.T) {
-	t.Parallel()
-
 	p := newTestBrowser(t).NewPage(nil)
 
 	r := p.Goto("data:text/html,hello", nil)
@@ -54,8 +50,6 @@ func TestPageGotoDataURI(t *testing.T) {
 }
 
 func TestPageGotoWaitUntilLoad(t *testing.T) {
-	t.Parallel()
-
 	b := newTestBrowser(t, withFileServer())
 
 	p := b.NewPage(nil)
@@ -72,8 +66,6 @@ func TestPageGotoWaitUntilLoad(t *testing.T) {
 }
 
 func TestPageGotoWaitUntilDOMContentLoaded(t *testing.T) {
-	t.Parallel()
-
 	b := newTestBrowser(t, withFileServer())
 
 	p := b.NewPage(nil)
@@ -90,8 +82,6 @@ func TestPageGotoWaitUntilDOMContentLoaded(t *testing.T) {
 }
 
 func TestPageSetExtraHTTPHeaders(t *testing.T) {
-	t.Parallel()
-
 	b := newTestBrowser(t, withHTTPServer())
 
 	p := b.NewPage(nil)
diff --git a/tests/page_title_test.go b/tests/page_title_test.go
@@ -28,8 +28,6 @@ import (
 )
 
 func TestPageTitle(t *testing.T) {
-	t.Parallel()
-
 	p := newTestBrowser(t).NewPage(nil)
 
 	p.SetContent(`<html><head><title>Some title</title></head></html>`, nil)

Original file line number	Diff line number	Diff line change
`@@ -29,8 +29,6 @@ import (`
`29`	`29`	`)`
`30`	`30`
`31`	`31`	`func TestElementHandleBoundingBoxInvisibleElement(t *testing.T) {`
`32`		`- t.Parallel()`
`33`		`-`
`34`	`32`	`p := newTestBrowser(t).NewPage(nil)`
`35`	`33`
`36`	`34`	p.SetContent(`<div style="display:none">hello</div>`, nil)
`@@ -40,8 +38,6 @@ func TestElementHandleBoundingBoxInvisibleElement(t *testing.T) {`
`40`	`38`	`}`
`41`	`39`
`42`	`40`	`func TestElementHandleBoundingBoxSVG(t *testing.T) {`
`43`		`- t.Parallel()`
`44`		`-`
`45`	`41`	`tb := newTestBrowser(t)`
`46`	`42`	`p := tb.NewPage(nil)`
`47`	`43`
Original file line number	Diff line number	Diff line change
`@@ -28,8 +28,6 @@ import (`
`28`	`28`	`)`
`29`	`29`
`30`	`30`	`func TestElementHandleGetAttribute(t *testing.T) {`
`31`		`- t.Parallel()`
`32`		`-`
`33`	`31`	`const want = "https://somewhere"`
`34`	`32`
`35`	`33`	`p := newTestBrowser(t).NewPage(nil)`