changeset ace1409f4cab in /home/hg/repos/gajim-plugins
author: Daniel Gultsch <daniel(a)gultsch.de>
branches:
details:gajim-plugins?cmd=changeset;node=ace1409f4cab
description: renamed add_devices and add_own_devices to set_devices and set_own_devices
diffstat:
omemo/__init__.py | 4 ++--
omemo/lib/python-omemo/src/omemo/state.py | 6 +++---
omemo/lib/python-omemo/tests/test_omemo_state.py | 20 ++++++++++----------
3 files changed, 15 insertions(+), 15 deletions(-)
diffs (124 lines):
diff -r c31a85bbdd58 -r ace1409f4cab omemo/__init__.py
--- a/omemo/__init__.py Fri Mar 25 17:46:19 2016 +0100
+++ b/omemo/__init__.py Fri Mar 25 18:07:46 2016 +0100
@@ -190,7 +190,7 @@
if contact_jid == my_jid:
log.info(account_name + ' ⇒ Received own device_list:' + str(
devices_list))
- state.add_own_devices(my_jid, devices_list)
+ state.set_own_devices(my_jid, devices_list)
if not state.own_device_id_published() or anydup(
state.own_devices):
@@ -203,7 +203,7 @@
else:
log.info(account_name + ' ⇒ Received device_list for ' +
contact_jid + ':' + str(devices_list))
- state.add_devices(contact_jid, devices_list)
+ state.set_devices(contact_jid, devices_list)
if account_name in self.ui_list and contact_jid not in self.ui_list[
account_name]:
diff -r c31a85bbdd58 -r ace1409f4cab omemo/lib/python-omemo/src/omemo/state.py
--- a/omemo/lib/python-omemo/src/omemo/state.py Fri Mar 25 17:46:19 2016 +0100
+++ b/omemo/lib/python-omemo/src/omemo/state.py Fri Mar 25 18:07:46 2016 +0100
@@ -77,7 +77,7 @@
sessionBuilder.processPreKeyBundle(prekey_bundle)
return self.get_session_cipher(recipient_id, device_id)
- def add_devices(self, name, devices):
+ def set_devices(self, name, devices):
""" Return a an.
Parameters
@@ -91,7 +91,7 @@
log.debug('Saving devices for ' + name + ' → ' + str(devices))
self.device_ids[name] = devices
- def add_own_devices(self, name, devices):
+ def set_own_devices(self, name, devices):
""" Overwrite the current :py:attribute:`OmemoState.own_devices` with
the given devices.
@@ -113,7 +113,7 @@
def own_device_id_published(self):
""" Return `True` only if own device id was added via
- :py:method:`OmemoState.add_own_devices()`.
+ :py:method:`OmemoState.set_own_devices()`.
"""
return self.own_device_id in self.own_devices
diff -r c31a85bbdd58 -r ace1409f4cab omemo/lib/python-omemo/tests/test_omemo_state.py
--- a/omemo/lib/python-omemo/tests/test_omemo_state.py Fri Mar 25 17:46:19 2016 +0100
+++ b/omemo/lib/python-omemo/tests/test_omemo_state.py Fri Mar 25 18:07:46 2016 +0100
@@ -38,7 +38,7 @@
assert len(omemo_state.own_devices) == 0
assert isinstance(omemo_state.own_device_id, int)
devices_update = [random.randint(0, sys.maxsize) for x in range(0,3)]
- omemo_state.add_own_devices(devices_update)
+ omemo_state.set_own_devices(devices_update)
assert len(omemo_state.own_devices) == 3
assert omemo_state.own_devices == devices_update
@@ -55,25 +55,25 @@
:py:attribute:`own_devices` should accept a list as argument, but should
not save duplicates
"""
- omemo_state.add_own_devices([1, 2, 2, 1])
+ omemo_state.set_own_devices([1, 2, 2, 1])
assert len(omemo_state.own_devices) == 2
def test_own_device_id_published(omemo_state):
""" :py:method:`OmemoState.own_device_id_published()` should return True
only if own device id was added via
- :py:method:`OmemoState.add_own_devices()`.
+ :py:method:`OmemoState.set_own_devices()`.
"""
assert omemo_state.own_device_id_published() == False
- omemo_state.add_own_devices([2,3,4,5])
+ omemo_state.set_own_devices([2,3,4,5])
assert omemo_state.own_device_id_published() == False
- omemo_state.add_own_devices([omemo_state.own_device_id])
+ omemo_state.set_own_devices([omemo_state.own_device_id])
assert omemo_state.own_device_id_published() == True
def test_add_device(omemo_state):
romeo = "romeo(a)example.com"
assert len(omemo_state.device_list_for(romeo)) == 0
- omemo_state.add_devices(romeo, (1,2,3,4))
+ omemo_state.set_devices(romeo, (1,2,3,4))
assert len(omemo_state.device_list_for(romeo)) == 4
julia = "julia(a)example.com"
@@ -83,25 +83,25 @@
@pytest.mark.skipif(True, reason="NOT IMPLEMENTED")
def test_device_list_tupple(omemo_state):
name = "romeo(a)example.com"
- omemo_state.add_devices(name, (1,2,3,4))
+ omemo_state.set_devices(name, (1,2,3,4))
assert isinstance(omemo_state.device_list_for(name), tuple)
def test_device_list_duplicate_handling(omemo_state):
""" Should not save duplicate device ids for the same user """
name = "romeo(a)example.com"
- omemo_state.add_devices(name, [1,2,2,1])
+ omemo_state.set_devices(name, [1,2,2,1])
assert len(omemo_state.device_list_for(name)) == 2
def test_own_devices_without_sessions(omemo_state):
own_jid = "romeo(a)example.com"
assert len(omemo_state.own_devices_without_sessions(own_jid)) == 0
- omemo_state.add_own_devices([1,2,3,4])
+ omemo_state.set_own_devices([1,2,3,4])
assert len(omemo_state.own_devices_without_sessions(own_jid)) == 4
def test_own_devices_without_sessions(omemo_state):
julia = "julia(a)example.com"
assert len(omemo_state.devices_without_sessions(julia)) == 0
- omemo_state.add_devices(julia, [1,2,3,4])
+ omemo_state.set_devices(julia, [1,2,3,4])
assert len(omemo_state.devices_without_sessions(julia)) == 4