delta/ruby-gems/chef.git/spec/unit/runner_spec.rb, branch https

Remove copyright dates

2020-04-13T22:47:17+00:00

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

Style/SymbolArray

2019-07-02T22:11:54+00:00

start enforcing using %i{} instead of arrays of symbols

Signed-off-by: Lamont Granquist

fix Layout/IndentHeredoc

2018-07-02T17:27:03+00:00

Signed-off-by: Lamont Granquist

add a pointer from the run_context back to the runner

2017-04-20T16:40:31+00:00

because surfing through Objectspace makes kittens cry

Signed-off-by: Lamont Granquist

Don't `rescue Exception` for retriable resources

2017-03-27T08:15:35+00:00

Without this, sending an interrupt signal (hitting Ctrl+C) during a
chef-client run that is currently executing a resources that has
`retries` set will not interrupt the run, although the interrupt handler
set up in Chef::Application calls Process.exit, which raises SystemExit.

If the retryable resource is shelling out, the spawned process also gets
the INT signal and will stop -- however, the chef-run would go on,
merely noticing that the resource has failed and theres X-1 retries
left.

It is imaginable that users depend on some (whacky) Ruby script in a
retriable resource that raises SystemExit, and they want this to _not_
stop the chef-run. So it's up for debate if we just want to merge this
or make it configurable.

In general, I'd like Ctrl+C to _stop my chef-run_, but this expectation
could also be misled.

When adapting tests to this change, it became apparent that for some
reason the runner_spec.rb test didn't behave as expected. I've thus made
those tests a little stricter by replacing some allows by expects.

Signed-off-by: Stephan Renatus

Chef-13: remove Chef::Platform.set and related methods

2017-03-17T19:35:58+00:00

Switch over the Chef-12.0 ProviderResolver is now completed.

Signed-off-by: Lamont Granquist

Properly deprecate old Chef::Platform methods

2016-12-07T21:42:39+00:00

These should have been deprecated formally in 12.5.1 after the
Dynamic Provider-Resolver'ing of the internal chef resources
was completed.

Signed-off-by: Lamont Granquist

fix Style/BlockDelimiters, Style/MultilineBlockLayout and 0.42.0 engine upgrade

2016-08-17T20:25:02+00:00

Signed-off-by: Lamont Granquist

flip multiline function arguments around

2016-02-19T17:28:33+00:00

no enforced trailing comma on arguments...

autofixing whitespace cops

2016-02-05T23:00:00+00:00

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