Skip to content
This repository has been archived by the owner on Dec 7, 2022. It is now read-only.

import unittest2 in compat module #1963

Merged
merged 1 commit into from Jul 7, 2015
Merged

Conversation

asmacdo
Copy link
Contributor

@asmacdo asmacdo commented Jun 30, 2015

In order to use new things like self.assertDictEqual, we are increasingly using unittest2 in older versions of python. By moving this import to the compat module, we can avoid doing this same import magic in every test module.

@asmacdo asmacdo changed the title import unittest2 in compat for older python import unittest2 in compat module Jun 30, 2015
@pulpbot
Copy link
Member

pulpbot commented Jun 30, 2015

Refer to this link for build results (access rights to CI server needed):
https://pulp-jenkins.rhev-ci-vms.eng.rdu2.redhat.com//job/unittest-pulp-pr/1031/
Test PASSed.

@ipanova ipanova self-assigned this Jul 1, 2015
@check_builtin(pkgutil)
def iter_modules(*args, **kwargs):
return backports.pkgutil.iter_modules(*args, **kwargs)


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe write a comment like previous imports have? this provides unittest

@ipanova ipanova assigned asmacdo and unassigned ipanova Jul 1, 2015
@ipanova ipanova added the LGTM label Jul 1, 2015
asmacdo added a commit that referenced this pull request Jul 7, 2015
import unittest2 in compat module
@asmacdo asmacdo merged commit 09f2ddd into pulp:master Jul 7, 2015
@asmacdo asmacdo deleted the unittest-to-compat branch December 23, 2015 20:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants