Move helper tests to separate files

test/commit-list-helper.js

@@ -0,0 +1,82 @@
+import { describe, it } from 'mocha'
+import { expect } from 'chai'
+import Handlebars from 'handlebars'
+
+describe('commit-list helper', () => {
+  const commits = [
+    { subject: 'Commit 1', message: 'Commit 1\n\nThis is commit 1, nothing special' },
+    { subject: 'Commit 2', message: 'Commit 2\n\nBREAKING CHANGE: This commit breaks something' },
+    { subject: 'feat: Commit 3', message: 'feat: Commit 3\n\nThis commit adds a feature' }
+  ]
+
+  it('returns nothing with no commits', () => {
+    const compile = Handlebars.compile(
+      '{{#commit-list commits heading="# Heading"}}\n' +
+        '- {{subject}}\n' +
+      '{{/commit-list}}'
+    )
+    const expected = ''
+    expect(compile({ commits: [] })).to.equal(expected)
+  })
+
+  it('returns all commits with no options', () => {
+    const compile = Handlebars.compile(
+      '{{#commit-list commits heading="# Heading"}}\n' +
+        '- {{subject}}\n' +
+      '{{/commit-list}}'
+    )
+    const expected =
+      '# Heading\n' +
+      '- Commit 1\n' +
+      '- Commit 2\n' +
+      '- feat: Commit 3\n'
+    expect(compile({ commits })).to.equal(expected)
+  })
+
+  it('supports subject pattern matching', () => {
+    const compile = Handlebars.compile(
+      '{{#commit-list commits heading="# Heading" subject="^feat: "}}\n' +
+        '- {{subject}}\n' +
+      '{{/commit-list}}'
+    )
+    const expected =
+      '# Heading\n' +
+      '- feat: Commit 3\n'
+    expect(compile({ commits })).to.equal(expected)
+  })
+
+  it('supports message pattern matching', () => {
+    const compile = Handlebars.compile(
+      '{{#commit-list commits heading="# Breaking Changes" message="^BREAKING CHANGE: "}}\n' +
+        '- {{subject}}\n' +
+      '{{/commit-list}}'
+    )
+    const expected =
+      '# Breaking Changes\n' +
+      '- Commit 2\n'
+    expect(compile({ commits })).to.equal(expected)
+  })
+
+  it('supports excludes option', () => {
+    const compile = Handlebars.compile(
+      '{{#commit-list commits heading="# Heading" exclude="^BREAKING CHANGE: "}}\n' +
+        '- {{subject}}\n' +
+      '{{/commit-list}}'
+    )
+    const expected =
+      '# Heading\n' +
+      '- Commit 1\n' +
+      '- feat: Commit 3\n'
+    expect(compile({ commits })).to.equal(expected)
+  })
+
+  it('returns nothing if nothing matches', () => {
+    const compile = Handlebars.compile(
+      '{{#commit-list commits heading="# Heading" message="A string that never appears"}}\n' +
+        '- {{subject}}\n' +
+      '{{/commit-list}}'
+    )
+    const expected = ''
+    expect(compile({ commits })).to.equal(expected)
+  })
+})

test/matches-helper.js

@@ -0,0 +1,72 @@
+import { describe, it } from 'mocha'
+import { expect } from 'chai'
+import Handlebars from 'handlebars'
+
+import releases from './data/releases'
+
+describe('matches helper', () => {
+  const compileCommits = (matches) => Handlebars.compile(
+    '{{#each releases}}\n' +
+      '{{#each commits}}\n' +
+        matches +
+      '{{/each}}\n' +
+    '{{/each}}'
+  )
+
+  it('matches on field value', () => {
+    const matches =
+      '{{#matches href "12c0624"}}\n' +
+        '- {{message}}\n' +
+      '{{/matches}}\n'
+    const expected =
+      '- Commit that fixes nothing\n'
+    expect(compileCommits(matches)({ releases })).to.equal(expected)
+  })
+
+  it('matches with case insensitive flag', () => {
+    const matches =
+      '{{#matches author "pete" flags="i"}}\n' +
+        '- {{shorthash}}\n' +
+      '{{/matches}}\n'
+    const expected =
+      '- b0b3040\n' +
+      '- 12c0624\n' +
+      '- e9a43b2\n' +
+      '- 158fdde\n'
+    expect(compileCommits(matches)({ releases })).to.equal(expected)
+  })
+
+  it('provides non-matching conditional', () => {
+    const matches =
+      '{{#matches shorthash "e9a43b2"}}\n' +
+        '- HIT {{date}}\n' +
+      '{{else}}\n' +
+        '- MISS {{date}}\n' +
+      '{{/matches}}\n'
+    const expected =
+      '- MISS 2015-12-29T21:57:19.000Z\n' +
+      '- MISS 2015-12-29T21:18:19.000Z\n' +
+      '- HIT 2015-12-29T21:19:19.000Z\n' +
+      '- MISS 2015-12-14T17:06:12.000Z\n'
+    expect(compileCommits(matches)({ releases })).to.equal(expected)
+  })
+
+  it('matches on multiline content', () => {
+    const multiReleases = [{ commits: [
+      {
+        shorthash: 'c0f25d7',
+        message: 'Hello\n\nWorld\n\nBREAKING CHANGE: mock break\n\nsome more text'
+      }, {
+        shorthash: '12cd728',
+        message: 'Nope'
+      }
+    ]}]
+    const matches =
+      '{{#matches message "BREAKING CHANGE"}}\n' +
+        '- {{shorthash}}\n' +
+      '{{/matches}}\n'
+    const expected =
+      '- c0f25d7\n'
+    expect(compileCommits(matches)({ releases: multiReleases })).to.equal(expected)
+  })
+})

test/template.js

@@ -2,7 +2,6 @@ import { describe, it } from 'mocha'
 import { expect } from 'chai'
 import { readFile } from 'fs-extra'
 import { join } from 'path'
-import Handlebars from 'handlebars'
 
 import releases from './data/releases'
 import { compileTemplate } from '../src/template'
@@ -35,149 +34,3 @@ describe('compileTemplate', () => {
       .catch(() => done())
   })
 })
-
-describe('commit-list helper', () => {
-  const commits = [
-    { subject: 'Commit 1', message: 'Commit 1\n\nThis is commit 1, nothing special' },
-    { subject: 'Commit 2', message: 'Commit 2\n\nBREAKING CHANGE: This commit breaks something' },
-    { subject: 'feat: Commit 3', message: 'feat: Commit 3\n\nThis commit adds a feature' }
-  ]
-
-  it('returns nothing with no commits', () => {
-    const compile = Handlebars.compile(
-      '{{#commit-list commits heading="# Heading"}}\n' +
-        '- {{subject}}\n' +
-      '{{/commit-list}}'
-    )
-    const expected = ''
-    expect(compile({ commits: [] })).to.equal(expected)
-  })
-
-  it('returns all commits with no options', () => {
-    const compile = Handlebars.compile(
-      '{{#commit-list commits heading="# Heading"}}\n' +
-        '- {{subject}}\n' +
-      '{{/commit-list}}'
-    )
-    const expected =
-      '# Heading\n' +
-      '- Commit 1\n' +
-      '- Commit 2\n' +
-      '- feat: Commit 3\n'
-    expect(compile({ commits })).to.equal(expected)
-  })
-
-  it('supports subject pattern matching', () => {
-    const compile = Handlebars.compile(
-      '{{#commit-list commits heading="# Heading" subject="^feat: "}}\n' +
-        '- {{subject}}\n' +
-      '{{/commit-list}}'
-    )
-    const expected =
-      '# Heading\n' +
-      '- feat: Commit 3\n'
-    expect(compile({ commits })).to.equal(expected)
-  })
-
-  it('supports message pattern matching', () => {
-    const compile = Handlebars.compile(
-      '{{#commit-list commits heading="# Breaking Changes" message="^BREAKING CHANGE: "}}\n' +
-        '- {{subject}}\n' +
-      '{{/commit-list}}'
-    )
-    const expected =
-      '# Breaking Changes\n' +
-      '- Commit 2\n'
-    expect(compile({ commits })).to.equal(expected)
-  })
-
-  it('supports excludes option', () => {
-    const compile = Handlebars.compile(
-      '{{#commit-list commits heading="# Heading" exclude="^BREAKING CHANGE: "}}\n' +
-        '- {{subject}}\n' +
-      '{{/commit-list}}'
-    )
-    const expected =
-      '# Heading\n' +
-      '- Commit 1\n' +
-      '- feat: Commit 3\n'
-    expect(compile({ commits })).to.equal(expected)
-  })
-
-  it('returns nothing if nothing matches', () => {
-    const compile = Handlebars.compile(
-      '{{#commit-list commits heading="# Heading" message="A string that never appears"}}\n' +
-        '- {{subject}}\n' +
-      '{{/commit-list}}'
-    )
-    const expected = ''
-    expect(compile({ commits })).to.equal(expected)
-  })
-})
-
-describe('matches helper', () => {
-  const compileCommits = (matches) => Handlebars.compile(
-    '{{#each releases}}\n' +
-      '{{#each commits}}\n' +
-        matches +
-      '{{/each}}\n' +
-    '{{/each}}'
-  )
-
-  it('matches on field value', () => {
-    const matches =
-      '{{#matches href "12c0624"}}\n' +
-        '- {{message}}\n' +
-      '{{/matches}}\n'
-    const expected =
-      '- Commit that fixes nothing\n'
-    expect(compileCommits(matches)({ releases })).to.equal(expected)
-  })
-
-  it('matches with case insensitive flag', () => {
-    const matches =
-      '{{#matches author "pete" flags="i"}}\n' +
-        '- {{shorthash}}\n' +
-      '{{/matches}}\n'
-    const expected =
-      '- b0b3040\n' +
-      '- 12c0624\n' +
-      '- e9a43b2\n' +
-      '- 158fdde\n'
-    expect(compileCommits(matches)({ releases })).to.equal(expected)
-  })
-
-  it('provides non-matching conditional', () => {
-    const matches =
-      '{{#matches shorthash "e9a43b2"}}\n' +
-        '- HIT {{date}}\n' +
-      '{{else}}\n' +
-        '- MISS {{date}}\n' +
-      '{{/matches}}\n'
-    const expected =
-      '- MISS 2015-12-29T21:57:19.000Z\n' +
-      '- MISS 2015-12-29T21:18:19.000Z\n' +
-      '- HIT 2015-12-29T21:19:19.000Z\n' +
-      '- MISS 2015-12-14T17:06:12.000Z\n'
-    expect(compileCommits(matches)({ releases })).to.equal(expected)
-  })
-
-  it('matches on multiline content', () => {
-    const multiReleases = [{ commits: [
-      {
-        shorthash: 'c0f25d7',
-        message: 'Hello\n\nWorld\n\nBREAKING CHANGE: mock break\n\nsome more text'
-      }, {
-        shorthash: '12cd728',
-        message: 'Nope'
-      }
-    ]}]
-    const matches =
-      '{{#matches message "BREAKING CHANGE"}}\n' +
-        '- {{shorthash}}\n' +
-      '{{/matches}}\n'
-    const expected =
-      '- c0f25d7\n'
-    expect(compileCommits(matches)({ releases: multiReleases })).to.equal(expected)
-  })
-})