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
1diff --git a/DistUpgrade/deb2snap.json b/DistUpgrade/deb2snap.json
2index e79fe2d..6207ab5 100644
3--- a/DistUpgrade/deb2snap.json
4+++ b/DistUpgrade/deb2snap.json
5@@ -5,9 +5,16 @@
6 "to_channel": "stable",
7 "metapkg": "ubuntu-desktop"
8 },
9+ "firefox": {
10+ "deb": "firefox",
11+ "metapkg": "ubuntu-desktop"
12+ },
13 "gnome-3-34-1804": {
14 "metapkg": "ubuntu-desktop"
15 },
16+ "gnome-3-38-2004": {
17+ "metapkg": "ubuntu-desktop"
18+ },
19 "gtk-common-themes": {
20 "metapkg": "ubuntu-desktop"
21 },
22diff --git a/tests/test_quirks.py b/tests/test_quirks.py
23index 2e96087..751961c 100644
24--- a/tests/test_quirks.py
25+++ b/tests/test_quirks.py
26@@ -755,6 +755,10 @@ class TestSnapQuirks(unittest.TestCase):
27 make_mock_pkg(
28 name="core18",
29 is_installed=True),
30+ 'firefox':
31+ make_mock_pkg(
32+ name="firefox",
33+ is_installed=True),
34 'gnome-3-28-1804':
35 make_mock_pkg(
36 name="gnome-3-28-1804",
37@@ -795,10 +799,19 @@ class TestSnapQuirks(unittest.TestCase):
38 'channel': 'stable',
39 'command': 'install',
40 'deb': None, 'snap-id': '1234'},
41+ 'firefox': {
42+ 'channel': 'stable/ubuntu-19.10',
43+ 'command': 'install',
44+ 'deb': 'firefox',
45+ 'snap-id': '1234'},
46 'gnome-3-34-1804': {
47 'channel': 'stable/ubuntu-19.10',
48 'command': 'install',
49 'deb': None, 'snap-id': '1234'},
50+ 'gnome-3-38-2004': {
51+ 'channel': 'stable/ubuntu-19.10',
52+ 'command': 'install',
53+ 'deb': None, 'snap-id': '1234'},
54 'gnome-logs': {
55 'command': 'remove'},
56 'snap-store': {

Subscribers

People subscribed via source and target branches