Add tests

Author: fisker

rules/no-zero-fractions.js

@@ -1,6 +1,6 @@
 'use strict';
 const getDocumentationUrl = require('./utils/get-documentation-url');
-const {isNumber, parseNumber} = require('./utils/numeric');
+const {isNumber, parseNumber, parseFloat} = require('./utils/numeric');
 
 const MESSAGE_ZERO_FRACTION = 'zero-fraction';
 const MESSAGE_DANGLING_DOT = 'dangling-dot';
@@ -12,9 +12,10 @@ const messages = {
 const format = text => {
 	// Legacy octal number `0777` and prefixed number `0o1234` can't has dot.
 	const {number, mark, sign, power} = parseNumber(text);
+	const {integer, dot, fractional} = parseFloat(number);
 	return {
-		hasDanglingDot: number.endsWith('.'),
-		formatted: number.replace(/[0_.]$/g, '') + mark + sign + power
+		hasDanglingDot: !fractional,
+		formatted: integer + (dot + fractional).replace(/[0_.]+$/g, '') + mark + sign + power
 	};
 }
 

rules/utils/numeric.js

@@ -28,7 +28,7 @@ function parseNumber(text) {
 		mark = '',
 		sign = '',
 		power = ''
-	} = text.match(/^(?<number>.*?)(?:(?<mark>e)(?<sign>[+-])?(?<power>\d+))?$/i).groups;
+	} = text.match(/^(?<number>.*?)(?:(?<mark>e)(?<sign>[+-])?(?<power>.+))?$/i).groups;
 
 	return {number, mark, sign, power};
 }

test/no-zero-fractions.mjs

@@ -30,6 +30,36 @@ test.snapshot({
 		'const foo = 1.e10',
 		'const foo = +1.e-10',
 		'const foo = -1.e+10',
-		'const foo = (1.).toString()'
+		'const foo = (1.).toString()',
+		...[
+			'123_000.',
+			'123_000.0',
+			'123_000.000',
+			'123_000.000_000',
+			'123_000.123_000',
+			'123_000.000_400'
+		]
+			.flatMap(number => [
+				number,
+				`${number}e1`,
+				`${number}e+1`,
+				`${number}e-1`,
+				`${number}e0`,
+				`${number}e+0`,
+				`${number}e-0`,
+				`${number}e10`,
+				`${number}e+10`,
+				`${number}e-10`,
+				`${number}E-10`,
+				`${number}E-10_10`
+			])
+			.flatMap(number => [
+				`+${number}`,
+				`-${number}`
+			])
+			.flatMap(number => [
+				`${number};`
+			])
+
 	]
 });