```ruby
require 'spec_helper'

describe Spree::Admin::VariantsController do
  let(:user) { mock_model Spree::User }
  before do
    controller.stub :current_user => user
    user.stub :has_role? => true
  end

  describe "PUT #update" do
    it "creates a volume price" do
      @variant = Factory :variant, :price => 10
      put :update,
        :product_id => @variant.product.permalink,
        :id => @variant.id,
        :variant => {
          "volume_prices_attributes" => {
            "1335830259720" => {
              "name"=>"5-10",
              "range"=>"5..10",
              "amount"=>"90",
              "position"=>"1",
              "_destroy"=>"false"
            }
          }
        }

      Spree::VolumePrice.last.name.should eql("5-10")
    end
  end
end
```

Make Spree::Admin::VariantsController spec less dumb
user
```ruby
require 'spec_helper'

describe Spree::Admin::VariantsController do
  let(:user) { mock_model Spree::User }
  before do
    controller.stub :current_user => user
    user.stub :has_role? => true
  end

  describe "PUT #update" do
    it "creates a volume price" do
      variant = Factory :variant

      lambda do
        put :update,
          :product_id => variant.product.permalink,
          :id => variant.id,
          :variant => {
            "volume_prices_attributes" => {
              "1335830259720" => {
                "name"=>"5-10",
                "range"=>"5..10",
                "amount"=>"90",
                "position"=>"1",
                "_destroy"=>"false"
              }
            }
          }
      end.should change(variant.volume_prices, :count).by(1)
    end
  end
end
```
assistant