remove some logs (#71)

design-first · Jan 11, 2020 · 9f85230 · 9f85230
1 parent 4a5e7a9
commit 9f85230
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 70 deletions.
diff --git a/src/db.js b/src/db.js
@@ -729,9 +729,8 @@ DatabaseCollection.prototype.insert = function insert(document) {
           $metamodel.prepareObject(obj, $metamodel.getModel(this.name));
 
           exports.store[this.name][obj[$mson.ID]] = obj;
-          result.push(obj[$mson.ID]);
 
-          $log.documentInserted(obj[$mson.ID], this.name);
+          result.push(obj[$mson.ID]);
 
           Component = $component.get(this.name);
           if (Component) {
@@ -873,12 +872,6 @@ DatabaseCollection.prototype.update = function update(query, update, options) {
 
                 docs[i][attributeName] = update[attributeName];
 
-                $log.documentUpdated(
-                  docs[i][$mson.ID],
-                  this.name,
-                  attributeName,
-                  update[attributeName]
-                );
                 result.push(docs[i][$mson.ID]);
 
                 if ($helper.isRuntime() && $helper.getRuntime().require('db')) {
@@ -936,12 +929,6 @@ DatabaseCollection.prototype.update = function update(query, update, options) {
 
             docs[i][attributeName] = update[attributeName];
 
-            $log.documentUpdated(
-              docs[i][$mson.ID],
-              this.name,
-              attributeName,
-              update[attributeName]
-            );
             result.push(docs[i][$mson.ID]);
 
             if ($helper.isRuntime() && $helper.getRuntime().require('db')) {
@@ -1002,7 +989,6 @@ DatabaseCollection.prototype.remove = function remove(query) {
 
               delete exports.store[this.name][id];
 
-              $log.documentRemoved(id, this.name);
               result.push(id);
 
               component = $component.get(id);
@@ -1038,7 +1024,6 @@ DatabaseCollection.prototype.remove = function remove(query) {
 
           delete exports.store[this.name][id];
 
-          $log.documentRemoved(id, this.name);
           result.push(id);
 
           component = $component.get(id);
@@ -1070,8 +1055,6 @@ DatabaseCollection.prototype.remove = function remove(query) {
 
       delete exports.store[this.name][id];
 
-      $log.documentRemoved(id, this.name);
-
       if (coreDb.indexOf(this.name) === -1) {
         component = $component.get(id);
         if (component) {

diff --git a/src/log.js b/src/log.js
@@ -1206,54 +1206,3 @@ exports.historyDocumentUpdated = function historyDocumentRemoved(
       "')"
   );
 };
-
-/**
- * @method documentRemoved
- * @param {String} id id of the document
- * @param {String} collection name of the collection
- * @description A document from a collection was removed
- */
-exports.documentRemoved = function documentRemoved(id, collection) {
-  getLogger().debug(
-    "the document '" + id + "' was removed (collection '" + collection + "')"
-  );
-};
-
-/**
- * @method documentInserted
- * @param {String} id id of the document
- * @param {String} collection name of the collection
- * @description A document was inserted into a collection
- */
-exports.documentInserted = function documentInserted(id, collection) {
-  getLogger().debug(
-    "the document '" + id + "' was inserted (collection '" + collection + "')"
-  );
-};
-
-/**
- * @method documentUpdated
- * @param {String} id id of the document
- * @param {String} collection name of the collection
- * @param {String} property property of the document
- * @param {String} value new value of the property
- * @description A document was updated
- */
-exports.documentUpdated = function documentUpdated(
-  id,
-  collection,
-  property,
-  value
-) {
-  getLogger().debug(
-    "the property '" +
-      property +
-      "' of the document '" +
-      id +
-      "' was updated with the value '" +
-      value +
-      "' (collection '" +
-      collection +
-      "')"
-  );
-};
diff --git a/test/module/db-spec.js b/test/module/db-spec.js
@@ -150,7 +150,15 @@ describe('System Runtime db component', () => {
   });
 
   it('can add a document', () => {
-    const docAdded = db._Runtime.insert({
+    let docAdded = db._Runtime.insert({
+      '_id': 'runtime1',
+      'version': '0.0.0'
+    });
+
+    expect(docAdded[0]).equal('runtime1');
+
+    // try to insert the same document
+    docAdded = db._Runtime.insert({
       '_id': 'runtime1',
       'version': '0.0.0'
     });