Improves autolink performance

Avoids static methods (screws with tree-shaking)

Author: eamodio

src/autolinks/__tests__/autolinks.test.ts

@@ -2,7 +2,7 @@ import * as assert from 'assert';
 import { suite, test } from 'mocha';
 import { map } from '../../system/iterable';
 import type { Autolink, RefSet } from '../autolinks';
-import { Autolinks } from '../autolinks';
+import { getAutolinks, getBranchAutolinks } from '../autolinks';
 
 const mockRefSets = (prefixes: string[] = ['']): RefSet[] =>
 	prefixes.map(prefix => [
@@ -25,27 +25,27 @@ function assertAutolinks(actual: Map<string, Autolink>, expected: Array<string>)
 
 suite('Autolinks Test Suite', () => {
 	test('Branch name autolinks', () => {
-		assertAutolinks(Autolinks._getBranchAutolinks('123', mockRefSets()), ['test/123']);
-		assertAutolinks(Autolinks._getBranchAutolinks('feature/123', mockRefSets()), ['test/123']);
-		assertAutolinks(Autolinks._getBranchAutolinks('feature/PRE-123', mockRefSets()), ['test/123']);
-		assertAutolinks(Autolinks._getBranchAutolinks('123.2', mockRefSets()), ['test/123', 'test/2']);
-		assertAutolinks(Autolinks._getBranchAutolinks('123', mockRefSets(['PRE-'])), []);
-		assertAutolinks(Autolinks._getBranchAutolinks('feature/123', mockRefSets(['PRE-'])), []);
-		assertAutolinks(Autolinks._getBranchAutolinks('feature/2-fa/123', mockRefSets([''])), ['test/123', 'test/2']);
-		assertAutolinks(Autolinks._getBranchAutolinks('feature/2-fa/123', mockRefSets([''])), ['test/123', 'test/2']);
-		// incorrectly solved case, maybe it worths to compare the blocks length so that the less block size (without possible link) is more likely a link
-		assertAutolinks(Autolinks._getBranchAutolinks('feature/2-fa/3', mockRefSets([''])), ['test/2', 'test/3']);
-		assertAutolinks(Autolinks._getBranchAutolinks('feature/PRE-123', mockRefSets(['PRE-'])), ['test/123']);
-		assertAutolinks(Autolinks._getBranchAutolinks('feature/PRE-123.2', mockRefSets(['PRE-'])), ['test/123']);
-		assertAutolinks(Autolinks._getBranchAutolinks('feature/3-123-PRE-123', mockRefSets(['PRE-'])), ['test/123']);
+		assertAutolinks(getBranchAutolinks('123', mockRefSets()), ['test/123']);
+		assertAutolinks(getBranchAutolinks('feature/123', mockRefSets()), ['test/123']);
+		assertAutolinks(getBranchAutolinks('feature/PRE-123', mockRefSets()), ['test/123']);
+		assertAutolinks(getBranchAutolinks('123.2', mockRefSets()), ['test/123', 'test/2']);
+		assertAutolinks(getBranchAutolinks('123', mockRefSets(['PRE-'])), []);
+		assertAutolinks(getBranchAutolinks('feature/123', mockRefSets(['PRE-'])), []);
+		assertAutolinks(getBranchAutolinks('feature/2-fa/123', mockRefSets([''])), ['test/123', 'test/2']);
+		assertAutolinks(getBranchAutolinks('feature/2-fa/123', mockRefSets([''])), ['test/123', 'test/2']);
+		// incorrectly solved cat worths to compare the blocks length so that the less block size (without possible link) is more likely a link
+		assertAutolinks(getBranchAutolinks('feature/2-fa/3', mockRefSets([''])), ['test/2', 'test/3']);
+		assertAutolinks(getBranchAutolinks('feature/PRE-123', mockRefSets(['PRE-'])), ['test/123']);
+		assertAutolinks(getBranchAutolinks('feature/PRE-123.2', mockRefSets(['PRE-'])), ['test/123']);
+		assertAutolinks(getBranchAutolinks('feature/3-123-PRE-123', mockRefSets(['PRE-'])), ['test/123']);
 		assertAutolinks(
-			Autolinks._getBranchAutolinks('feature/3-123-PRE-123', mockRefSets(['', 'PRE-'])),
+			getBranchAutolinks('feature/3-123-PRE-123', mockRefSets(['', 'PRE-'])),
 
 			['test/123', 'test/3'],
 		);
 	});
 
 	test('Commit message autolinks', () => {
-		assertAutolinks(Autolinks._getAutolinks('test message 123 sd', mockRefSets()), ['test/123']);
+		assertAutolinks(getAutolinks('test message 123 sd', mockRefSets()), ['test/123']);
 	});
 });