#791 Implement version.compose_id version generator
Merged 6 years ago by ausil. Opened 6 years ago by puiterwijk.
puiterwijk/pungi release-version-compose-id  into  master

file modified
+5 -2
@@ -782,9 +782,12 @@ 

      """

      if gen == '!OSTREE_VERSION_FROM_LABEL_DATE_TYPE_RESPIN':

          return '%s.%s' % (compose.image_version, compose.image_release)

-     if gen == '!RELEASE_FROM_LABEL_DATE_TYPE_RESPIN':

+     elif gen == '!RELEASE_FROM_LABEL_DATE_TYPE_RESPIN':

          return compose.image_release

-     if gen and gen[0] == '!':

+     elif gen == '!RELEASE_FROM_VERSION_COMPOSE_ID':

+         return '%s.%s' % (compose.ci_base.release.version,

+                           compose.ci_base.id)

+     elif gen and gen[0] == '!':

          raise RuntimeError("Unknown version generator '%s'" % gen)

      return gen

  

file modified
+18
@@ -601,6 +601,20 @@ 

  

  

  class TestVersionGenerator(unittest.TestCase):

+     def setUp(self):

+         ci = mock.MagicMock()

+         ci.respin = 0

+         ci.id = 'RHEL-8.0-20180101.0'

+         ci.release.version = '8'

+         ci.date = '20160101'

+         ci.type = 'nightly'

+         ci.type_suffix = ''

+         ci.label = 'RC-1.0'

+         ci.label_major_version = '1'

+ 

+         self.compose = mock.MagicMock()

+         self.compose.ci_base = ci

+ 

      def test_unknown_generator(self):

          compose = mock.Mock()

          with self.assertRaises(RuntimeError) as ctx:
@@ -617,6 +631,10 @@ 

          compose = mock.Mock()

          self.assertEqual(util.version_generator(compose, None), None)

  

+     def test_release_from_version_compose_id(self):

+         self.assertEqual(util.version_generator(self.compose, '!RELEASE_FROM_VERSION_COMPOSE_ID'),

+                          '8.RHEL-8.0-20180101.0')

+ 

  

  class TestTZOffset(unittest.TestCase):

      @mock.patch('time.daylight', new=False)

Implemented for version goal as pointed out in https://pagure.io/releng/issue/7113

Pull-Request has been merged by ausil

6 years ago