| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
These are required by Ruby out of the box so lets avoid trying to
require them again.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
|
|
|
|
| |
Legally incredibly dubious, particularly since we don't follow it
strictly as policy, and we have git history instead, which does it right.
This is just a waste of time and a cargo cult.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
| |
since we use double quotes, be consistent everywhere.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
|
|
|
| |
We were trying to parse out a readme file that doesn't exist and the license file which produced errors
We namespaced the task in a way we don't do anywhere else. This matches our other gems now
Tell people to use bundler in the errors instead of gem installing things
We required rubygems which is not necessary
Signed-off-by: Tim Smith <tsmith@chef.io>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. Wait for the start callback rather than the do block, as it happens
later. This prevents us from getting into many conditions where the
start returns before the server is fully initialized, where if stop()
is called too soon after start returns, it would cause exceptions.
2. Wait for the thread to finish completely
3. Use Thread.current.abort_on_exception` to show the actual listen errors,
so that problems like this can be diagnosed.
4. Use :each to start and stop TinyServer so that race conditions like this
are exposed earlier and more often (this will let us rid ourselves of them
more quickly).
5. Use WEBrick::HTTPServer directly, which gets rid of the need for the
complicated trap workaround (rack is the one that traps).
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
4174 Style/SpaceInsideHashLiteralBraces
1860 Style/SpaceAroundOperators
1336 Style/SpaceInsideBlockBraces
1292 Style/AlignHash
997 Style/SpaceAfterComma
860 Style/SpaceAroundEqualsInParameterDefault
310 Style/EmptyLines
294 Style/IndentationConsistency
267 Style/TrailingWhitespace
238 Style/ExtraSpacing
212 Style/SpaceBeforeBlockBraces
166 Style/MultilineOperationIndentation
144 Style/TrailingBlankLines
120 Style/EmptyLineBetweenDefs
101 Style/IndentationWidth
82 Style/SpaceAroundBlockParameters
40 Style/EmptyLinesAroundMethodBody
29 Style/EmptyLinesAroundAccessModifier
1 Style/RescueEnsureAlignment
|
|
|
| |
Generated via git ls-files | xargs perl -pi -e "s/(Author.*?<[^@]+@)(?:opscode\\.com|getchef\\.com)(>)/\\1chef.io\\2/gi"
|
|
|
| |
Generated via git ls-files | xargs perl -pi -e "s/[Cc]opyright (?:\([Cc]\) )?((?\!$(date +%Y))\\d{4})(-\\d{4})?([, ][ \d]+)*(,|(?= ))/Copyright \\1-$(date +%Y),/g"
|
|
|
|
|
|
|
| |
This is an entirely mechanically generated (chefstyle -a) change, to go
along with chef/chefstyle#5 . We should pick something and use it
consistently, and my opinion is that double quotes are the appropriate
thing.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
AIX seems to raise Errno::EHOSTUNREACH when the server
is not ready yet. Our tests on AIX fail with:
```
Errno::EHOSTUNREACH:
No route to host - connect(2) for "localhost" port 9000
Shared Example Group: "a file with the wrong content" called from ./spec/support/shared/functional/file_resource.rb:936
Shared Example Group: "a configured file resource" called from ./spec/support/shared/functional/file_resource.rb:279
Shared Example Group: "a file resource" called from ./spec/functional/resource/remote_file_spec.rb:122
# ./spec/tiny_server.rb:91:in `started?'
# ./spec/tiny_server.rb:82:in `block in block_until_started'
# ./spec/tiny_server.rb:81:in `times'
# ./spec/tiny_server.rb:81:in `block_until_started'
# ./spec/tiny_server.rb:72:in `start'
# ./spec/support/shared/functional/http.rb:45:in `start_tiny_server'
# ./spec/functional/resource/remote_file_spec.rb:111:in `block (3 levels) in <top (required)>'
```
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
we can't fork off tinyserver without a lot of work in the test
codebase, so instead save a copy of the old rspec signal handler and then
after tinyserver has started (and we're sure that rack has scribbled
over the signal handler), restore the rspec signal handler.
there's an obvious race if someone hits ctrl-c after tinyserver has
scribbled over the sigint handler but before we restore it, but this is
99.9% better of a solution than nothing.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
EOFError sometimes occurs when running remote file tests with HTTPS,
should be a recoverable condition. This is not 100% verified because the
error is very intermittent.
|
|
|
|
|
|
| |
- Run standard file behavior functional tests with HTTPS on port 9000.
- Confirms CHEF-3005 fixed.
- Refactor tests to run against different server configurations.
|
| |
|
| |
|
| |
|
|
The opscode/chef repository now only contains the core Chef library code
used by chef-client, knife and chef-solo!
|