From ab72c7103902081a332c09121a5ea671f27e874d Mon Sep 17 00:00:00 2001
From: Sergi Delgado Segura <sergi.delgado.s@gmail.com>
Date: Wed, 9 Oct 2019 15:19:33 +0100
Subject: [PATCH] Adds appointment unit test

---
 pisa/appointment.py           |  1 +
 test/unit/test_appointment.py | 47 +++++++++++++++++++++++++++++++++++
 2 files changed, 48 insertions(+)
 create mode 100644 test/unit/test_appointment.py

diff --git a/pisa/appointment.py b/pisa/appointment.py
index a4d5718..ac1d3a2 100644
--- a/pisa/appointment.py
+++ b/pisa/appointment.py
@@ -3,6 +3,7 @@ from pisa.encrypted_blob import EncryptedBlob
 
 # Basic appointment structure
 class Appointment:
+    # TODO: 35-appointment-checks
     def __init__(self, locator, start_time, end_time, dispute_delta, encrypted_blob, cipher, hash_function):
         self.locator = locator
         self.start_time = start_time    # ToDo: #4-standardize-appointment-fields
diff --git a/test/unit/test_appointment.py b/test/unit/test_appointment.py
new file mode 100644
index 0000000..0caf459
--- /dev/null
+++ b/test/unit/test_appointment.py
@@ -0,0 +1,47 @@
+from os import urandom
+from pytest import fixture
+
+from pisa.appointment import Appointment
+from pisa.encrypted_blob import EncryptedBlob
+
+
+# Not much to test here, adding it for completeness
+
+@fixture
+def appointment_data():
+    locator = urandom(32).hex()
+    start_time = 100
+    end_time = 120
+    dispute_delta = 20
+    encrypted_blob_data = urandom(100).hex()
+    cipher = "AES-GCM-128"
+    hash_function = "SHA256"
+
+    return locator, start_time, end_time, dispute_delta, encrypted_blob_data, cipher, hash_function
+
+
+def test_init_appointment(appointment_data):
+    # The appointment has no checks whatsoever, since the inspector is the one taking care or that, and the only one
+    # creating appointments.
+    # DISCUSS: whether this makes sense by design or checks should be ported from the inspector to the appointment
+    #          35-appointment-checks
+
+    locator, start_time, end_time, dispute_delta, encrypted_blob_data, cipher, hash_function = appointment_data
+
+    appointment = Appointment(locator, start_time, end_time, dispute_delta, encrypted_blob_data, cipher, hash_function)
+
+    assert (locator == appointment.locator and start_time == appointment.start_time and end_time == appointment.end_time
+            and EncryptedBlob(encrypted_blob_data) == appointment.encrypted_blob and cipher == appointment.cipher
+            and dispute_delta == appointment.dispute_delta and hash_function == appointment.hash_function)
+
+
+def test_to_json(appointment_data):
+    locator, start_time, end_time, dispute_delta, encrypted_blob_data, cipher, hash_function = appointment_data
+    appointment = Appointment(locator, start_time, end_time, dispute_delta, encrypted_blob_data, cipher, hash_function)
+
+    json_appointment = appointment.to_json()
+
+    assert (locator == json_appointment.get("locator") and start_time == json_appointment.get("start_time")
+            and end_time == json_appointment.get("end_time") and dispute_delta == json_appointment.get("dispute_delta")
+            and cipher == json_appointment.get("cipher") and hash_function == json_appointment.get("hash_function")
+            and encrypted_blob_data == json_appointment.get("encrypted_blob"))