okTurtles / group-income

A decentralized and private (end-to-end encrypted) financial safety net for you and your friends.
https://groupincome.org
GNU Affero General Public License v3.0
331 stars 44 forks source link

#2342 - `bypassUI: true` option in `group-large.spec.js` and etc. #2377

Closed SebinSong closed 1 month ago

SebinSong commented 1 month ago

closes #2342

Currently group-large.spec.js test-suite has relatively high number of repeated action such as, 'A group with 12 members shows correctly the pledging month overview widget' etc. The goal of this PR is to reduce the duration of this test by implementing bypassUI: true options for those repeated actions, which are

cy.giAddRandomIncome() cy.giLogout()

This lead to below improvement when tested locally on my Mac OS.

[Tested in master]

before-update



[After update]

after-update



while at it, cy.giLogout({ bypassUI: true }) has been added in other *.spec.js files.

cypress[bot] commented 1 month ago

group-income    Run #3265

Run Properties:  status check passed Passed #3265  •  git commit 942cb66105 ℹ️: Merge 932db52c696d7b39468609fb77d1b62623c1d5b9 into b64ecda81d1b1c9574b1b4490945...
Project group-income
Run status status check passed Passed #3265
Run duration 08m 36s
Commit git commit 942cb66105 ℹ️: Merge 932db52c696d7b39468609fb77d1b62623c1d5b9 into b64ecda81d1b1c9574b1b4490945...
Committer Sebin Song
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 10
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 111
SebinSong commented 1 month ago

it appears that adding cy.giLoyout({ bypassUI: true }) to various test-suites leads to reducing the total test duration by around over 15 secs which is not bad.

[Before]

logout-before



[After]

logout-after


Another attempt: