Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
13 changes: 12 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -104,4 +104,15 @@ For full application setup, see the [detailed instructions in the wiki](https://
Use the `--debug` switch to enable stacktraces and debug output.

## Contributing
Pull requests, bug reports, and feature suggestions are welcome! Before starting on a contribution, we recommend opening an issue or replying to an existing one to give others some initial context on the work needing to be done.
Pull requests, bug reports, and feature suggestions are welcome!

Before starting on a contribution, we recommend opening an issue or replying to an existing one to give others some initial context on the work needing to be done.

**Specs must pass on pull requests for them to be considered.**

### Running Specs
Broadside has a lot of tests for most of its behaviors - just run
```
bundle exec rspec
```
in the broadside directory. Don't open pull requests without passing specs.
7 changes: 7 additions & 0 deletions spec/broadside_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,13 @@
describe Broadside do
include_context 'deploy configuration'

it 'should be able to display the help menu' do
silence_warnings do
exit_value = system('bundle exec broadside --help >/dev/null')
expect(exit_value).to be_truthy
end
end

describe '#load_config_file' do
it 'calls load for both the system and app config files' do
expect(Broadside).to receive(:load).with(system_config_path).ordered
Expand Down