Open Bug 1960964 Opened 7 months ago Updated 2 months ago

Create marionette test to verify data upload pref copying between profiles

Categories

(Toolkit :: Startup and Profile System, enhancement, P3)

enhancement

Tracking

()

ASSIGNED

People

(Reporter: jhirsch, Assigned: jhirsch)

References

(Depends on 2 open bugs, Blocks 3 open bugs)

Details

(Whiteboard: [fidefe-profile-management])

Attachments

(1 file, 1 obsolete file)

No description provided.
Assignee: nobody → tbersentes
Status: NEW → ASSIGNED
Attachment #9488070 - Attachment is obsolete: true
Assignee: tbersentes → jhirsch
Blocks: 1972055
No longer depends on: 1972055

An initial patch - looking for feedback on the general approach.

Rather than override existing DesktopInstance behavior, create a new
DesktopSelectableProfileInstance subclass of DesktopInstance that
overrides _update_profile to simply assign the passed-in profile to
self._profile, avoiding the existing clone-or-create logic.

Unfortunately, because we need to use a non-standard port for a second
session, and because this port needs to be hard-coded in the profile
before it is launched, we have to modify the Marionette client
constructor to allow a profile to be passed in directly. Using the
obviously wrong "specialprofile" argument for now, and we'll see what
the Marionette experts suggest instead.

Many things still TODO, including:

  • cleanup correctly:
    • remove the marionette profile from profiles.ini at end of test
    • remove the SQLite database file shared by the profile group
    • remove the additional profile from the user profiles directory
    • investigate mocking out directory service so the additional profiles
      and cross-profile SQLite database go in TmpD not UAppData
  • actually test the cross-profile pref passing behavior
  • add tests for marionette changes
Attachment #9495694 - Attachment description: Bug 1960964 - Initial WIP patch - Create multiple profiles and connect Marionette to two separate instances in the same test. r?#profiles-reviewers,whimboo → Bug 1960964 - Initial WIP patch - Create multiple profiles and connect Marionette to two separate instances in the same test. r?#profiles-reviewers
Attachment #9495694 - Attachment description: Bug 1960964 - Initial WIP patch - Create multiple profiles and connect Marionette to two separate instances in the same test. r?#profiles-reviewers → Bug 1960964 - Initial WIP patch - Create multiple profiles and connect Marionette to two separate instances in the same test. r?#profiles-reviewers,whimboo
Attachment #9495694 - Attachment description: Bug 1960964 - Initial WIP patch - Create multiple profiles and connect Marionette to two separate instances in the same test. r?#profiles-reviewers,whimboo → Bug 1960964 - Revised WIP patch - Create multiple profiles and connect Marionette to two separate instances in the same test. r?#profiles-reviewers,whimboo
Attachment #9495694 - Attachment description: Bug 1960964 - Revised WIP patch - Create multiple profiles and connect Marionette to two separate instances in the same test. r?#profiles-reviewers,whimboo → Bug 1960964 - Create multiple profiles and connect Marionette to two separate instances in the same test. r?#profiles-reviewers,whimboo
Severity: S2 → S3
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: