From b6d63d40c6d877ce39f2c2b48f9ef289db1cb3ae Mon Sep 17 00:00:00 2001
From: Rob Harrington <oeoeaio@gmail.com>
Date: Wed, 6 May 2015 22:35:43 +1000
Subject: [PATCH] Pulling columns logic out into its own services/controller in
 a new indexUtils module

---
 .../controllers/columns_controller.js.coffee    |  4 ++++
 .../admin/index_utils/index_utils.js.coffee     |  1 +
 .../index_utils/services/columns.js.coffee      |  8 ++++++++
 .../columns_controller_spec.js.coffee           | 17 +++++++++++++++++
 .../index_utils/services/columns.js.coffee      | 15 +++++++++++++++
 5 files changed, 45 insertions(+)
 create mode 100644 app/assets/javascripts/admin/index_utils/controllers/columns_controller.js.coffee
 create mode 100644 app/assets/javascripts/admin/index_utils/index_utils.js.coffee
 create mode 100644 app/assets/javascripts/admin/index_utils/services/columns.js.coffee
 create mode 100644 spec/javascripts/unit/admin/index_utils/controllers/columns_controller_spec.js.coffee
 create mode 100644 spec/javascripts/unit/admin/index_utils/services/columns.js.coffee

diff --git a/app/assets/javascripts/admin/index_utils/controllers/columns_controller.js.coffee b/app/assets/javascripts/admin/index_utils/controllers/columns_controller.js.coffee
new file mode 100644
index 0000000000..39556983b3
--- /dev/null
+++ b/app/assets/javascripts/admin/index_utils/controllers/columns_controller.js.coffee
@@ -0,0 +1,4 @@
+angular.module("admin.indexUtils").controller "ColumnsCtrl", ($scope, Columns) ->
+  $scope.columns = Columns.columns
+  $scope.predicate = ""
+  $scope.reverse = false
diff --git a/app/assets/javascripts/admin/index_utils/index_utils.js.coffee b/app/assets/javascripts/admin/index_utils/index_utils.js.coffee
new file mode 100644
index 0000000000..eef7b0aa5f
--- /dev/null
+++ b/app/assets/javascripts/admin/index_utils/index_utils.js.coffee
@@ -0,0 +1 @@
+angular.module("admin.indexUtils", [])
\ No newline at end of file
diff --git a/app/assets/javascripts/admin/index_utils/services/columns.js.coffee b/app/assets/javascripts/admin/index_utils/services/columns.js.coffee
new file mode 100644
index 0000000000..46e7ccd3b8
--- /dev/null
+++ b/app/assets/javascripts/admin/index_utils/services/columns.js.coffee
@@ -0,0 +1,8 @@
+angular.module("admin.indexUtils").factory 'Columns', ->
+  new class Columns
+    columns: {}
+
+    setColumns: (columns) ->
+      @columns = {}
+      @columns[name] = column for name, column of columns
+      @columns
diff --git a/spec/javascripts/unit/admin/index_utils/controllers/columns_controller_spec.js.coffee b/spec/javascripts/unit/admin/index_utils/controllers/columns_controller_spec.js.coffee
new file mode 100644
index 0000000000..5fd79e71bf
--- /dev/null
+++ b/spec/javascripts/unit/admin/index_utils/controllers/columns_controller_spec.js.coffee
@@ -0,0 +1,17 @@
+describe "ColumnsCtrl", ->
+  ctrl = null
+  scope = null
+  Columns = null
+
+  beforeEach ->
+    Columns = { columns: { name: { visible: true} } }
+
+    module('admin.indexUtils')
+    inject ($controller, $rootScope) ->
+      scope = $rootScope
+      ctrl = $controller 'ColumnsCtrl', {$scope: scope, Columns: Columns}
+
+  it "initialises data", ->
+    expect(scope.columns).toEqual Columns.columns
+    expect(scope.predicate).toEqual ""
+    expect(scope.reverse).toEqual false
diff --git a/spec/javascripts/unit/admin/index_utils/services/columns.js.coffee b/spec/javascripts/unit/admin/index_utils/services/columns.js.coffee
new file mode 100644
index 0000000000..0b6093f31d
--- /dev/null
+++ b/spec/javascripts/unit/admin/index_utils/services/columns.js.coffee
@@ -0,0 +1,15 @@
+describe "Columns service", ->
+  Columns = null
+
+  beforeEach ->
+    module 'admin.indexUtils'
+
+    inject (_Columns_) ->
+      Columns = _Columns_
+
+    Columns.columns = ["something"]
+
+  describe "setting columns", ->
+    it "sets resets @columns and copies each column of the provided object across", ->
+      Columns.setColumns({ name: { visible: true } })
+      expect(Columns.columns).toEqual { name: { visible: true } }