refactor logger utility to accept multiple arguments

jlmakes · jlmakes · commit 2efc0307c447 · 2017-01-22T00:56:15.000+01:00
diff --git a/src/utils/core.js b/src/utils/core.js
@@ -117,6 +117,10 @@ export function getScrolled (container) {
 }
 
 
-export function logger (message) {
-	if (console) console.log(`ScrollReveal: ${message}`) // eslint-disable-line no-console
+export function logger (message, ...details) {
+	if (console) {
+		let report = `ScrollReveal: ${message}`
+		details.forEach(detail => report += `\n  - ${detail}`)
+		console.log(report) // eslint-disable-line no-console
+	}
 }
diff --git a/test/utils/core.spec.js b/test/utils/core.spec.js
@@ -111,16 +111,21 @@ describe('Core Utilities', () => {
 			stub = sinon.stub(console, 'log', spy)
 		})
 
-		it('should invoke console.log', () => {
+		it('invokes console.log', () => {
 			logger()
 			expect(spy).to.have.been.called
 		})
 
-		it('should prepend output with `ScrollReveal: `', () => {
+		it('prepends output with `ScrollReveal: `', () => {
 			logger('test')
 			expect(spy).to.have.been.calledWith('ScrollReveal: test')
 		})
 
+		it('accepts multiple arguments as message details', () => {
+			logger('message', 'detail one', 'detail two')
+			expect(spy).to.have.been.calledWith('ScrollReveal: message\n  - detail one\n  - detail two')
+		})
+
 		after('restore console log', () => {
 			stub.restore()
 		})

Original file line number	Diff line number	Diff line change
`@@ -117,6 +117,10 @@ export function getScrolled (container) {`
`117`	`117`	`}`
`118`	`118`
`119`	`119`
`120`		`-export function logger (message) {`
`121`		- if (console) console.log(`ScrollReveal: ${message}`) // eslint-disable-line no-console
	`120`	`+export function logger (message, ...details) {`
	`121`	`+ if (console) {`
	`122`	+ let report = `ScrollReveal: ${message}`
	`123`	+ details.forEach(detail => report += `\n - ${detail}`)
	`124`	`+ console.log(report) // eslint-disable-line no-console`
	`125`	`+ }`
`122`	`126`	`}`