Skip to content
This repository has been archived by the owner on Jan 30, 2024. It is now read-only.

Update benchmarks #184

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update benchmarks #184

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a541f8c
update input and button selectors
pgronkievitz Jun 28, 2023
05202d5
update room url to follow new format
pgronkievitz Jun 28, 2023
8b5ee04
update div ids and playwright selectors
pgronkievitz Jul 19, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions assets/src/features/home-page/components/VideoroomHomePage.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ const VideoroomHomePage: React.FC = () => {

<div className="flex w-full flex-col items-center justify-center gap-x-12 sm:max-h-[400px] sm:flex-row lg:gap-x-24">
{/* mobile view */}
<div className="flex w-full flex-col items-center gap-y-6 sm:hidden">
<div id="mobile_home_form" className="flex w-full flex-col items-center gap-y-6 sm:hidden">
{mobileLoginSteps[mobileCurrentLoginStep].content}
{mobileLoginSteps[mobileCurrentLoginStep].button}
{!joiningExistingRoom && (
Expand All @@ -190,7 +190,7 @@ const VideoroomHomePage: React.FC = () => {
<HomePageVideoTile displayName={displayNameInput} />
</div>

<div className="hidden w-auto flex-col items-center justify-center gap-y-6 sm:flex">
<div id="desktop_home_form" className="hidden w-auto flex-col items-center justify-center gap-y-6 sm:flex">
{inputs}
<div className="space-y-1">
{checkboxes.map(({ label, id, status, onChange }) => (
Expand Down
2 changes: 1 addition & 1 deletion benchmarks/multiroom.exs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ defmodule MultiroomScenario do
@peer_delay 1_000
# in miliseconds
@peer_duration 120_000
@url "http://localhost:4000/?room_id=benchmark"
@url "http://localhost:4000/room/benchmark"

@impl true
def run() do
Expand Down
2 changes: 1 addition & 1 deletion benchmarks/simple.exs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ defmodule SimpleScenario do
@peer_delay 1_000
# in miliseconds
@peer_duration 120_000
@room_url "http://localhost:4000/?room_id=benchmark"
@room_url "http://localhost:4000/room/benchmark"

# stampede or chrome cannot connect more than 16 peers using one instance of browser
# the problem is probably with granting permissions to mic/cam
Expand Down
4 changes: 2 additions & 2 deletions benchmarks/simple_mustang.ex
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ defmodule SimpleMustang do
page = browser |> Playwright.Browser.new_page()
_response = Playwright.Page.goto(page, options.target_url)

:ok = Playwright.Page.fill(page, "[name=display_name]", "stampede")
:ok = Playwright.Page.click(page, "[type=submit]")
:ok = Playwright.Page.fill(page, "#desktop_home_form [name=display_name]", "stampede")
:ok = Playwright.Page.click(page, "#join")

{browser, page}
end
Expand Down