Open Bug 1283576 Opened 8 years ago Updated 2 years ago

Add a test helper method to add multiple logins in the same transaction

Categories

(Firefox for iOS :: Build & Test, defect)

Other
iOS
defect

Tracking

()

People

(Reporter: nalcock, Unassigned)

Details

(Keywords: good-first-bug)

Some of the tests in `TestLogins` would benefit from a method to add multiple logins with a single database commit, especially `testRemoveManyLogins`, to improve the overall suite speed.

Is this bug still open? Can I work on it?

Can i work on this bug??

is it still open?

Flags: needinfo?(fpatel)

Hi Shuban, for the UI or XCUITest we would not need that. In the first case we create several logins while running the tests and in the second one we create a real login to test the ui completely.

But it may be a different case for the unit tests, ni Garvan in case he thinks that tool is interesting there.

Thank you for showing your interest in this!

Flags: needinfo?(fpatel) → needinfo?(gkeeley)

Removing the ni since this contributor's comment looks like was for firefox, not firefox-iOS...(emailed to say that)

Flags: needinfo?(gkeeley)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.