If you remove this does any spec break? I wouldn't expect them to. This method is meant to be called within a controller context inside spree so we should be able to access both as string and symbols. Maybe you just need to turn them into symbols directly for consistence sake?