diff --git a/validations_libs/group.py b/validations_libs/group.py index 89daf17d..c3c1f615 100644 --- a/validations_libs/group.py +++ b/validations_libs/group.py @@ -42,4 +42,4 @@ class Group(object): @property def get_groups_keys_list(self): - return [gp for gp in self.data.keys()] + return [gp for gp in sorted(self.data.keys())] diff --git a/validations_libs/tests/test_group.py b/validations_libs/tests/test_group.py index dac8f904..ae36c006 100644 --- a/validations_libs/tests/test_group.py +++ b/validations_libs/tests/test_group.py @@ -47,7 +47,7 @@ class TestGroup(TestCase): @mock.patch('six.moves.builtins.open') def test_get_groups_keys_list(self, mock_open, mock_yaml): grp = Group('/tmp/foo') - ret = ['no-op', 'pre', 'post'] + ret = ['no-op', 'post', 'pre'] data = grp.get_groups_keys_list self.assertEquals(data, ret) diff --git a/validations_libs/tests/test_utils.py b/validations_libs/tests/test_utils.py index df997f20..e8e00677 100644 --- a/validations_libs/tests/test_utils.py +++ b/validations_libs/tests/test_utils.py @@ -118,7 +118,7 @@ class TestUtils(TestCase): def test_get_validation_group_name_list(self, mock_open, mock_load): result = utils.get_validation_group_name_list('/foo/groups.yaml') - self.assertEqual(result, ['no-op', 'pre', 'post']) + self.assertEqual(result, ['no-op', 'post', 'pre']) @mock.patch('validations_libs.utils.parse_all_validations_on_disk', return_value=[fakes.FAKE_METADATA])