add File#exists?

[jhoblitt]

When doing negative testing, I.e., asserting that 'this' path should not exist, it would be more concise to be able to express this in serverspec as:

describe File('/does_not_exist') do
  it { should_not exist }
end

[jhoblitt]

Depends on mizzy/specinfra#384

[mizzy]

You should test file resource strictly with be_file, be_directory, be_socket and so on. exists? is a bit ambiguous. So I will not add exists? to file resource.

[jhoblitt]

@mizzy What is the correct way to test for the non-existence of a file?

[mizzy]

@jhoblitt Ah, I see what you would like to do with this PR. Please write your aim of the PR in the description field.

[jhoblitt]

@mizzy I thought it was just a simple oversight since many of the other base types respond to exists?. I've added a description to the PR, please let me know if that is sufficient A negative test isn't included because the test harness is hard-wired to have all the [specinfra] checks pass.

[mizzy]

Thanks for updating the description! It's sufficient.
Please do not mind about test. I understand it.

[mizzy]

Released as v2.17.0.

[jhoblitt]

@mizzy Thanks!

[conorsch]

@jhoblitt Thanks for this! It's frustrated me in the past, too.

[jhoblitt]

@conorsch your welcome!