✨ feat: Test empty string or pattern and allow to skip those cases.

Author: make-github-pseudonymous-again

src/data.js

@@ -1,7 +1,24 @@
 const data = [
+	{
+		string: '',
+		patterns: [
+			{
+				pattern: '',
+				hits: [0],
+			},
+			{
+				pattern: 'abcd',
+				hits: [],
+			},
+		],
+	},
 	{
 		string: 'abcd',
 		patterns: [
+			{
+				pattern: '',
+				hits: [0, 1, 2, 3, 4],
+			},
 			{
 				pattern: 'abcde',
 				hits: [],
@@ -34,6 +51,43 @@ const data = [
 			},
 		],
 	},
+	{
+		string:
+			'Some books are to be tasted, others to be swallowed, and some few to be chewed and digested.',
+		patterns: [
+			{
+				pattern: 'to',
+				hits: [15, 36, 66],
+			},
+		],
+	},
+	{
+		string: 'aaaaaaaaab',
+		patterns: [
+			{
+				pattern: 'aaaab',
+				hits: [5],
+			},
+		],
+	},
+	{
+		string: 'tototo',
+		patterns: [
+			{
+				pattern: 'to',
+				hits: [0, 2, 4],
+			},
+		],
+	},
+	{
+		string: 'aaaaa',
+		patterns: [
+			{
+				pattern: 'aa',
+				hits: [0, 1, 2, 3],
+			},
+		],
+	},
 ];
 
 export default data;

src/mock.js

@@ -19,7 +19,8 @@ export default function* mock(string, patterns) {
 		while (true) {
 			const i = s.indexOf(p, fromIndex);
 			if (i === -1) break;
-			yield new Hit(i, pattern, pattern.i);
+			yield new Hit(string.i + i, pattern, pattern.i);
+			if (fromIndex >= s.length) break;
 			fromIndex = i + 1;
 		}
 	}

src/single.js

@@ -1,7 +1,15 @@
-const single = ({test, algorithms, data}) => {
+const single = ({
+	test,
+	algorithms,
+	data,
+	skipEmptyString,
+	skipEmptyPattern,
+}) => {
 	for (const algorithm of algorithms) {
 		for (const {string, patterns} of data) {
+			if (string === '' && skipEmptyString) continue;
 			for (const {pattern, hits} of patterns) {
+				if (pattern === '' && skipEmptyPattern) continue;
 				test(macro, algorithm, string, pattern, hits);
 			}
 		}

test/src/_fixtures.js

@@ -0,0 +1,9 @@
+import {map} from '@iterable-iterator/map';
+
+import {mock, StringSlice} from '../../src/index.js';
+
+export const mockedSingle = (s, si, sj, p, pi, pj) =>
+	map(
+		(hit) => hit.si,
+		mock(new StringSlice(s, si, sj), [new StringSlice(p, pi, pj)]),
+	);

test/src/single.js

@@ -1,14 +1,8 @@
 import test from 'ava';
 
-import {map} from '@iterable-iterator/map';
+import {single, data} from '../../src/index.js';
 
-import {mock, single, data, StringSlice} from '../../src/index.js';
-
-const mockedSingle = (s, si, sj, p, pi, pj) =>
-	map(
-		(hit) => hit.si,
-		mock(new StringSlice(s, si, sj), [new StringSlice(p, pi, pj)]),
-	);
+import {mockedSingle} from './_fixtures.js';
 
 single({
 	test,

test/src/skip.js

@@ -0,0 +1,13 @@
+import test from 'ava';
+
+import {single, data} from '../../src/index.js';
+
+import {mockedSingle} from './_fixtures.js';
+
+single({
+	test,
+	algorithms: [mockedSingle],
+	data,
+	skipEmptyString: true,
+	skipEmptyPattern: true,
+});