Issue 4574 - Adds Tests to createNightlies platform specific revisions

sitescripts/extensions/test/test_createNightlies.py

 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-2016 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
 #
 # You should have received a copy of the GNU General Public License
 # along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
+
 import os
 from subprocess import CalledProcessError
 
 import pytest
 
 from sitescripts.extensions.bin import createNightlies
 from sitescripts.utils import get_config
 
 
 @pytest.fixture(scope='session')
@@ skipping 15 matching lines @@
 @pytest.fixture(scope='session')
 def nightlybuild(config):
     return createNightlies.NightlyBuild(config)
 
 
 def test_nightly_object_bookmark(nightlybuild):
     assert nightlybuild.config.revision == 'safari'
 
 
 def test_current_revision(nightlybuild):
-    # This inline hash is for the expected revision of the safari bookmark rev

[Vasily Kuznetsov, 2016/11/21 12:21:45]

I think this comment could be improved a bit. We see from the code that the
magic string is the expected revision hash. What is not so obvious is why we
expect this value and where it's coming from.

So maybe something like: "Revision hash of 'safari' bookmark (see
adblockplusnightly.bookmarks)."

What do you think?

[Jon Sonesen, 2016/11/21 17:00:07]

Done.

+    # The hash is the commit that the safari bookmark points to.
+    # (see adblockplusnightly.bookmark in test/data/bookmarks)
     assert nightlybuild.revision == '1291590ddd0f'
 
 
 def test_copy_repository(nightlybuild, nightlydir):
     nightlybuild.copyRepository()
     assert os.listdir(nightlybuild.tempdir) == ['.hg', 'README.txt']
 
 
 def test_get_changes(nightlybuild, nightlydir):
     # The bookmark 'safari' contains only 2 revisions
     # default contains 51 so here we ensure that erroneous changes
     # are not returned
     for change in nightlybuild.getChanges():
         assert change['revision'] < '2'
 
     nightlybuild.config.revision = 'default'
     for change in nightlybuild.getChanges():
         assert change['revision'] > '1'
 
 
 def test_missing_bookmark(config):
     config.revision = 'foo'
     config.type = 'type'
     try:
         createNightlies.NightlyBuild(config)
     except CalledProcessError as e:
         assert e.returncode == 255