spree/spree_volume_pricing — pull request #32 by iloveitaly (2012-10-12 20:33:53+00:00)
Title: Remove Order Decorator, 1.2 Specs Passing

* Dependency cleanup. Removed dependencies out of Gemfile per [this conversation](https://github.com/spree/spree_reviews/pull/32). Matched rspec + factory girl dependencies to what spree_core uses.
* Variant touch on volume price modification for robust caching support (fragment caching on product will work)
* Using the spree core `stub_authorization!` helper method (the new auth in 1.2 caused those tests to fail without using this method)

#### Removed Order#add_variant Override

Overriding the `add_variant` method isn't very future proof and doesn't play well when you want to use another extension which modifies LineItem price (e.g. [spree_sale_products](https://github.com/iloveitaly/spree_sale_products)).

I've switched to overriding `LineItem#copy_price` which is much DRYer, more futureproof, and allows for 'stacking' of `copy_price` overrides (like the spree_sale_products override).

This PR will fix #31, #29, #27, and includes the changes in #26.