Hi all.

Applied pPOP8 requirements to the code, removed unused imports. Please, disregard the first version of this patch


On 08/10/2015 08:20 AM, Oleg Fayans wrote:
Hi list,

Here are 2 integration tests for topology plugin. Plus I got rid of
using nose in test_ipaserver/test_topology_plugin.py: I am using
@pytest.mark.skipif instead




--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From c906784ded416eec70704a07e3923601fe509927 Mon Sep 17 00:00:00 2001
From: Tomas Babej <tba...@redhat.com>
Date: Fri, 7 Aug 2015 18:03:48 +0200
Subject: [PATCH] dcerpc: Simplify generation of LSA-RPC binding strings

https://fedorahosted.org/freeipa/ticket/5183

Reviewed-By: Tomas Babej <tba...@redhat.com>
---
 ipaserver/dcerpc.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/ipaserver/dcerpc.py b/ipaserver/dcerpc.py
index 5f04d95cfd9bff9a066f77bf5e641ce5383b87ec..5428d0e38626cd01fb52d765f55144e84edddbf5 100644
--- a/ipaserver/dcerpc.py
+++ b/ipaserver/dcerpc.py
@@ -865,8 +865,7 @@ class TrustDomainInstance(object):
         """
         transports = (u'ncacn_np', u'ncacn_ip_tcp')
         options = ( u'smb2,print', u'print')
-        binding_template=lambda x,y,z: u'%s:%s[%s]' % (x, y, z)
-        return [binding_template(t, remote_host, o) for t in transports for o in options]
+        return [u'%s:%s[%s]' % (t, remote_host, o) for t in transports for o in options]
 
     def retrieve_anonymously(self, remote_host, discover_srv=False, search_pdc=False):
         """
-- 
2.4.3

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to