Buildpack blobstore deletion is now async

  * instead of using Controller #after_destroy hook

[#60026682]

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Refactor specs

  * prefer let() to instance variables
  * cleanup naming
  * make DelayedJob expectations more readable

[#60026682]

Signed-off-by: Aram Price <aprice@pivotallabs.com>
RestController no longer supports #after_destroy

[#60026682]

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Inline callbacks where we override ModelController

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Revert "RestController no longer supports #after_destroy"

This reverts commit 76d17442582ff90bd357be54045f9674502f467e.

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Cleanup style and convert hashes to 1.9 syntax

Signed-off-by: Aram Price <aprice@pivotallabs.com>
ModelController has #recursive? method

Signed-off-by: Aram Price <aprice@pivotallabs.com>
ModelController has #async? method

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Stop stubbing #v2_api? on the class under test

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Use idiomatic ruby and be less verbose

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Cleanup naming and use ruby-isms

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Move #async? and #recursive? methods to Base

  * ServicesController can not see methods on ModelController

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Limit log file size, and default to info
Operator and broker can now update unique_id

[Finishes #61090068]
Merge remote-tracking branch 'upstream/master'