[Issue #6] Stack traces should be filtered by .ts extension

Author: DrewImm

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## [1.0.2]
 
 ### Fixes
+- [Issue #6] Stack traces should be filtered by .ts extension
 - Typescript Template v2
 
 ## [1.0.1]

jasmine.json

@@ -3,5 +3,5 @@
 	"spec_files": [ "**/*[sS]pec.js" ],
 	"helpers": [ "helpers/**/*.js" ],
 	"stopSpecOnExpectationFailure": false,
-	"random": true
+	"random": false
 }

src/index.ts

@@ -26,6 +26,9 @@ export interface ErrorHandlerOptions {
 	// Ignore node internals? (justMyCode must be true)
 	justMyCodeIncludeInternals: boolean,
 
+	// Ignore javascript & compiled files?
+	includeJsFiles: boolean,
+
 	// Error handler
 	handler?: HandlerFunction,
 }
@@ -38,6 +41,7 @@ export const defaultOptions: ErrorHandlerOptions = {
 	justMyCode: true,
 	justMyCodeIncludeNodeModules: false,
 	justMyCodeIncludeInternals: false,
+	includeJsFiles: false,
 
 	// eslint-disable-next-line no-console
 	handler: console.error
@@ -72,6 +76,10 @@ export function setupErrorHandling(options: Partial<ErrorHandlerOptions>) {
 				.filter(line => {
 					return (
 						line &&
+						(
+							options.includeJsFiles ||
+							!line.includes('.js')
+						) &&
 						(
 							options.justMyCodeIncludeNodeModules ||
 							!line.includes('node_modules')

test/spec/index.spec.ts

@@ -6,7 +6,8 @@ import { getStackTrace } from '../util/get-stack-trace';
 describe('index', () => {
 	it('sets stack trace to "Just My Code"', () => {
 		setupErrorHandling({
-			justMyCode: true
+			justMyCode: true,
+			includeJsFiles: true,
 		});
 
 		const trace = getStackTrace();
@@ -15,4 +16,14 @@ describe('index', () => {
 		expect(trace).not.toContain('node_modules');
 		expect(trace).not.toContain('internal/');
 	});
+
+	it('allows filtering of .js extension', () => {
+		setupErrorHandling({
+			justMyCode: true
+		});
+
+		const trace = getStackTrace();
+
+		expect(trace).not.toContain('.js');
+	});
 });