show more than 20 feature flags

test plan:
* visit the site admin feature flags
* should show all of the feature flags
 (particularly "Differentiated Assignments")

closes #CNVS-19721

Change-Id: Ibf5d864986713cc7aff6f5d4bdc88a608916fe90
Reviewed-on: https://gerrit.instructure.com/51882
Tested-by: Jenkins
Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
Product-Review: Jeremy Stanley <jeremy@instructure.com>
QA-Review: Clare Strong <clare@instructure.com>

Author: maneframe

app/coffeescripts/bundles/account_settings.coffee

@@ -4,4 +4,4 @@ require [
   'compiled/bundles/modules/account_quota_settings'
 ], (FeatureFlagAdminView) ->
   featureFlags = new FeatureFlagAdminView(el: '#tab-features')
-  featureFlags.collection.fetch()
+  featureFlags.collection.fetchAll()

app/coffeescripts/bundles/course_settings.coffee

@@ -13,7 +13,7 @@ require [
     el: $('#tab-navigation')
 
   featureFlagView = new FeatureFlagAdminView(el: '#tab-features')
-  featureFlagView.collection.fetch()
+  featureFlagView.collection.fetchAll()
 
   $ ->
     nav_view.render()

app/coffeescripts/bundles/profile.coffee

@@ -8,4 +8,4 @@ require [
 ], (FeatureFlagAdminView) ->
 
   view = new FeatureFlagAdminView(el: '.feature-flag-wrapper')
-  view.collection.fetch()
+  view.collection.fetchAll()

app/coffeescripts/collections/FeatureFlagCollection.coffee

@@ -1,13 +1,23 @@
 define [
   'underscore',
   'Backbone',
+  'compiled/collections/PaginatedCollection'
   'compiled/models/FeatureFlag'
-], (_, Backbone, FeatureFlag) ->
+], (_, Backbone, PaginatedCollection, FeatureFlag) ->
 
-  class FeatureFlagCollection extends Backbone.Collection
+  class FeatureFlagCollection extends PaginatedCollection
 
     model: FeatureFlag
 
+    fetchAll: ->
+      @fetch(success: @fetchNext)
+
+    fetchNext: =>
+      if @canFetch 'next'
+        @fetch(page: 'next', success: @fetchNext)
+      else
+        @trigger('finish')
+
     fetch: (options = {}) ->
       options.data = _.extend per_page: 20, options.data || {}
       super options

app/coffeescripts/views/feature_flags/FeatureFlagAdminView.coffee

@@ -34,7 +34,7 @@ define [
       @attachEvents()
 
     attachEvents: ->
-      @collection.on('sync', @onSync)
+      @collection.on('finish', @onSync)
 
     onSync: (collection, response, xhr) =>
       # only listen for the first sync event; others are updates to existing flags

app/views/jst/feature_flags/featureFlag.handlebars

@@ -1,4 +1,4 @@
-<div class="row-fluid">
+<div class="row-fluid feature {{id}}">
   <div class="span4">
     <span
        class="element_toggler"

spec/selenium/admin/admin_settings_tab_spec.rb

@@ -467,4 +467,15 @@ def set_checkbox(checkbox, checked)
       expect(f("#account_external_integration_keys_external_key0").attribute('value')).to eq ''
     end
   end
+
+  it "should show all the feature flags" do
+    course_with_admin_logged_in(:account => Account.site_admin)
+    get "/accounts/#{Account.site_admin.id}/settings"
+    f("#tab-features-link").click
+    wait_for_ajaximations
+    
+    Feature.applicable_features(Account.site_admin).each do |feature|
+      expect(f(".feature.#{feature.feature}")).to be_displayed
+    end
+  end
 end