Fix angular specs by moving specs to the same place as the code was moved previously

Quite a few copy pasted specs were also removed
2026-02-24 01:13:21 +00:00 · 2019-11-03 11:40:27 +00:00
parent 7c54781d90
commit 537dbd3dc0
13 changed files with 226 additions and 246 deletions
--- a/spec/javascripts/unit/admin/order_cycles/controllers/create_spec.js.coffee
+++ b/spec/javascripts/unit/admin/order_cycles/controllers/create_spec.js.coffee
@@ -8,46 +8,23 @@ describe 'AdminCreateOrderCycleCtrl', ->

  beforeEach ->
    scope =
-      order_cycle_form: jasmine.createSpyObj('order_cycle_form', ['$dirty'])
      $watch: jasmine.createSpy('$watch')
-    event =
-      preventDefault: jasmine.createSpy('preventDefault')
    OrderCycle =
-      exchangeSelectedVariants: jasmine.createSpy('exchangeSelectedVariants').and.returnValue('variants selected')
-      productSuppliedToOrderCycle: jasmine.createSpy('productSuppliedToOrderCycle').and.returnValue('product supplied')
-      variantSuppliedToOrderCycle: jasmine.createSpy('variantSuppliedToOrderCycle').and.returnValue('variant supplied')
-      exchangeDirection: jasmine.createSpy('exchangeDirection').and.returnValue('exchange direction')
-      toggleProducts: jasmine.createSpy('toggleProducts')
-      setExchangeVariants: jasmine.createSpy('setExchangeVariants')
-      addSupplier: jasmine.createSpy('addSupplier')
-      addDistributor: jasmine.createSpy('addDistributor')
-      removeExchange: jasmine.createSpy('removeExchange')
-      addCoordinatorFee: jasmine.createSpy('addCoordinatorFee')
-      removeCoordinatorFee: jasmine.createSpy('removeCoordinatorFee')
-      addExchangeFee: jasmine.createSpy('addExchangeFee')
-      removeExchangeFee: jasmine.createSpy('removeExchangeFee')
-      removeDistributionOfVariant: jasmine.createSpy('removeDistributionOfVariant')
      create: jasmine.createSpy('create')
      new: jasmine.createSpy('new').and.returnValue "my order cycle"
    Enterprise =
      index: jasmine.createSpy('index').and.returnValue('enterprises list')
-      supplied_products: 'supplied products'
-      suppliedVariants: jasmine.createSpy('suppliedVariants').and.returnValue('supplied variants')
-      totalVariants: jasmine.createSpy('totalVariants').and.returnValue('variants total')
    EnterpriseFee =
      index: jasmine.createSpy('index').and.returnValue('enterprise fees list')
-      forEnterprise: jasmine.createSpy('forEnterprise').and.returnValue('enterprise fees for enterprise')
    ocInstance = {}

    module('admin.orderCycles')
    inject ($controller) ->
      ctrl = $controller 'AdminCreateOrderCycleCtrl', {$scope: scope, OrderCycle: OrderCycle, Enterprise: Enterprise, EnterpriseFee: EnterpriseFee, ocInstance: ocInstance}

-
-  it 'Loads enterprises and supplied products', ->
+  it 'Loads enterprises', ->
    expect(Enterprise.index).toHaveBeenCalled()
    expect(scope.enterprises).toEqual('enterprises list')
-    expect(scope.supplied_products).toEqual('supplied products')

  it 'Loads enterprise fees', ->
    expect(EnterpriseFee.index).toHaveBeenCalled()
@@ -56,104 +33,6 @@ describe 'AdminCreateOrderCycleCtrl', ->
  it 'Loads order cycles', ->
    expect(scope.order_cycle).toEqual('my order cycle')

-  describe 'Reporting when all resources are loaded', ->
-    beforeEach inject (RequestMonitor) ->
-      RequestMonitor.loading = false
-      Enterprise.loaded = true
-      EnterpriseFee.loaded = true
-      OrderCycle.loaded = true
-
-    it 'returns true when all resources are loaded', ->
-      expect(scope.loaded()).toBe(true)
-
-    it 'returns false otherwise', ->
-      EnterpriseFee.loaded = false
-      expect(scope.loaded()).toBe(false)
-
-  it "delegates suppliedVariants to Enterprise", ->
-    expect(scope.suppliedVariants('enterprise_id')).toEqual('supplied variants')
-    expect(Enterprise.suppliedVariants).toHaveBeenCalledWith('enterprise_id')
-
-  it 'Delegates exchangeSelectedVariants to OrderCycle', ->
-    expect(scope.exchangeSelectedVariants('exchange')).toEqual('variants selected')
-    expect(OrderCycle.exchangeSelectedVariants).toHaveBeenCalledWith('exchange')
-
-  it "delegates setExchangeVariants to OrderCycle", ->
-    scope.setExchangeVariants('exchange', 'variants', 'selected')
-    expect(OrderCycle.setExchangeVariants).toHaveBeenCalledWith('exchange', 'variants', 'selected')
-
-  it 'Delegates enterpriseTotalVariants to Enterprise', ->
-    expect(scope.enterpriseTotalVariants('enterprise')).toEqual('variants total')
-    expect(Enterprise.totalVariants).toHaveBeenCalledWith('enterprise')
-
-  it 'Delegates productSuppliedToOrderCycle to OrderCycle', ->
-    expect(scope.productSuppliedToOrderCycle('product')).toEqual('product supplied')
-    expect(OrderCycle.productSuppliedToOrderCycle).toHaveBeenCalledWith('product')
-
-  it 'Delegates variantSuppliedToOrderCycle to OrderCycle', ->
-    expect(scope.variantSuppliedToOrderCycle('variant')).toEqual('variant supplied')
-    expect(OrderCycle.variantSuppliedToOrderCycle).toHaveBeenCalledWith('variant')
-
-  it 'Delegates exchangeDirection to OrderCycle', ->
-    expect(scope.exchangeDirection('exchange')).toEqual('exchange direction')
-    expect(OrderCycle.exchangeDirection).toHaveBeenCalledWith('exchange')
-
-  it 'Finds enterprises participating in the order cycle that have fees', ->
-    scope.enterprises =
-      1: {id: 1, name: 'Eaterprises'}
-      2: {id: 2, name: 'Pepper Tree Place'}
-      3: {id: 3, name: 'South East'}
-    OrderCycle.participatingEnterpriseIds = jasmine.createSpy('participatingEnterpriseIds').and.returnValue([2])
-    EnterpriseFee.enterprise_fees = [ {enterprise_id: 2} ] # Pepper Tree Place has a fee
-    expect(scope.enterprisesWithFees()).toEqual([
-      {id: 2, name: 'Pepper Tree Place'}
-      ])
-
-  it 'Delegates enterpriseFeesForEnterprise to EnterpriseFee', ->
-    scope.enterpriseFeesForEnterprise('123')
-    expect(EnterpriseFee.forEnterprise).toHaveBeenCalledWith(123)
-
-  it 'Adds order cycle suppliers', ->
-    scope.new_supplier_id = 'new supplier id'
-    scope.addSupplier(event)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.addSupplier).toHaveBeenCalledWith('new supplier id')
-
-  it 'Adds order cycle distributors', ->
-    scope.new_distributor_id = 'new distributor id'
-    scope.addDistributor(event)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.addDistributor).toHaveBeenCalledWith('new distributor id')
-
-  it 'Removes order cycle exchanges', ->
-    scope.removeExchange(event, 'exchange')
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.removeExchange).toHaveBeenCalledWith('exchange')
-
-  it 'Adds coordinator fees', ->
-    scope.addCoordinatorFee(event)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.addCoordinatorFee).toHaveBeenCalled()
-
-  it 'Removes coordinator fees', ->
-    scope.removeCoordinatorFee(event, 0)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.removeCoordinatorFee).toHaveBeenCalledWith(0)
-
-  it 'Adds exchange fees', ->
-    scope.addExchangeFee(event)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.addExchangeFee).toHaveBeenCalled()
-
-  it 'Removes exchange fees', ->
-    scope.removeExchangeFee(event, 'exchange', 0)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.removeExchangeFee).toHaveBeenCalledWith('exchange', 0)
-
-  it 'Removes distribution of a variant', ->
-    scope.removeDistributionOfVariant('variant')
-    expect(OrderCycle.removeDistributionOfVariant).toHaveBeenCalledWith('variant')
-
  it 'Submits the order cycle via OrderCycle create', ->
    eventMock = {preventDefault: jasmine.createSpy()}
    scope.submit(eventMock,'/admin/order_cycles')
--- a/spec/javascripts/unit/admin/order_cycles/controllers/edit_spec.js.coffee
+++ b/spec/javascripts/unit/admin/order_cycles/controllers/edit_spec.js.coffee
@@ -18,29 +18,12 @@ describe 'AdminEditOrderCycleCtrl', ->
        'example.com/admin/order_cycles/27/edit'
    OrderCycle =
      load: jasmine.createSpy('load')
-      exchangeSelectedVariants: jasmine.createSpy('exchangeSelectedVariants').and.returnValue('variants selected')
-      productSuppliedToOrderCycle: jasmine.createSpy('productSuppliedToOrderCycle').and.returnValue('product supplied')
-      variantSuppliedToOrderCycle: jasmine.createSpy('variantSuppliedToOrderCycle').and.returnValue('variant supplied')
-      exchangeDirection: jasmine.createSpy('exchangeDirection').and.returnValue('exchange direction')
-      toggleProducts: jasmine.createSpy('toggleProducts')
-      setExchangeVariants: jasmine.createSpy('setExchangeVariants')
-      addSupplier: jasmine.createSpy('addSupplier')
-      addDistributor: jasmine.createSpy('addDistributor')
-      removeExchange: jasmine.createSpy('removeExchange')
-      addCoordinatorFee: jasmine.createSpy('addCoordinatorFee')
      removeCoordinatorFee: jasmine.createSpy('removeCoordinatorFee')
-      addExchangeFee: jasmine.createSpy('addExchangeFee')
-      removeExchangeFee: jasmine.createSpy('removeExchangeFee')
-      removeDistributionOfVariant: jasmine.createSpy('removeDistributionOfVariant')
      update: jasmine.createSpy('update')
    Enterprise =
      index: jasmine.createSpy('index').and.returnValue('enterprises list')
-      supplied_products: 'supplied products'
-      suppliedVariants: jasmine.createSpy('suppliedVariants').and.returnValue('supplied variants')
-      totalVariants: jasmine.createSpy('totalVariants').and.returnValue('variants total')
    EnterpriseFee =
      index: jasmine.createSpy('index').and.returnValue('enterprise fees list')
-      forEnterprise: jasmine.createSpy('forEnterprise').and.returnValue('enterprise fees for enterprise')
    ocInstance = {}

    module('admin.orderCycles')
@@ -50,7 +33,6 @@ describe 'AdminEditOrderCycleCtrl', ->
  it 'Loads enterprises and supplied products', ->
    expect(Enterprise.index).toHaveBeenCalled()
    expect(scope.enterprises).toEqual('enterprises list')
-    expect(scope.supplied_products).toEqual('supplied products')

  it 'Loads enterprise fees', ->
    expect(EnterpriseFee.index).toHaveBeenCalled()
@@ -59,107 +41,12 @@ describe 'AdminEditOrderCycleCtrl', ->
  it 'Loads order cycles', ->
    expect(OrderCycle.load).toHaveBeenCalledWith('27')

-  describe 'Reporting when all resources are loaded', ->
-    beforeEach inject (RequestMonitor) ->
-      RequestMonitor.loading = false
-      Enterprise.loaded = true
-      EnterpriseFee.loaded = true
-      OrderCycle.loaded = true
-
-    it 'returns true when all resources are loaded', ->
-      expect(scope.loaded()).toBe(true)
-
-    it 'returns false otherwise', ->
-      EnterpriseFee.loaded = false
-      expect(scope.loaded()).toBe(false)
-
-  it "delegates suppliedVariants to Enterprise", ->
-    expect(scope.suppliedVariants('enterprise_id')).toEqual('supplied variants')
-    expect(Enterprise.suppliedVariants).toHaveBeenCalledWith('enterprise_id')
-
-  it 'Delegates exchangeSelectedVariants to OrderCycle', ->
-    expect(scope.exchangeSelectedVariants('exchange')).toEqual('variants selected')
-    expect(OrderCycle.exchangeSelectedVariants).toHaveBeenCalledWith('exchange')
-
-  it "delegates setExchangeVariants to OrderCycle", ->
-    scope.setExchangeVariants('exchange', 'variants', 'selected')
-    expect(OrderCycle.setExchangeVariants).toHaveBeenCalledWith('exchange', 'variants', 'selected')
-
-  it 'Delegates totalVariants to Enterprise', ->
-    expect(scope.enterpriseTotalVariants('enterprise')).toEqual('variants total')
-    expect(Enterprise.totalVariants).toHaveBeenCalledWith('enterprise')
-
-  it 'Delegates productSuppliedToOrderCycle to OrderCycle', ->
-    expect(scope.productSuppliedToOrderCycle('product')).toEqual('product supplied')
-    expect(OrderCycle.productSuppliedToOrderCycle).toHaveBeenCalledWith('product')
-
-  it 'Delegates variantSuppliedToOrderCycle to OrderCycle', ->
-    expect(scope.variantSuppliedToOrderCycle('variant')).toEqual('variant supplied')
-    expect(OrderCycle.variantSuppliedToOrderCycle).toHaveBeenCalledWith('variant')
-
-  it 'Delegates exchangeDirection to OrderCycle', ->
-    expect(scope.exchangeDirection('exchange')).toEqual('exchange direction')
-    expect(OrderCycle.exchangeDirection).toHaveBeenCalledWith('exchange')
-
-  it 'Finds enterprises participating in the order cycle that have fees', ->
-    scope.enterprises =
-      1: {id: 1, name: 'Eaterprises'}
-      2: {id: 2, name: 'Pepper Tree Place'}
-      3: {id: 3, name: 'South East'}
-    OrderCycle.participatingEnterpriseIds = jasmine.createSpy('participatingEnterpriseIds').and.returnValue([2])
-    EnterpriseFee.enterprise_fees = [ {enterprise_id: 2} ] # Pepper Tree Place has a fee
-    expect(scope.enterprisesWithFees()).toEqual([
-      {id: 2, name: 'Pepper Tree Place'}
-      ])
-
-  it 'Delegates enterpriseFeesForEnterprise to EnterpriseFee', ->
-    scope.enterpriseFeesForEnterprise('123')
-    expect(EnterpriseFee.forEnterprise).toHaveBeenCalledWith(123)
-
-  it 'Adds order cycle suppliers', ->
-    scope.new_supplier_id = 'new supplier id'
-    scope.addSupplier(event)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.addSupplier).toHaveBeenCalledWith('new supplier id')
-
-  it 'Adds order cycle distributors', ->
-    scope.new_distributor_id = 'new distributor id'
-    scope.addDistributor(event)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.addDistributor).toHaveBeenCalledWith('new distributor id')
-
-  it 'Removes order cycle exchanges', ->
-    scope.removeExchange(event, 'exchange')
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.removeExchange).toHaveBeenCalledWith('exchange')
-    expect(scope.order_cycle_form.$dirty).toEqual true
-
-  it 'Adds coordinator fees', ->
-    scope.addCoordinatorFee(event)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.addCoordinatorFee).toHaveBeenCalled()
-
  it 'Removes coordinator fees', ->
    scope.removeCoordinatorFee(event, 0)
    expect(event.preventDefault).toHaveBeenCalled()
    expect(OrderCycle.removeCoordinatorFee).toHaveBeenCalledWith(0)
    expect(scope.order_cycle_form.$dirty).toEqual true

-  it 'Adds exchange fees', ->
-    scope.addExchangeFee(event)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.addExchangeFee).toHaveBeenCalled()
-
-  it 'Removes exchange fees', ->
-    scope.removeExchangeFee(event, 'exchange', 0)
-    expect(event.preventDefault).toHaveBeenCalled()
-    expect(OrderCycle.removeExchangeFee).toHaveBeenCalledWith('exchange', 0)
-    expect(scope.order_cycle_form.$dirty).toEqual true
-
-  it 'Removes distribution of a variant', ->
-    scope.removeDistributionOfVariant('variant')
-    expect(OrderCycle.removeDistributionOfVariant).toHaveBeenCalledWith('variant')
-
  it 'Submits the order cycle via OrderCycle update', ->
    eventMock = {preventDefault: jasmine.createSpy()}
    scope.submit(eventMock,'/admin/order_cycles')
--- a/spec/javascripts/unit/admin/order_cycles/controllers/incoming_controller_spec.js.coffee
+++ b/spec/javascripts/unit/admin/order_cycles/controllers/incoming_controller_spec.js.coffee
@@ -0,0 +1,26 @@
+describe 'AdminOrderCycleIncomingCtrl', ->
+  ctrl = null
+  scope = null
+  event = null
+  location = null
+  OrderCycle = null
+  Enterprise = null
+  EnterpriseFee = null
+
+  beforeEach ->
+    scope =
+      $watch: jasmine.createSpy('$watch')
+    location =
+      absUrl: ->
+        'example.com/admin/order_cycles/27/edit'
+    Enterprise =
+      totalVariants: jasmine.createSpy('totalVariants').and.returnValue('variants total')
+    ocInstance = {}
+
+    module('admin.orderCycles')
+    inject ($controller) ->
+      ctrl = $controller 'AdminOrderCycleIncomingCtrl', {$scope: scope, $location: location, OrderCycle: OrderCycle, Enterprise: Enterprise, EnterpriseFee: EnterpriseFee, ocInstance: ocInstance}
+
+  it 'Delegates totalVariants to Enterprise', ->
+    expect(scope.enterpriseTotalVariants('enterprise')).toEqual('variants total')
+    expect(Enterprise.totalVariants).toHaveBeenCalledWith('enterprise')
--- a/spec/javascripts/unit/admin/order_cycles/controllers/order_cycle_basic_controller_spec.js.coffee
+++ b/spec/javascripts/unit/admin/order_cycles/controllers/order_cycle_basic_controller_spec.js.coffee
@@ -0,0 +1,63 @@
+describe 'AdminOrderCycleBasicCtrl', ->
+  ctrl = null
+  scope = null
+  event = null
+  location = null
+  OrderCycle = null
+  Enterprise = null
+  EnterpriseFee = null
+
+  beforeEach ->
+    scope =
+      $watch: jasmine.createSpy('$watch')
+    event =
+      preventDefault: jasmine.createSpy('preventDefault')
+    OrderCycle =
+      setExchangeVariants: jasmine.createSpy('setExchangeVariants')
+      addCoordinatorFee: jasmine.createSpy('addCoordinatorFee')
+      removeCoordinatorFee: jasmine.createSpy('removeCoordinatorFee')
+    Enterprise =
+      suppliedVariants: jasmine.createSpy('suppliedVariants').and.returnValue('supplied variants')
+    EnterpriseFee =
+      forEnterprise: jasmine.createSpy('forEnterprise').and.returnValue('enterprise fees for enterprise')
+    ocInstance = {}
+
+    module('admin.orderCycles')
+    inject ($controller) ->
+      ctrl = $controller 'AdminOrderCycleBasicCtrl', {$scope: scope, $location: location, OrderCycle: OrderCycle, Enterprise: Enterprise, EnterpriseFee: EnterpriseFee, ocInstance: ocInstance}
+
+  describe 'Reporting when all resources are loaded', ->
+    beforeEach inject (RequestMonitor) ->
+      RequestMonitor.loading = false
+      Enterprise.loaded = true
+      EnterpriseFee.loaded = true
+      OrderCycle.loaded = true
+
+    it 'returns true when all resources are loaded', ->
+      expect(scope.loaded()).toBe(true)
+
+    it 'returns false otherwise', ->
+      EnterpriseFee.loaded = false
+      expect(scope.loaded()).toBe(false)
+
+  it "delegates suppliedVariants to Enterprise", ->
+    expect(scope.suppliedVariants('enterprise_id')).toEqual('supplied variants')
+    expect(Enterprise.suppliedVariants).toHaveBeenCalledWith('enterprise_id')
+
+  it "delegates setExchangeVariants to OrderCycle", ->
+    scope.setExchangeVariants('exchange', 'variants', 'selected')
+    expect(OrderCycle.setExchangeVariants).toHaveBeenCalledWith('exchange', 'variants', 'selected')
+
+  it 'Delegates enterpriseFeesForEnterprise to EnterpriseFee', ->
+    scope.enterpriseFeesForEnterprise('123')
+    expect(EnterpriseFee.forEnterprise).toHaveBeenCalledWith(123)
+
+  it 'Adds coordinator fees', ->
+    scope.addCoordinatorFee(event)
+    expect(event.preventDefault).toHaveBeenCalled()
+    expect(OrderCycle.addCoordinatorFee).toHaveBeenCalled()
+
+  it 'Removes coordinator fees', ->
+    scope.removeCoordinatorFee(event, 0)
+    expect(event.preventDefault).toHaveBeenCalled()
+    expect(OrderCycle.removeCoordinatorFee).toHaveBeenCalledWith(0)
--- a/spec/javascripts/unit/admin/order_cycles/controllers/order_cycle_exchanges_controller_spec.js.coffee
+++ b/spec/javascripts/unit/admin/order_cycles/controllers/order_cycle_exchanges_controller_spec.js.coffee
@@ -0,0 +1,95 @@
+describe 'AdminOrderCycleExchangesCtrl', ->
+  ctrl = null
+  scope = null
+  event = null
+  location = null
+  OrderCycle = null
+  Enterprise = null
+  EnterpriseFee = null
+
+  beforeEach ->
+    scope =
+      order_cycle_form: jasmine.createSpyObj('order_cycle_form', ['$dirty', '$setPristine'])
+      $watch: jasmine.createSpy('$watch')
+    event =
+      preventDefault: jasmine.createSpy('preventDefault')
+    location =
+      absUrl: ->
+        'example.com/admin/order_cycles/27/edit'
+    OrderCycle =
+      exchangeSelectedVariants: jasmine.createSpy('exchangeSelectedVariants').and.returnValue('variants selected')
+      exchangeDirection: jasmine.createSpy('exchangeDirection').and.returnValue('exchange direction')
+      addSupplier: jasmine.createSpy('addSupplier')
+      addDistributor: jasmine.createSpy('addDistributor')
+      removeExchange: jasmine.createSpy('removeExchange')
+      addExchangeFee: jasmine.createSpy('addExchangeFee')
+      removeExchangeFee: jasmine.createSpy('removeExchangeFee')
+      removeDistributionOfVariant: jasmine.createSpy('removeDistributionOfVariant')
+    Enterprise =
+      supplied_products: 'supplied products'
+    EnterpriseFee =
+      forEnterprise: jasmine.createSpy('forEnterprise').and.returnValue('enterprise fees for enterprise')
+    ocInstance = {}
+
+    module('admin.orderCycles')
+    inject ($controller) ->
+      ctrl = $controller 'AdminOrderCycleExchangesCtrl', {$scope: scope, $location: location, OrderCycle: OrderCycle, Enterprise: Enterprise, EnterpriseFee: EnterpriseFee, ocInstance: ocInstance}
+
+  it 'Loads supplied products', ->
+    expect(scope.supplied_products).toEqual('supplied products')
+
+  it 'Delegates exchangeSelectedVariants to OrderCycle', ->
+    expect(scope.exchangeSelectedVariants('exchange')).toEqual('variants selected')
+    expect(OrderCycle.exchangeSelectedVariants).toHaveBeenCalledWith('exchange')
+
+  it 'Delegates exchangeDirection to OrderCycle', ->
+    expect(scope.exchangeDirection('exchange')).toEqual('exchange direction')
+    expect(OrderCycle.exchangeDirection).toHaveBeenCalledWith('exchange')
+
+  it 'Finds enterprises participating in the order cycle that have fees', ->
+    # this is inhetited from order_cycle_basic_controller
+    scope.enterpriseFeesForEnterprise = (enterprise_id) ->
+      EnterpriseFee.forEnterprise(parseInt(enterprise_id))
+    scope.enterprises =
+      1: {id: 1, name: 'Eaterprises'}
+      2: {id: 2, name: 'Pepper Tree Place'}
+      3: {id: 3, name: 'South East'}
+    OrderCycle.participatingEnterpriseIds = jasmine.createSpy('participatingEnterpriseIds').and.returnValue([2])
+    EnterpriseFee.enterprise_fees = [ {enterprise_id: 2} ] # Pepper Tree Place has a fee
+    expect(scope.enterprisesWithFees()).toEqual([
+      {id: 2, name: 'Pepper Tree Place'}
+      ])
+
+  it 'Removes order cycle exchanges', ->
+    scope.removeExchange(event, 'exchange')
+    expect(event.preventDefault).toHaveBeenCalled()
+    expect(OrderCycle.removeExchange).toHaveBeenCalledWith('exchange')
+    expect(scope.order_cycle_form.$dirty).toEqual true
+
+  it 'Adds exchange fees', ->
+    scope.addExchangeFee(event)
+    expect(event.preventDefault).toHaveBeenCalled()
+    expect(OrderCycle.addExchangeFee).toHaveBeenCalled()
+
+  it 'Removes exchange fees', ->
+    scope.removeExchangeFee(event, 'exchange', 0)
+    expect(event.preventDefault).toHaveBeenCalled()
+    expect(OrderCycle.removeExchangeFee).toHaveBeenCalledWith('exchange', 0)
+    expect(scope.order_cycle_form.$dirty).toEqual true
+
+  it 'Adds order cycle suppliers', ->
+    scope.new_supplier_id = 'new supplier id'
+    scope.addSupplier(event)
+    expect(event.preventDefault).toHaveBeenCalled()
+    expect(OrderCycle.addSupplier).toHaveBeenCalledWith('new supplier id')
+
+  it 'Adds order cycle distributors', ->
+    scope.new_distributor_id = 'new distributor id'
+    scope.addDistributor(event)
+    expect(event.preventDefault).toHaveBeenCalled()
+    expect(OrderCycle.addDistributor).toHaveBeenCalledWith('new distributor id')
+
+  it 'Removes distribution of a variant', ->
+    scope.removeDistributionOfVariant('variant')
+    expect(OrderCycle.removeDistributionOfVariant).toHaveBeenCalledWith('variant')
+
--- a/spec/javascripts/unit/admin/order_cycles/controllers/outgoing_controller_spec.js.coffee
+++ b/spec/javascripts/unit/admin/order_cycles/controllers/outgoing_controller_spec.js.coffee
@@ -0,0 +1,31 @@
+describe 'AdminOrderCycleOutgoingCtrl', ->
+  ctrl = null
+  scope = null
+  event = null
+  location = null
+  OrderCycle = null
+  Enterprise = null
+  EnterpriseFee = null
+
+  beforeEach ->
+    scope =
+      $watch: jasmine.createSpy('$watch')
+    location =
+      absUrl: ->
+        'example.com/admin/order_cycles/27/edit'
+    OrderCycle =
+      productSuppliedToOrderCycle: jasmine.createSpy('productSuppliedToOrderCycle').and.returnValue('product supplied')
+      variantSuppliedToOrderCycle: jasmine.createSpy('variantSuppliedToOrderCycle').and.returnValue('variant supplied')
+    ocInstance = {}
+
+    module('admin.orderCycles')
+    inject ($controller) ->
+      ctrl = $controller 'AdminOrderCycleOutgoingCtrl', {$scope: scope, $location: location, OrderCycle: OrderCycle, Enterprise: Enterprise, EnterpriseFee: EnterpriseFee, ocInstance: ocInstance}
+
+  it 'Delegates productSuppliedToOrderCycle to OrderCycle', ->
+    expect(scope.productSuppliedToOrderCycle('product')).toEqual('product supplied')
+    expect(OrderCycle.productSuppliedToOrderCycle).toHaveBeenCalledWith('product')
+
+  it 'Delegates variantSuppliedToOrderCycle to OrderCycle', ->
+    expect(scope.variantSuppliedToOrderCycle('variant')).toEqual('variant supplied')
+    expect(OrderCycle.variantSuppliedToOrderCycle).toHaveBeenCalledWith('variant')