Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: explicitly run cli with node to fix tests on windows #53

Merged
merged 1 commit into from
Jun 8, 2016
Merged

test: explicitly run cli with node to fix tests on windows #53

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
test: explicitly run cli with node to fix tests on windows 
Explicitly run the cli with 'node <index.js path>' to fix running tests on windows.

Fixes #50
  • Loading branch information
@Tapppi
Tapppi committed Jun 8, 2016
commit afef731b43a54d8db5edd3f88a851c021e9a9f3e
28 changes: 16 additions & 12 deletions test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,10 @@ function commit (msg) {
shell.exec('git commit --allow-empty -m"' + msg + '"')
}

function execCli (argString) {
return shell.exec('node ' + cliPath + (argString != null ? ' ' + argString : ''))
}

function writePackageJson (version) {
fs.writeFileSync('package.json', JSON.stringify({
version: version
Expand Down Expand Up @@ -48,7 +52,7 @@ describe('cli', function () {
shell.exec('git tag -a v1.0.0 -m "my awesome first release"')
commit('fix: patch release')

shell.exec(cliPath).code.should.equal(0)
execCli().code.should.equal(0)

var content = fs.readFileSync('CHANGELOG.md', 'utf-8')
content.should.match(/patch release/)
Expand All @@ -60,7 +64,7 @@ describe('cli', function () {

commit('feat: first commit')
commit('fix: patch release')
shell.exec(cliPath + ' --first-release').code.should.equal(0)
execCli('--first-release').code.should.equal(0)

var content = fs.readFileSync('CHANGELOG.md', 'utf-8')
content.should.match(/patch release/)
Expand All @@ -78,7 +82,7 @@ describe('cli', function () {
shell.exec('git tag -a v1.0.0 -m "my awesome first release"')
commit('fix: patch release')

shell.exec(cliPath).code.should.equal(0)
execCli().code.should.equal(0)
var content = fs.readFileSync('CHANGELOG.md', 'utf-8')
content.should.match(/1\.0\.1/)
content.should.not.match(/legacy header format/)
Expand All @@ -92,7 +96,7 @@ describe('cli', function () {
.then(function (unmock) {
writePackageJson('1.0.0')

shell.exec(cliPath + ' --sign').code.should.equal(0)
execCli('--sign').code.should.equal(0)

var captured = shell.cat('gitcapture.log').stdout.split('\n').map(function (line) {
return line ? JSON.parse(line) : line
Expand All @@ -110,7 +114,7 @@ describe('cli', function () {
.then(function (unmock) {
writePackageJson('1.0.0')

var result = shell.exec(cliPath)
var result = execCli()
result.code.should.equal(1)
result.stdout.should.match(/commit yourself/)

Expand All @@ -124,7 +128,7 @@ describe('cli', function () {
.then(function (unmock) {
writePackageJson('1.0.0')

var result = shell.exec(cliPath)
var result = execCli()
result.code.should.equal(1)
result.stdout.should.match(/addition is hard/)

Expand All @@ -138,7 +142,7 @@ describe('cli', function () {
.then(function (unmock) {
writePackageJson('1.0.0')

var result = shell.exec(cliPath)
var result = execCli()
result.code.should.equal(1)
result.stdout.should.match(/tag, you're it/)

Expand All @@ -154,7 +158,7 @@ describe('cli', function () {
shell.exec('git tag -a v1.0.0 -m "my awesome first release"')
commit('fix: this is my fairly long commit message which is testing whether or not we allow for long commit messages')

shell.exec(cliPath).code.should.equal(0)
execCli().code.should.equal(0)

var content = fs.readFileSync('CHANGELOG.md', 'utf-8')
content.should.match(/this is my fairly long commit message which is testing whether or not we allow for long commit messages/)
Expand All @@ -167,7 +171,7 @@ describe('cli', function () {
shell.exec('git tag -a v1.0.0 -m "my awesome first release"')
commit('feat: new feature!')

shell.exec(cliPath).code.should.equal(0)
execCli().code.should.equal(0)

// check last commit message
shell.exec('git log --oneline -n1').stdout.should.match(/chore\(release\): 1\.1\.0/)
Expand All @@ -178,7 +182,7 @@ describe('cli', function () {
it('appends line feed at end of package.json', function () {
writePackageJson('1.0.0')

shell.exec(cliPath).code.should.equal(0)
execCli().code.should.equal(0)

var pkgJson = fs.readFileSync('package.json', 'utf-8')
pkgJson.should.equal(['{', ' "version": "1.0.1"', '}', ''].join('\n'))
Expand All @@ -189,8 +193,8 @@ describe('cli', function () {
writeGitPreCommitHook()

commit('feat: first commit')
shell.exec(cliPath + ' --no-verify').code.should.equal(0)
execCli('--no-verify').code.should.equal(0)
commit('feat: second commit')
shell.exec(cliPath + ' -n').code.should.equal(0)
execCli('-n').code.should.equal(0)
})
})