Restoring engine as any structure in test

forcedotcom · Aug 16, 2021 · a63b95c · a63b95c
1 parent d09b8dd
commit a63b95c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 14 deletions.
diff --git a/src/lib/cpd/CpdEngine.ts b/src/lib/cpd/CpdEngine.ts
@@ -121,8 +121,8 @@ export class CpdEngine extends AbstractRuleEngine {
 		return results;
 	}
 
-	//protected visibility to be available in unit tests
-	protected sortPaths(targets: RuleTarget[]): Map<LANGUAGE, string[]> {
+
+	private sortPaths(targets: RuleTarget[]): Map<LANGUAGE, string[]> {
 		const languageToPaths = new Map<LANGUAGE, string[]>();
 		const unmatchedPaths: string[] = [];
 		for (const target of targets) {

diff --git a/test/lib/cpd/CpdEngine.test.ts b/test/lib/cpd/CpdEngine.test.ts
@@ -7,16 +7,8 @@ import { RuleTarget } from '../../../src/types';
 TestOverrides.initializeTestSetup();
 
 describe('CpdEngine', () => {
-	class CpdEngineSubclass extends CpdEngine {
-		constructor() {
-			super();
-		}
-		public sortPaths(targets: RuleTarget[]): Map<LANGUAGE, string[]> {
-			return super.sortPaths(targets);
-		}
-	}
-
-	const testEngine: CpdEngineSubclass = new CpdEngineSubclass();
+
+	const testEngine: CpdEngine = new CpdEngine();
 
 	before(async () => {
 		await testEngine.init();
@@ -38,7 +30,7 @@ describe('CpdEngine', () => {
 			let langPathMap: Map<LANGUAGE, string[]> = new Map();
 			langPathMap.set(LANGUAGE.APEX, ['file.cls','file.trigger']);
 			langPathMap.set(LANGUAGE.VISUALFORCE, ['file.page','file.component']);
-			expect(testEngine.sortPaths(targets)).to.eql(langPathMap);
+			expect((testEngine as any).sortPaths(targets)).to.eql(langPathMap);
 		});
 
 		it('should compare extensions case-insensitively', () => {
@@ -49,7 +41,7 @@ describe('CpdEngine', () => {
 
 			let langPathMap: Map<LANGUAGE, string[]> = new Map();
 			langPathMap.set(LANGUAGE.APEX, ['file.Cls','file.tRIGGER']);
-			expect(testEngine.sortPaths(targets)).to.eql(langPathMap);
+			expect((testEngine as any).sortPaths(targets)).to.eql(langPathMap);
 		});
 	});