@emartech/data-aggregator-language - npm Package Compare versions

Comparing version 2.6.0 to 2.6.1

package.json

@@ -30,3 +30,3 @@ {
   },
-  "version": "2.6.0"
+  "version": "2.6.1"
 }

src/aggregator/index.spec.js

@@ -78,2 +78,14 @@ 'use strict';
         });
+        it('works with a single negative number', () => {
+          expect(aggregate('-5')).to.eql(-5);
+        });
+        it('Subtraction with negative number', () => {
+          expect(aggregate('5 - -5')).to.eql(10);
+        });
+        it('should throw error when -- is used', () => {
+          expect(() => aggregate('--5')).to.throw('Error parsing "--5"');
+        });
+        it('should work with positive number(with plus sign)', () => {
+          expect(aggregate('+5')).to.eql(5);
+        });
       });
@@ -80,0 +92,0 @@

src/aggregator/interpreter.js

@@ -88,2 +88,5 @@ 'use strict';
     numberExpression(ctx) {
+      if (ctx.minusOperator) {
+        return parseFloat(ctx.numberLiteral[0].image) * -1;
+      }
       return parseFloat(ctx.numberLiteral[0].image);
@@ -90,0 +93,0 @@ }

src/aggregator/parser.js

@@ -133,2 +133,7 @@ 'use strict';
     $.RULE('numberExpression', () => {
+      $.OPTION(() => {
+        $.OR([
+          { ALT: () => $.CONSUME(minusOperator) },
+          { ALT: () => $.CONSUME(plusOperator) }]);
+      });
       $.CONSUME(numberLiteral);
@@ -135,0 +140,0 @@ });

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

28656

increased by2.7%

Lines of code

690

increased by2.99%

No dependency changes