Skip to content

Conversation

kbrock
Copy link
Member

@kbrock kbrock commented Mar 7, 2014

When the os can't be found, we tend to get '--arg' is not a valid command.

This raises an exception to say that the command is not found.

issue: not sure the best way to communicate this with the calling app.

Copy link
Member

Choose a reason for hiding this comment

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

Since this is being done everywhere, does it make sense to stub in a global before in the spec_helper...e.g.

RSpec.configure do |config|
  config.before do
    stub_distro
  end
end

@cfme-bot
Copy link
Member

cfme-bot commented Mar 7, 2014

Checked commit kbrock@ad5cdee with rubocop
12 files checked, 2 offenses detected

This comment is on an outdated set of commits.

Copy link
Member

Choose a reason for hiding this comment

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

Prefer expect {}.to

@kbrock
Copy link
Member Author

kbrock commented Mar 8, 2014

@Fryguy Do we even want this to raise errors?

@cfme-bot
Copy link
Member

cfme-bot commented Mar 8, 2014

Checked commit kbrock@3fd65ca with rubocop
5 files checked, 1 offense detected

This comment is on an outdated set of commits.

The various operating systems have defined specific commands.
When it is not found on a particular OS, then the error message is confusing

- better stubbing support
@cfme-bot
Copy link
Member

cfme-bot commented Mar 8, 2014

Checked commit kbrock@3747f89 with rubocop
5 files checked, 0 offenses detected

Everything looks good. 👍

@cfme-bot
Copy link
Member

This pull request is not mergeable. Please rebase and repush.

@kbrock
Copy link
Member Author

kbrock commented Mar 30, 2014

Errors aren't going to creep up like before if we use PATH. I prefer to go the #96 route.
Also, due to mo's changes, this was no longer able to be merged.

@kbrock kbrock closed this Mar 30, 2014
@kbrock kbrock deleted the warn_missing_commands branch March 30, 2014 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants