Skip to content

Unit tests: from kunzimariano on versionone/VersionOne.SDK.Python #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mtalexan wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Unit tests: from kunzimariano on versionone/VersionOne.SDK.Python #11

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
7235325
modified API to use moments
kdahlhaus Oct 7, 2015
9020a8a
Merge pull request #29 from kdahlhaus/history_fix
danielgruesso Dec 15, 2015
e72a2c3
First unit tests for client.py
kunzimariano May 2, 2016
999c91b
Added test_build_url.
kunzimariano May 2, 2016
6feeb5c
Refactored V1Server by extracting some logic into the new HttpClient …
kunzimariano May 4, 2016
88cdc19
Fixed query since it was broken after the refactor.
kunzimariano May 5, 2016
054e630
Fixed a bug in query.py
kunzimariano May 6, 2016
3c3e44d
Added TestV1Server. Also grouped V1Server and HttpClient tests under …
kunzimariano May 6, 2016
870f929
Added tests for V1Server.get_attr
kunzimariano May 6, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions setup.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
'testtools',
'iso8601',
'python-ntlm',
'mock'
],

test_suite = "v1pysdk.tests",
Expand Down
16 changes: 10 additions & 6 deletions v1pysdk/base_asset.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,14 +54,15 @@ def __iter__(Class):
"that knows how to be iterated over, so we can say list(v1.Story)"
__metaclass__ = IterableType

def __new__(Class, oid):
def __new__(Class, oid, moment=None):
"Tries to get an instance out of the cache first, otherwise creates one"
cache_key = (Class._v1_asset_type_name, int(oid))
cache_key = (Class._v1_asset_type_name, oid, moment)
cache = Class._v1_v1meta.global_cache
if cache.has_key(cache_key):
self = cache[cache_key]
else:
self = object.__new__(Class)
self = object.__new__(Class)
self._v1_moment = moment
self._v1_oid = oid
self._v1_new_data = {}
self._v1_current_data = {}
Expand All @@ -86,7 +87,10 @@ def idref(self):

@property
def reprref(self):
return "{0}({1})".format(self._v1_asset_type_name, self._v1_oid)
if self._v1_moment:
return "{0}({1}:{2})".format(self._v1_asset_type_name, self._v1_oid, self._v1_moment)
else:
return "{0}({1})".format(self._v1_asset_type_name, self._v1_oid)

@property
def url(self):
Expand Down Expand Up @@ -176,11 +180,11 @@ def _v1_commit(self):

def _v1_refresh(self):
'Syncs the objects from current server data'
self._v1_current_data = self._v1_v1meta.read_asset(self._v1_asset_type_name, self._v1_oid)
self._v1_current_data = self._v1_v1meta.read_asset(self._v1_asset_type_name, self._v1_oid, self._v1_moment)
self._v1_needs_refresh = False

def _v1_get_single_attr(self, attr):
return self._v1_v1meta.get_attr(self._v1_asset_type_name, self._v1_oid, attr)
return self._v1_v1meta.get_attr(self._v1_asset_type_name, self._v1_oid, attr, self._v1_moment)

def _v1_execute_operation(self, opname):
result = self._v1_v1meta.execute_operation(self._v1_asset_type_name, self._v1_oid, opname)
Expand Down
Loading