fix: rename and improve function

loopbackio · Jul 10, 2020 · ad96e63 · ad96e63
1 parent 0cd851b
commit ad96e63
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 13 deletions.
diff --git a/lib/mongodb.js b/lib/mongodb.js
@@ -992,7 +992,7 @@ MongoDB.prototype.buildWhere = function(modelName, where, options) {
     const modelCtor = self._models[modelName];
 
     if (spec) {
-      spec = removingDollarSign(spec);
+      spec = trimLeadingDollarSigns(spec);
       if (spec === 'between') {
         query[k] = {$gte: cond[0], $lte: cond[1]};
       } else if (spec === 'inq') {
@@ -1999,14 +1999,11 @@ function isObjectIDProperty(modelCtor, propDef, value, options) {
  *
  * @param {*} spec the operator for Where filter
  */
-function removingDollarSign(spec) {
-  while (spec.charAt(0) === '$') {
-    spec = spec.substr(1);
-  }
-  return spec;
+function trimLeadingDollarSigns(spec) {
+  return spec.replace(/^(\$)+/, '');
 }
 
-exports.removingDollarSign = removingDollarSign;
+exports.trimLeadingDollarSigns = trimLeadingDollarSigns;
 
 function sanitizeFilter(filter, options) {
   options = Object.assign({}, options);

diff --git a/test/mongodb.test.js b/test/mongodb.test.js
@@ -12,7 +12,7 @@ const testUtils = require('../lib/test-utils');
 const async = require('async');
 const sinon = require('sinon');
 const sanitizeFilter = require('../lib/mongodb').sanitizeFilter;
-const removingDollarSign = require('../lib/mongodb').removingDollarSign;
+const trimLeadingDollarSigns = require('../lib/mongodb').trimLeadingDollarSigns;
 
 const GeoPoint = require('loopback-datasource-juggler').GeoPoint;
 
@@ -3727,24 +3727,24 @@ describe('mongodb connector', function() {
     });
   });
 
-  context('removingDollarSign', () =>{
+  context('trimLeadingDollarSigns', () =>{
     it('removes extra dollar sign(s) in ths operators', () => {
       let spec = '$eq';
-      let updatedSpec = removingDollarSign(spec);
+      let updatedSpec = trimLeadingDollarSigns(spec);
       updatedSpec.should.equal('eq');
 
       spec = '$$eq';
-      updatedSpec = removingDollarSign(spec);
+      updatedSpec = trimLeadingDollarSigns(spec);
       updatedSpec.should.equal('eq');
     });
 
     it('remains the same if there is the input does not start with dollar sign(s)', () => {
       let spec = 'eq';
-      let updatedSpec = removingDollarSign(spec);
+      let updatedSpec = trimLeadingDollarSigns(spec);
       updatedSpec.should.equal(spec);
 
       spec = 'eq$';
-      updatedSpec = removingDollarSign(spec);
+      updatedSpec = trimLeadingDollarSigns(spec);
       updatedSpec.should.equal('eq$');
     });
   });