Allow for inspection of environment and config_path.

[damncabbage]

As part of some tests I have, I need to be able to temporarily swap out the environment and config_path, eg.

def modify_app_constants(&block)
  old_config_path = AppConstants.config_path
  old_environment = AppConstants.environment
  yield if block_given?
  AppConstants.config_path = old_config_path
  AppConstants.environment = old_environment
  AppConstants.load!
end

There's no way to do this at present; this pull requests adds a couple of attribute readers (with tests) for this purpose.

[damncabbage]

It's up to you as to whether or not you want to accept this, but we're switching our tests to stubbing methods on the AppConstants class directly when our tests need to test code that calls it directly. In case it's useful to anyone:

before { AppConstants.stub(:gitBinary, "/usr/local/bin/gitwrapper.sh") }
it "uses the gitBinary setting when constructing commands" do
  Shell.should_receive(:run).with('/usr/local/bin/gitwrapper.sh clone /foo/bar/.git /baz/destination')
  Git.clone('/foo/bar', '/baz/destination')
end

[theleoborges]

Hey @damncabbage ,

Sorry about the delay but I've been flat out in the past couple of weeks. I'll have a closer look at your PR this week and let you know about it.

Thanks!

[theleoborges]

Well there's no excuse I left you hanging on this :(

Anyway, I just merged your PR and will make a new release, 1.0.5, so you can update. I received another PR recently so another update might be coming as well.

Thank you for taking the time to submit a PR.

[theleoborges]

Done. New release is up: 1.0.5