public inbox for gentoo-portage-dev@lists.gentoo.org
 help / color / mirror / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download: 
* Re: [gentoo-portage-dev] [PATCH 4/6] tests: Fix running on installed copy of Portage
  @ 2014-08-22 22:52 99%     ` Michał Górny
  0 siblings, 0 replies; 1+ results
From: Michał Górny @ 2014-08-22 22:52 UTC (permalink / raw
  To: Brian Dolbec; +Cc: gentoo-portage-dev

[-- Attachment #1: Type: text/plain, Size: 1884 bytes --]

Dnia 2014-08-22, o godz. 15:19:12
Brian Dolbec <dolsen@gentoo.org> napisał(a):

> On Thu, 21 Aug 2014 22:19:42 +0200
> Michał Górny <mgorny@gentoo.org> wrote:
> 
> > ---
> >  pym/portage/tests/__init__.py                    | 20
> > ++++++++++++++++++ pym/portage/tests/dbapi/test_portdb_cache.py
> > |  5 ++--- pym/portage/tests/emerge/test_emerge_slot_abi.py |  7
> > +++---- pym/portage/tests/emerge/test_simple.py          | 26
> > ++++++++++++------------
> > pym/portage/tests/lint/test_compile_modules.py   | 10 +++++----
> > pym/portage/tests/lint/test_import_modules.py    |  8 ++++++--
> > pym/portage/tests/repoman/test_simple.py         |  8 +++++---
> > pym/portage/tests/resolver/ResolverPlayground.py |  8 +++++---
> > pym/portage/tests/util/test_getconfig.py         |  5 ++--- 9 files
> > changed, 62 insertions(+), 35 deletions(-)
> > 
>  
> diff --git a/pym/portage/tests/util/test_getconfig.py
> > b/pym/portage/tests/util/test_getconfig.py index e5fd60f..16f415c
> > 100644 --- a/pym/portage/tests/util/test_getconfig.py
> > +++ b/pym/portage/tests/util/test_getconfig.py
> > @@ -3,6 +3,7 @@
> >  
> >  import tempfile
> >  
> > +import portage
> >  from portage import os
> >  from portage import shutil
> >  from portage import _unicode_encode
> > @@ -26,9 +27,7 @@ class GetConfigTestCase(TestCase):
> >  	}
> >  
> >  	def testGetConfig(self):
> > -
> > -		make_globals_file = os.path.join(PORTAGE_BASE_PATH,
> > -			'cnf', 'make.globals')
> > +		make_globals_file = os.path.join(self.cnf_path,
> > "make.globals") d = getconfig(make_globals_file)
> >  		for k, v in self._cases.items():
> >  			self.assertEqual(d[k], v)
> 
> Why was "import portage" added but not used anywhere in additional code?

Good catch. It was used in the previous version of the patch. Fixed now.


-- 
Best regards,
Michał Górny

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 949 bytes --]

^ permalink raw reply	[relevance 99%]

Results 1-1 of 1 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2014-08-21 20:19     [gentoo-portage-dev] [PATCHES] setup.py install for Portage Michał Górny
2014-08-21 20:19     ` [gentoo-portage-dev] [PATCH 4/6] tests: Fix running on installed copy of Portage Michał Górny
2014-08-22 22:19       ` Brian Dolbec
2014-08-22 22:52 99%     ` Michał Górny

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox