From 9d97579e4879ae3e88a896a2ce5c03355584b8e0 Mon Sep 17 00:00:00 2001
From: Keerthi Niranjan <keerthiniranjan16@gmail.com>
Date: Thu, 17 Aug 2017 15:04:24 +0530
Subject: [PATCH] SEARCH-154 - Changed the isInitialized method

---
 demo/search.html    |  3 +--
 js/search/search.js | 40 ++++++++++++++++++++--------------------
 2 files changed, 21 insertions(+), 22 deletions(-)

diff --git a/demo/search.html b/demo/search.html
index 6d5e5f12..f4f46477 100644
--- a/demo/search.html
+++ b/demo/search.html
@@ -132,8 +132,7 @@
 
     buttonEl.addEventListener('click', function () {
         if (!search.isLibInit()) {
-            alert('Library is not initiated. Init first...');
-            return;
+            search.init();
         }
         let out;
         table.innerHTML = '';
diff --git a/js/search/search.js b/js/search/search.js
index 1bc434e8..f99a0a61 100644
--- a/js/search/search.js
+++ b/js/search/search.js
@@ -59,21 +59,11 @@ class Search {
     }
 
     /**
-     * TO check whether SymphonySearchEngine library
-     * initialized or initialise it
-     * @returns {Promise}
+     * returns isInitialized boolean
+     * @returns {boolean}
      */
-    initLib() {
-        return new Promise((resolve) => {
-            if (!this.isInitialized) {
-                this.isInitialized = true;
-            }
-            resolve(libSymphonySearch.symSEInit());
-        });
-    }
-
     isLibInit() {
-        return this.initLib();
+        return this.isInitialized;
     }
 
     /**
@@ -103,12 +93,12 @@ class Search {
     indexBatch(messages) {
         return new Promise((resolve, reject) => {
             if (!Array.isArray(messages)) {
-                reject('Messages should be an array');
+                reject(new Error('Messages should be an array'));
                 return;
             }
 
             if (!this.isInitialized) {
-                reject("Not initialized");
+                reject(new Error("Library not initialized"));
                 return;
             }
 
@@ -140,6 +130,11 @@ class Search {
         if (!Array.isArray(message)) {
             return new Error('Messages should be an array');
         }
+
+        if (!this.isInitialized) {
+            return new Error("Library not initialized");
+        }
+
         let result = libSymphonySearch.symSEIndexRealTime(this.realTimeIndex, JSON.stringify(message));
         return result === 0 ? "Successful" : result
     }
@@ -191,20 +186,20 @@ class Search {
         let _sortOrder = sortOrder;
 
         return new Promise((resolve, reject) => {
-            if (!fs.existsSync(this.indexFolderName) && !fs.existsSync(this.realTimeIndex)) {
-                reject('Index folder does not exist.');
+            if (!this.isInitialized) {
+                reject(new Error("Library not initialized"));
                 return;
             }
 
-            if (!this.isInitialized) {
-                reject("Not initialized");
+            if (!fs.existsSync(this.indexFolderName) || !fs.existsSync(this.realTimeIndex)) {
+                reject('Index folder does not exist.');
                 return;
             }
 
             let q = Search.constructQuery(query, senderIds, threadIds);
 
             if (q === undefined) {
-                reject("Search query error");
+                reject(new Error("Search query error"));
                 return;
             }
 
@@ -251,6 +246,11 @@ class Search {
      */
     getLatestMessageTimestamp() {
         return new Promise((resolve, reject) => {
+            if (!this.isInitialized) {
+                reject("Not initialized");
+                return;
+            }
+
             if (!fs.existsSync(this.indexFolderName)) {
                 reject('Index folder does not exist.');
                 return;