Skip to content

Commit 6004a1d

Browse files
kblokkblok
committed
prettier
1 parent 288320b commit 6004a1d

2 files changed

Lines changed: 74 additions & 75 deletions

File tree

‎lib/PuppeteerSharp.Tests/ConnectTests/ConnectTests.cs‎

Lines changed: 52 additions & 53 deletions
Original file line numberDiff line numberDiff line change
@@ -2,61 +2,60 @@
22
using NUnit.Framework;
33
using PuppeteerSharp.Nunit;
44

5-
namespace PuppeteerSharp.Tests.ConnectTests
5+
namespace PuppeteerSharp.Tests.ConnectTests;
6+
7+
public class ConnectTests : PuppeteerPageBaseTest
68
{
7-
public class ConnectTests : PuppeteerPageBaseTest
9+
[Test, PuppeteerTest("connect.spec", "Puppeteer.connect", "should be able to connect using browserUrl, with and without trailing slash")]
10+
public async Task ShouldBeAbleToConnectUsingBrowserURLWithAndWithoutTrailingSlash()
811
{
9-
[Test, PuppeteerTest("connect.spec", "Puppeteer.connect", "should be able to connect using browserUrl, with and without trailing slash")]
10-
public async Task ShouldBeAbleToConnectUsingBrowserURLWithAndWithoutTrailingSlash()
11-
{
12-
var options = TestConstants.DefaultBrowserOptions();
13-
options.Args = ["--remote-debugging-port=21222"];
14-
var originalBrowser = await Puppeteer.LaunchAsync(options);
15-
var browserURL = "http://127.0.0.1:21222";
16-
17-
var browser1 = await Puppeteer.ConnectAsync(new ConnectOptions { BrowserURL = browserURL });
18-
var page1 = await browser1.NewPageAsync();
19-
Assert.That(await page1.EvaluateExpressionAsync<int>("7 * 8"), Is.EqualTo(56));
20-
browser1.Disconnect();
21-
22-
var browser2 = await Puppeteer.ConnectAsync(new ConnectOptions { BrowserURL = browserURL + "/" });
23-
var page2 = await browser2.NewPageAsync();
24-
Assert.That(await page2.EvaluateExpressionAsync<int>("7 * 8"), Is.EqualTo(56));
25-
browser2.Disconnect();
26-
await originalBrowser.CloseAsync();
27-
}
28-
29-
[Test, PuppeteerTest("connect.spec", "Puppeteer.connect", "should throw when using both browserWSEndpoint and browserURL")]
30-
public async Task ShouldThrowWhenUsingBothBrowserWSEndpointAndBrowserURL()
12+
var options = TestConstants.DefaultBrowserOptions();
13+
options.Args = ["--remote-debugging-port=21222"];
14+
var originalBrowser = await Puppeteer.LaunchAsync(options);
15+
var browserURL = "http://127.0.0.1:21222";
16+
17+
var browser1 = await Puppeteer.ConnectAsync(new ConnectOptions { BrowserURL = browserURL });
18+
var page1 = await browser1.NewPageAsync();
19+
Assert.That(await page1.EvaluateExpressionAsync<int>("7 * 8"), Is.EqualTo(56));
20+
browser1.Disconnect();
21+
22+
var browser2 = await Puppeteer.ConnectAsync(new ConnectOptions { BrowserURL = browserURL + "/" });
23+
var page2 = await browser2.NewPageAsync();
24+
Assert.That(await page2.EvaluateExpressionAsync<int>("7 * 8"), Is.EqualTo(56));
25+
browser2.Disconnect();
26+
await originalBrowser.CloseAsync();
27+
}
28+
29+
[Test, PuppeteerTest("connect.spec", "Puppeteer.connect", "should throw when using both browserWSEndpoint and browserURL")]
30+
public async Task ShouldThrowWhenUsingBothBrowserWSEndpointAndBrowserURL()
31+
{
32+
var options = TestConstants.DefaultBrowserOptions();
33+
options.Args = ["--remote-debugging-port=21222"];
34+
var originalBrowser = await Puppeteer.LaunchAsync(options);
35+
var browserURL = "http://127.0.0.1:21222";
36+
37+
Assert.ThrowsAsync<PuppeteerException>(() => Puppeteer.ConnectAsync(new ConnectOptions
3138
{
32-
var options = TestConstants.DefaultBrowserOptions();
33-
options.Args = ["--remote-debugging-port=21222"];
34-
var originalBrowser = await Puppeteer.LaunchAsync(options);
35-
var browserURL = "http://127.0.0.1:21222";
36-
37-
Assert.ThrowsAsync<PuppeteerException>(() => Puppeteer.ConnectAsync(new ConnectOptions
38-
{
39-
BrowserURL = browserURL,
40-
BrowserWSEndpoint = originalBrowser.WebSocketEndpoint
41-
}));
42-
43-
await originalBrowser.CloseAsync();
44-
}
45-
46-
[Test, PuppeteerTest("connect.spec", "Puppeteer.connect", "should throw when trying to connect to non-existing browser")]
47-
public async Task ShouldThrowWhenTryingToConnectToNonExistingBrowser()
39+
BrowserURL = browserURL,
40+
BrowserWSEndpoint = originalBrowser.WebSocketEndpoint
41+
}));
42+
43+
await originalBrowser.CloseAsync();
44+
}
45+
46+
[Test, PuppeteerTest("connect.spec", "Puppeteer.connect", "should throw when trying to connect to non-existing browser")]
47+
public async Task ShouldThrowWhenTryingToConnectToNonExistingBrowser()
48+
{
49+
var options = TestConstants.DefaultBrowserOptions();
50+
options.Args = ["--remote-debugging-port=21222"];
51+
var originalBrowser = await Puppeteer.LaunchAsync(options);
52+
var browserURL = "http://127.0.0.1:2122";
53+
54+
Assert.ThrowsAsync<ProcessException>(() => Puppeteer.ConnectAsync(new ConnectOptions
4855
{
49-
var options = TestConstants.DefaultBrowserOptions();
50-
options.Args = ["--remote-debugging-port=21222"];
51-
var originalBrowser = await Puppeteer.LaunchAsync(options);
52-
var browserURL = "http://127.0.0.1:2122";
53-
54-
Assert.ThrowsAsync<ProcessException>(() => Puppeteer.ConnectAsync(new ConnectOptions
55-
{
56-
BrowserURL = browserURL
57-
}));
58-
59-
await originalBrowser.CloseAsync();
60-
}
56+
BrowserURL = browserURL
57+
}));
58+
59+
await originalBrowser.CloseAsync();
6160
}
62-
}
61+
}

‎lib/PuppeteerSharp.Tests/InterventionHeadersTests/InterventionHeadersTests.cs‎

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -3,33 +3,33 @@
33
using NUnit.Framework;
44
using PuppeteerSharp.Nunit;
55

6-
namespace PuppeteerSharp.Tests.InterventionHeadersTests
6+
namespace PuppeteerSharp.Tests.InterventionHeadersTests;
7+
8+
public class InterventionHeadersTests : PuppeteerPageBaseTest
79
{
8-
public class InterventionHeadersTests : PuppeteerPageBaseTest
10+
[Test, PuppeteerTest("interventionHeaders.spec", "Page.setRequestInterception", "should work with intervention headers")]
11+
public async Task ShouldWorkWithInterventionHeaders()
912
{
10-
[Test, PuppeteerTest("interventionHeaders.spec", "Page.setRequestInterception", "should work with intervention headers")]
11-
public async Task ShouldWorkWithInterventionHeaders()
12-
{
13-
Server.SetRoute("/intervention", context => context.Response.WriteAsync($@"
14-
<script>
15-
document.write('<script src=""{TestConstants.CrossProcessHttpPrefix}/intervention.js"">' + '</scr' + 'ipt>');
16-
</script>
17-
"));
18-
Server.SetRedirect("/intervention.js", "/redirect.js");
13+
Server.SetRoute("/intervention", context => context.Response.WriteAsync($@"
14+
<script>
15+
document.write('<script src=""{TestConstants.CrossProcessHttpPrefix}/intervention.js"">' + '</scr' + 'ipt>');
16+
</script>
17+
"));
18+
Server.SetRedirect("/intervention.js", "/redirect.js");
1919

20-
string interventionHeader = null;
21-
Server.SetRoute("/redirect.js", context =>
22-
{
23-
interventionHeader = context.Request.Headers["intervention"];
24-
return context.Response.WriteAsync("console.log(1);");
25-
});
20+
string interventionHeader = null;
21+
Server.SetRoute("/redirect.js", context =>
22+
{
23+
interventionHeader = context.Request.Headers["intervention"];
24+
return context.Response.WriteAsync("console.log(1);");
25+
});
2626

27-
await Page.SetRequestInterceptionAsync(true);
28-
Page.Request += async (_, e) => await e.Request.ContinueAsync();
27+
await Page.SetRequestInterceptionAsync(true);
28+
Page.Request += async (_, e) => await e.Request.ContinueAsync();
2929

30-
await Page.GoToAsync(TestConstants.ServerUrl + "/intervention");
30+
await Page.GoToAsync(TestConstants.ServerUrl + "/intervention");
3131

32-
Assert.That(interventionHeader, Does.Contain("feature/5718547946799104"));
33-
}
32+
Assert.That(interventionHeader, Does.Contain("feature/5718547946799104"));
3433
}
3534
}
35+

0 commit comments

Comments
 (0)