From 9fe4bc50e70a1ff79de950ab8cb1596d013fbecc Mon Sep 17 00:00:00 2001
From: Karishma Chadha <kchadha@media.mit.edu>
Date: Tue, 20 Nov 2018 13:47:51 -0500
Subject: [PATCH] Add unit tests for new renameVariable and deleteVariable
 APIs.

---
 test/unit/util/cloud-provider.test.js | 28 +++++++++++++++++++++------
 1 file changed, 22 insertions(+), 6 deletions(-)

diff --git a/test/unit/util/cloud-provider.test.js b/test/unit/util/cloud-provider.test.js
index 3510339c0..7822ba0b4 100644
--- a/test/unit/util/cloud-provider.test.js
+++ b/test/unit/util/cloud-provider.test.js
@@ -43,6 +43,14 @@ describe('CloudProvider', () => {
         cloudProvider.randomizeDuration = t => t;
     });
 
+    test('createVariable', () => {
+        cloudProvider.createVariable('hello', 1);
+        const obj = JSON.parse(cloudProvider.connection._sentMessages[0]);
+        expect(obj.method).toEqual('create');
+        expect(obj.name).toEqual('hello');
+        expect(obj.value).toEqual(1);
+    });
+
     test('updateVariable', () => {
         cloudProvider.updateVariable('hello', 1);
         const obj = JSON.parse(cloudProvider.connection._sentMessages[0]);
@@ -59,13 +67,21 @@ describe('CloudProvider', () => {
         expect(obj.value).toEqual(0);
     });
 
-    test('writeToServer with falsey index value', () => {
-        cloudProvider.writeToServer('method', 'name', 5, 0);
+    test('renameVariable', () => {
+        cloudProvider.renameVariable('oldName', 'newName');
         const obj = JSON.parse(cloudProvider.connection._sentMessages[0]);
-        expect(obj.method).toEqual('method');
-        expect(obj.name).toEqual('name');
-        expect(obj.value).toEqual(5);
-        expect(obj.index).toEqual(0);
+        expect(obj.method).toEqual('rename');
+        expect(obj.name).toEqual('oldName');
+        expect(typeof obj.value).toEqual('undefined');
+        expect(obj.new_name).toEqual('newName');
+    });
+
+    test('deleteVariable', () => {
+        cloudProvider.deleteVariable('hello');
+        const obj = JSON.parse(cloudProvider.connection._sentMessages[0]);
+        expect(obj.method).toEqual('delete');
+        expect(obj.name).toEqual('hello');
+        expect(typeof obj.value).toEqual('undefined');
     });
 
     test('onMessage ack', () => {
-- 
GitLab