Skip to content

create tests/config.bats (fixes #2119) #2121

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

create tests/config.bats (fixes #2119) #2121

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6f6e624
update config.bats (fixes #2119)
rjpadilla Mar 22, 2021
4236e52
update config.bats (fixes #2119)
rjpadilla Mar 31, 2021
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
46 changes: 46 additions & 0 deletions tests/config.bats
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
#!/usr/bin/env bats
load test-helper

@test "$clinom config" {
skip "Run $clinom config with the [update|add|delete|clear] subcommands"
Copy link
Member

@JLKwong JLKwong Mar 26, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test could check for the 2 possible outputs of treehouses config:

Error: config file is empty; no variables to show

CONFIG_NAME=VARIABLE

run "${clicmd}" config
[ "$status" -eq 0 ]
}

@test "$clinom config update" {
run "${clicmd}" config update abc 123
if [[ "$output" == *"Error"* ]]; then
skip "Missing varname or varvalue"
fi
Copy link
Member

@JLKwong JLKwong Mar 26, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test could create a fake config variable like an apitoken for Gitter/Discord/Slack.

assert_success
}

@test "$clinom config add" {
run "${clicmd}" config add abc 123
if [[ "$output" == *"Error"* ]]; then
skip "Missing varname or varvalue"
fi
Copy link
Member

@JLKwong JLKwong Mar 26, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test could create a fake config variable like an apitoken for Gitter/Discord/Slack.

assert_success
}

@test "$clinom config delete" {
run "${clicmd}" config delete abc
if [[ "$output" == *"Error"* ]]; then
skip "Missing varname"
fi
Copy link
Member

@JLKwong JLKwong Mar 26, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test could create a fake config variable like an apitoken for Gitter/Discord/Slack.

assert_success
}

@test "$clinom config delete (non-existent user)" {
run "${clicmd}" config clear
run "${clicmd}" config delete abc
if [[ "$output" == *"Error"* ]]; then
skip "Can't delete a non-existent user"
fi
Copy link
Member

@JLKwong JLKwong Mar 26, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test could assert_output --partial for the error output.

assert_success
}

@test "$clinom config clear" {
run "${clicmd}" config clear
assert_success
}