Small tweak to how extArgs is handled, removed getCollection from unit tests because it is binding specific, fixed references in browser binding logger
git-svn-id: https://svn.apache.org/repos/asf/chemistry/cmislib/trunk@1592412 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/cmislib/browser/binding.py b/src/cmislib/browser/binding.py
index 9fa215b..7168e48 100644
--- a/src/cmislib/browser/binding.py
+++ b/src/cmislib/browser/binding.py
@@ -142,8 +142,8 @@
self._properties = {}
self._allowableActions = {}
self.data = data
- self._kwargs = kwargs
- self.logger = logging.getLogger('cmislib.browser_binding.BrowserCmisObject')
+ self._extArgs = kwargs
+ self.logger = logging.getLogger('cmislib.browser.BrowserCmisObject')
self.logger.info('Creating an instance of CmisObject')
def __str__(self):
@@ -173,11 +173,8 @@
'*'.
"""
- if kwargs:
- if self._kwargs:
- self._kwargs.update(kwargs)
- else:
- self._kwargs = kwargs
+ if self._extArgs:
+ kwargs.update(self._extArgs)
byObjectIdUrl = self._repository.getRootFolderUrl() + "?objectId=" + self.getObjectId() + "&cmisselector=object"
self.data = self._cmisClient.binding.get(byObjectIdUrl.encode('utf-8'),
@@ -520,7 +517,7 @@
self._permMap = {}
self._permissions = None
self._propagation = None
- self.logger = logging.getLogger('cmislib.browser_binding.BrowserRepository')
+ self.logger = logging.getLogger('cmislib.browser.BrowserRepository')
self.logger.info('Creating an instance of Repository')
def __str__(self):
@@ -648,10 +645,10 @@
"""
if kwargs:
- if self._kwargs:
- self._kwargs.update(kwargs)
+ if self._extArgs:
+ self._extArgs.update(kwargs)
else:
- self._kwargs = kwargs
+ self._extArgs = kwargs
#TODO why is quoting the path required for the browser binding and not for atom pub
#on inmemory 0.9?
@@ -1227,8 +1224,8 @@
propCount = 2
for prop in properties:
- props["propertyId[%s]" % propCount] = prop.key
- props["propertyValue[%s]" % propCount] = prop
+ props["propertyId[%s]" % propCount] = prop
+ props["propertyValue[%s]" % propCount] = properties[prop]
propCount += 1
contentType, body = encode_multipart_formdata(props, contentFile, contentType)
@@ -1332,7 +1329,7 @@
self._repository = repository
self._data = data
self._results = []
- self.logger = logging.getLogger('cmislib.model.browser_binding.BrowserResultSet')
+ self.logger = logging.getLogger('cmislib.browser.BrowserResultSet')
self.logger.info('Creating an instance of ResultSet')
def __iter__(self):
@@ -2266,10 +2263,10 @@
""" Constructor """
self._cmisClient = cmisClient
self._repository = repository
- self._kwargs = None
+ self._extArgs = None
self._typeId = typeId
self.data = data
- self.logger = logging.getLogger('cmislib.browser_binding.BrowserObjectType')
+ self.logger = logging.getLogger('cmislib.browser.BrowserObjectType')
self.logger.info('Creating an instance of BrowserObjectType')
def __str__(self):
@@ -2412,10 +2409,8 @@
This method will reload the object's data from the CMIS service.
"""
if kwargs:
- if self._kwargs:
- self._kwargs.update(kwargs)
- else:
- self._kwargs = kwargs
+ if self._extArgs:
+ kwargs.update(self._extArgs)
typesUrl = self._repository.getRepositoryUrl()
kwargs['cmisselector'] = 'typeDefinition'
@@ -2453,7 +2448,7 @@
def __init__(self, data):
"""Constructor"""
self.data = data
- self.logger = logging.getLogger('cmislib.browser_binding.BrowserProperty')
+ self.logger = logging.getLogger('cmislib.browser.BrowserProperty')
self.logger.info('Creating an instance of BrowserProperty')
def __str__(self):
@@ -2556,7 +2551,7 @@
else:
self._data = None
- self.logger = logging.getLogger('cmislib.browser_binding.BrowserACL')
+ self.logger = logging.getLogger('cmislib.browser.BrowserACL')
self.logger.info('Creating an instance of ACL')
def _getEntriesFromData(self):
@@ -2795,7 +2790,7 @@
def __init__(self, propNode):
"""Constructor"""
self.xmlDoc = propNode
- self.logger = logging.getLogger('cmislib.browser_binding.BrowserRendition')
+ self.logger = logging.getLogger('cmislib.browser.BrowserRendition')
self.logger.info('Creating an instance of Rendition')
def __str__(self):
diff --git a/src/tests/cmislibtest.py b/src/tests/cmislibtest.py
index 43e9107..19f69ce 100644
--- a/src/tests/cmislibtest.py
+++ b/src/tests/cmislibtest.py
@@ -816,8 +816,6 @@
self.assertTrue(newDoc.isCheckedOut())
self.assert_('cmis:objectId' in newDoc.getProperties())
self.assert_('cmis:objectId' in pwcDoc.getProperties())
- checkedOutDocs = self._repo.getCollection('checkedout')
- self.assertTrue(isInResultSet(checkedOutDocs, pwcDoc))
finally:
pwcDoc.delete()
@@ -836,8 +834,6 @@
self.assertTrue(newDoc.isCheckedOut())
self.assert_('cmis:objectId' in newDoc.getProperties())
self.assert_('cmis:objectId' in pwcDoc.getProperties())
- checkedOutDocs = self._repo.getCollection('checkedout')
- self.assertTrue(isInResultSet(checkedOutDocs, pwcDoc))
finally:
pwcDoc.delete()
@@ -929,13 +925,9 @@
self.assertTrue(newDoc.isCheckedOut())
self.assert_('cmis:objectId' in newDoc.getProperties())
self.assert_('cmis:objectId' in pwcDoc.getProperties())
- checkedOutDocs = self._repo.getCollection('checkedout')
- self.assertTrue(isInResultSet(checkedOutDocs, pwcDoc))
finally:
pwcDoc.delete()
self.assertFalse(newDoc.isCheckedOut())
- checkedOutDocs = self._repo.getCollection('checkedout')
- self.assertFalse(isInResultSet(checkedOutDocs, pwcDoc))
def testDeleteDocument(self):
"""Create a document in a test folder, then delete it"""
diff --git a/src/tests/settings.py b/src/tests/settings.py
index 8c50f19..fd631d2 100644
--- a/src/tests/settings.py
+++ b/src/tests/settings.py
@@ -24,16 +24,16 @@
#
# CMIS repository's service URL
#REPOSITORY_URL = 'http://cmis.alfresco.com/s/cmis' # Alfresco demo
-REPOSITORY_URL = 'http://localhost:8081/chemistry/atom' # Apache Chemistry AtomPub
-#REPOSITORY_URL = 'http://localhost:8081/chemistry/browser' # Apache Chemistry Browser
+#REPOSITORY_URL = 'http://localhost:8081/chemistry/atom' # Apache Chemistry AtomPub
+REPOSITORY_URL = 'http://localhost:8081/chemistry/browser' # Apache Chemistry Browser
#REPOSITORY_URL = 'http://localhost:8080/alfresco/cmisatom' # Alfresco 4.0 AtomPub
#REPOSITORY_URL = 'http://localhost:8080/alfresco/s/api/cmis' # Alfresco
#REPOSITORY_URL = 'http://cmis.demo.nuxeo.org/nuxeo/atom/cmis' # Nuxeo demo
#REPOSITORY_URL = 'http://localhost:8080/nuxeo/atom/cmis' # Nuxeo local
# Choose a binding. The AtomPubBinding is the only one you should really be using right now
-BINDING = AtomPubBinding()
-#BINDING = BrowserBinding()
+#BINDING = AtomPubBinding()
+BINDING = BrowserBinding()
# CMIS repository credentials
USERNAME = 'admin' # Alfresco