Merge ~osomon/ubuntu-release-upgrader:firefox-snap into ubuntu-release-upgrader:ubuntu/master

Proposed by Olivier Tilloy
Status: Merged
Merged at revision: a81a61845bcd311ac8f52d1b0c957b048b88d082
Proposed branch: ~osomon/ubuntu-release-upgrader:firefox-snap
Merge into: ubuntu-release-upgrader:ubuntu/master
Diff against target: 56 lines (+20/-0)
2 files modified
DistUpgrade/deb2snap.json (+7/-0)
tests/test_quirks.py (+13/-0)
Reviewer Review Type Date Requested Status
Ken VanDine Approve
Review via email: mp+408838@code.launchpad.net

Commit message

Add the firefox snap to the list of deb2snap transitions, along with gnome-3-38-2004, on which it is built (LP: #1943840).

To post a comment you must log in.
Revision history for this message
Ken VanDine (ken-vandine) wrote :

I think 19.10 should be 21.10?

review: Needs Fixing
Revision history for this message
Olivier Tilloy (osomon) wrote :

No, that's just test data, the corresponding test case starts like this:

    def test_prepare_snap_replacement_data(self):
        # Prepare the state for testing
        controller = mock.Mock()
        config = mock.Mock()
        q = DistUpgradeQuirks(controller, config)
        q._from_version = "19.04"
        q._to_version = "19.10"

Revision history for this message
Ken VanDine (ken-vandine) wrote :

Looks good to me

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/DistUpgrade/deb2snap.json b/DistUpgrade/deb2snap.json
index e79fe2d..6207ab5 100644
--- a/DistUpgrade/deb2snap.json
+++ b/DistUpgrade/deb2snap.json
@@ -5,9 +5,16 @@
5 "to_channel": "stable",5 "to_channel": "stable",
6 "metapkg": "ubuntu-desktop"6 "metapkg": "ubuntu-desktop"
7 },7 },
8 "firefox": {
9 "deb": "firefox",
10 "metapkg": "ubuntu-desktop"
11 },
8 "gnome-3-34-1804": {12 "gnome-3-34-1804": {
9 "metapkg": "ubuntu-desktop"13 "metapkg": "ubuntu-desktop"
10 },14 },
15 "gnome-3-38-2004": {
16 "metapkg": "ubuntu-desktop"
17 },
11 "gtk-common-themes": {18 "gtk-common-themes": {
12 "metapkg": "ubuntu-desktop"19 "metapkg": "ubuntu-desktop"
13 },20 },
diff --git a/tests/test_quirks.py b/tests/test_quirks.py
index 2e96087..751961c 100644
--- a/tests/test_quirks.py
+++ b/tests/test_quirks.py
@@ -755,6 +755,10 @@ class TestSnapQuirks(unittest.TestCase):
755 make_mock_pkg(755 make_mock_pkg(
756 name="core18",756 name="core18",
757 is_installed=True),757 is_installed=True),
758 'firefox':
759 make_mock_pkg(
760 name="firefox",
761 is_installed=True),
758 'gnome-3-28-1804':762 'gnome-3-28-1804':
759 make_mock_pkg(763 make_mock_pkg(
760 name="gnome-3-28-1804",764 name="gnome-3-28-1804",
@@ -795,10 +799,19 @@ class TestSnapQuirks(unittest.TestCase):
795 'channel': 'stable',799 'channel': 'stable',
796 'command': 'install',800 'command': 'install',
797 'deb': None, 'snap-id': '1234'},801 'deb': None, 'snap-id': '1234'},
802 'firefox': {
803 'channel': 'stable/ubuntu-19.10',
804 'command': 'install',
805 'deb': 'firefox',
806 'snap-id': '1234'},
798 'gnome-3-34-1804': {807 'gnome-3-34-1804': {
799 'channel': 'stable/ubuntu-19.10',808 'channel': 'stable/ubuntu-19.10',
800 'command': 'install',809 'command': 'install',
801 'deb': None, 'snap-id': '1234'},810 'deb': None, 'snap-id': '1234'},
811 'gnome-3-38-2004': {
812 'channel': 'stable/ubuntu-19.10',
813 'command': 'install',
814 'deb': None, 'snap-id': '1234'},
802 'gnome-logs': {815 'gnome-logs': {
803 'command': 'remove'},816 'command': 'remove'},
804 'snap-store': {817 'snap-store': {

Subscribers

People subscribed via source and target branches