Fix and tests for unclosed tag handling being an infinite loop. Fixes #7

.

src/index.js

@@ -53,6 +53,14 @@ export default function parse(md) {
 		chunk, prev, token, inner, t;
 
 	md = trim(md).replace(/^\[(.+?)\]:\s*(.+)$/gm, (s, name, url) => {
+	function flush() {
+		let str = '';
+		for (let i=context.length; i--; ) {
+			str += tag(context[i]);
+		}
+		return str;
+	}
+
 		links[name.toLowerCase()] = url;
 		return '';
 	});
@@ -89,7 +97,7 @@ export default function parse(md) {
 		// Links:
 		else if (token[9]) {
 			out = out.replace('<a>', `<a href="${encodeAttr(token[10] || links[prev.toLowerCase()])}">`);
-			chunk = '</a>';
+			chunk = flush() + '</a>';
 		}
 		else if (token[8]) {
 			chunk = '<a>';
@@ -113,9 +121,7 @@ export default function parse(md) {
 
 	out += md.substring(last);
 
-	while ((token=context.pop())) {
-		out += tag(context, token);
-	}
+	out += flush();
 
 	return trim(out);
 }

test/index.js

@@ -113,4 +113,21 @@ describe('snarkdown()', () => {
 			expect(snarkdown('> To be or not to be')).to.equal('<blockquote>\nTo be or not to be\n</blockquote>');
 		});
 	});
+
+	describe('edge cases', () => {
+		it('should close unclosed tags', () => {
+			expect(snarkdown('*foo')).to.equal('<em>foo</em>');
+			expect(snarkdown('foo**')).to.equal('foo<strong></strong>');
+			expect(snarkdown('[some **bold text](#winning)')).to.equal('<a href="#winning">some <strong>bold text</strong></a>');
+			expect(snarkdown('`foo')).to.equal('`foo');
+		});
+
+		it('should not choke on single characters', () => {
+			expect(snarkdown('*')).to.equal('<em></em>');
+			expect(snarkdown('_')).to.equal('<em></em>');
+			expect(snarkdown('**')).to.equal('<strong></strong>');
+			expect(snarkdown('>')).to.equal('>');
+			expect(snarkdown('`')).to.equal('`');
+		});
+	});
 });