feat(gather): introduce sync gather for easier async tests migration

[pavelfeldman]

No description provided.

[coveralls]

Pull Request Test Coverage Report for Build 192649967

• 24 of 25 (96.0%) changed or added relevant lines in 1 file are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage increased (+0.07%) to 88.642%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
playwright/sync_base.py	24	25	96.0%

Files with Coverage Reduction	New Missed Lines	%
playwright/async_api.py	1	90.76%

Totals
Change from base Build 192377433:	0.07%
Covered Lines:	3395
Relevant Lines:	3830

---

💛 - Coveralls

[mxschmitt]

Suggested change

	new_page._gather([
	lambda: new_page.evaluate("() => new Promise(r => {})"),
	lambda: new_page.close()
	])
	new_page._gather(
	lambda: new_page.evaluate("() => new Promise(r => {})"),
	lambda: new_page.close()
	)

[mxschmitt]

Suggested change

	def _gather(self, actions: List[Callable]) -> List[Any]:
	def _gather(self, *actions: List[Callable]) -> List[Any]:

To align more with the default asyncio.gather I would not pass a list and more go like with the additional args approach (like arguments keyword in JS). (If it does not work, try renaming *actions to *args).

[pavelfeldman]

I started like that, but then I chose to make it return array of values and input array felt more natural.

[pavelfeldman]

I started like that, but then I chose to make it return array of values and imput array felt more natural.

Having said that, if we manage to generate tests - happy to do it that way.