From a9c71b51986dc8f7842e124d0ad3ce002a456e8b Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Thu, 16 Mar 2017 13:15:18 -0400
Subject: [PATCH 01/14] MMT-717: Updates set_provider to return user to page
 they were on instead of to the manage metadata page.

Former-commit-id: e2bae274041b9e5c01195fcd5a71f4b67262fd7a
---
 app/controllers/users_controller.rb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index 41804a247..fcb072020 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -25,8 +25,10 @@ def set_provider
 
     set_provider_context_token
 
+    session[:return_to] = request.referer
+
     respond_to do |format|
-      format.html { redirect_to manage_metadata_path }
+      format.html { redirect_to session[:return_to] }
       format.json { render json: nil, status: :ok }
     end
   end

From 417b5265760ae9073fb8d14e6066ad82ec6f2a40 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Wed, 22 Mar 2017 23:02:10 -0400
Subject: [PATCH 02/14] MMT-717: Adding helper that encapsulates redirect map
 and logic when changing providers.

Former-commit-id: 6dd3f3096dffb76eeded7343c656c1f46a01205d
---
 app/controllers/users_controller.rb |  3 +-
 app/helpers/set_provider_helper.rb  | 81 +++++++++++++++++++++++++++++
 2 files changed, 83 insertions(+), 1 deletion(-)
 create mode 100644 app/helpers/set_provider_helper.rb

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index fcb072020..3e94d518e 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -1,4 +1,5 @@
 class UsersController < ApplicationController
+  include SetProviderHelper
   skip_before_filter :is_logged_in, except: [:set_provider, :refresh_providers]
   skip_before_filter :setup_query
 
@@ -25,7 +26,7 @@ def set_provider
 
     set_provider_context_token
 
-    session[:return_to] = request.referer
+    session[:return_to] = get_redirect_route(request.referer)
 
     respond_to do |format|
       format.html { redirect_to session[:return_to] }
diff --git a/app/helpers/set_provider_helper.rb b/app/helpers/set_provider_helper.rb
new file mode 100644
index 000000000..a3cc5ade4
--- /dev/null
+++ b/app/helpers/set_provider_helper.rb
@@ -0,0 +1,81 @@
+# :nodoc:
+module SetProviderHelper
+
+  routes = Rails.application.routes.url_helpers
+
+  # This map contains routes and route patterns and how they should
+  # be redirected when the user switches providers.
+  #
+  # For example, on the Groups page, if the user is editing, creating,
+  # or viewing a group, they will be redirected to the "index" page for groups.
+  RedirectMap = {
+      # Groups
+      '/groups/*/edit' => routes.groups_path,
+      '/groups/*' => routes.groups_path,
+
+      # Orders
+      '/orders/*' => routes.orders_path,
+
+      # Provider holdings
+      '/provider_holdings/*' => routes.provider_holdings_path,
+
+      # Collections
+      '/collections/*' => routes.manage_metadata_path,
+
+      # Order Options
+      '/order_options/*' => routes.order_options_path,
+      '/order_options/*/edit' => routes.order_options_path,
+
+      # Order option assignments
+      '/order_option_assignments/*' => routes.order_option_assignments_path,
+      '/order_option_assignments/*/edit' => routes.order_option_assignments_path,
+
+      # Order Policies
+      '/order_policies/edit' => routes.order_policies_path,
+      '/order_policies/*/edit' => routes.order_policies_path,
+
+      # Permissions
+      '/permissions/*/edit' => routes.permissions_path,
+      '/permissions/*' => routes.permissions_path,
+
+      # DQS
+      '/data_quality_summaries/*/edit' => routes.data_quality_summaries_path,
+      '/data_quality_summaries/*' => routes.data_quality_summaries_path,
+
+      # DQS Assignments
+      '/data_quality_summary_assignments/*/edit' => routes.data_quality_summary_assignments_path,
+      '/data_quality_summary_assignments/*' => routes.data_quality_summary_assignments_path,
+
+      # Service entries
+      '/service_entries/*/edit' => routes.service_entries_path,
+      '/service_entries/*' => routes.service_entries_path,
+
+      # Service options
+      '/service_options/*/edit' => routes.service_options_path,
+      '/service_options/*' => routes.service_options_path,
+
+      # Service option assignments
+      '/service_option_assignments/*/edit' => routes.service_option_assignments_path,
+      '/service_option_assignments/*' => routes.service_option_assignments_path,
+
+      # Drafts
+      '/drafts/*' => routes.manage_metadata_path
+  }
+
+  def get_redirect_route(original_route)
+    # Parse out the path
+    curr_route = URI::parse(original_route).path
+
+    # Make some regex substitutions so that we can match routes in the map.
+    # Examples:
+    # /service_entries/12ADD5EB-9213-1FD1-A03E-519C07C2A9B1 ---> service_entries/*
+    # /order_options/34920B0B-24B2-E534-32A4-4E6F71857937/edit ---> order_options/*/edit
+
+    curr_route.sub!(/\/(\w+)\/.+?\/edit/, "/\\1/*/edit")
+    curr_route.sub!(/\/(\w+)\/.+/, "/\\1/*")
+    curr_route.sub!(/\/drafts\/.+/, '/drafts/*')
+
+    SetProviderHelper::RedirectMap.fetch(curr_route, original_route)
+  end
+
+end

From f934e620b4c2dd0ee847b315c07753f0259f9733 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Thu, 23 Mar 2017 11:37:52 -0400
Subject: [PATCH 03/14] MMT-717: Changing logic to redirect to the "index"
 action of the original controller instead of using  a map.

Former-commit-id: 7329281ac567b194ef4d36d67911bc77dbb4a14e
---
 app/controllers/users_controller.rb | 15 +++++-
 app/helpers/set_provider_helper.rb  | 81 -----------------------------
 2 files changed, 14 insertions(+), 82 deletions(-)
 delete mode 100644 app/helpers/set_provider_helper.rb

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index 3e94d518e..50450ab22 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -26,7 +26,20 @@ def set_provider
 
     set_provider_context_token
 
-    session[:return_to] = get_redirect_route(request.referer)
+    # Add any controller names we want to exclude here. For anything in this list,
+    # the user will be redirected to the Manage Metadata page
+    route_exceptions = %w(pages)
+
+    # Get the controller name from the original path
+    controller_name = Rails.application.routes.recognize_path(request.referer)[:controller]
+
+    # Return the user to the "index" action of the original controller they were on,
+    # unless it's in the exception list.
+    if route_exceptions.include?(controller_name)
+      session[:return_to] = manage_metadata_path
+    else
+      session[:return_to] = '/' + controller_name
+    end
 
     respond_to do |format|
       format.html { redirect_to session[:return_to] }
diff --git a/app/helpers/set_provider_helper.rb b/app/helpers/set_provider_helper.rb
deleted file mode 100644
index a3cc5ade4..000000000
--- a/app/helpers/set_provider_helper.rb
+++ /dev/null
@@ -1,81 +0,0 @@
-# :nodoc:
-module SetProviderHelper
-
-  routes = Rails.application.routes.url_helpers
-
-  # This map contains routes and route patterns and how they should
-  # be redirected when the user switches providers.
-  #
-  # For example, on the Groups page, if the user is editing, creating,
-  # or viewing a group, they will be redirected to the "index" page for groups.
-  RedirectMap = {
-      # Groups
-      '/groups/*/edit' => routes.groups_path,
-      '/groups/*' => routes.groups_path,
-
-      # Orders
-      '/orders/*' => routes.orders_path,
-
-      # Provider holdings
-      '/provider_holdings/*' => routes.provider_holdings_path,
-
-      # Collections
-      '/collections/*' => routes.manage_metadata_path,
-
-      # Order Options
-      '/order_options/*' => routes.order_options_path,
-      '/order_options/*/edit' => routes.order_options_path,
-
-      # Order option assignments
-      '/order_option_assignments/*' => routes.order_option_assignments_path,
-      '/order_option_assignments/*/edit' => routes.order_option_assignments_path,
-
-      # Order Policies
-      '/order_policies/edit' => routes.order_policies_path,
-      '/order_policies/*/edit' => routes.order_policies_path,
-
-      # Permissions
-      '/permissions/*/edit' => routes.permissions_path,
-      '/permissions/*' => routes.permissions_path,
-
-      # DQS
-      '/data_quality_summaries/*/edit' => routes.data_quality_summaries_path,
-      '/data_quality_summaries/*' => routes.data_quality_summaries_path,
-
-      # DQS Assignments
-      '/data_quality_summary_assignments/*/edit' => routes.data_quality_summary_assignments_path,
-      '/data_quality_summary_assignments/*' => routes.data_quality_summary_assignments_path,
-
-      # Service entries
-      '/service_entries/*/edit' => routes.service_entries_path,
-      '/service_entries/*' => routes.service_entries_path,
-
-      # Service options
-      '/service_options/*/edit' => routes.service_options_path,
-      '/service_options/*' => routes.service_options_path,
-
-      # Service option assignments
-      '/service_option_assignments/*/edit' => routes.service_option_assignments_path,
-      '/service_option_assignments/*' => routes.service_option_assignments_path,
-
-      # Drafts
-      '/drafts/*' => routes.manage_metadata_path
-  }
-
-  def get_redirect_route(original_route)
-    # Parse out the path
-    curr_route = URI::parse(original_route).path
-
-    # Make some regex substitutions so that we can match routes in the map.
-    # Examples:
-    # /service_entries/12ADD5EB-9213-1FD1-A03E-519C07C2A9B1 ---> service_entries/*
-    # /order_options/34920B0B-24B2-E534-32A4-4E6F71857937/edit ---> order_options/*/edit
-
-    curr_route.sub!(/\/(\w+)\/.+?\/edit/, "/\\1/*/edit")
-    curr_route.sub!(/\/(\w+)\/.+/, "/\\1/*")
-    curr_route.sub!(/\/drafts\/.+/, '/drafts/*')
-
-    SetProviderHelper::RedirectMap.fetch(curr_route, original_route)
-  end
-
-end

From 3e82fdd31ecc2f0386f968bfb112865be08f22ac Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Thu, 23 Mar 2017 11:40:54 -0400
Subject: [PATCH 04/14] MMT-717: Removes include for SetProviderHelper

Former-commit-id: dd19a186448f6ed568d186012d912ad359e2b739
---
 app/controllers/users_controller.rb | 1 -
 1 file changed, 1 deletion(-)

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index 50450ab22..9ec0d98c3 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -1,5 +1,4 @@
 class UsersController < ApplicationController
-  include SetProviderHelper
   skip_before_filter :is_logged_in, except: [:set_provider, :refresh_providers]
   skip_before_filter :setup_query
 

From 89cc16f82323ab531537281c967d4bc761ad2374 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Thu, 23 Mar 2017 14:45:09 -0400
Subject: [PATCH 05/14] MMT-717: Placing revised redirect logic into a new
 concern.

Former-commit-id: 848eef1448972f1af79c545863fa349f79f6fe1c
---
 app/concerns/redirector.rb          | 22 ++++++++++++++++++++++
 app/controllers/users_controller.rb | 18 ++----------------
 2 files changed, 24 insertions(+), 16 deletions(-)
 create mode 100644 app/concerns/redirector.rb

diff --git a/app/concerns/redirector.rb b/app/concerns/redirector.rb
new file mode 100644
index 000000000..438ae042f
--- /dev/null
+++ b/app/concerns/redirector.rb
@@ -0,0 +1,22 @@
+module Redirector
+  extend ActiveSupport::Concern
+
+  routes = Rails.application.routes.url_helpers
+
+  # Add any controller names we want to exclude here. For anything in this map,
+  # the user will be redirected to the Manage Metadata page
+  ROUTE_EXCEPTIONS = {
+    'pages' => routes.manage_metadata_path,
+    'provider_orders' => routes.orders_path
+  }
+
+  def get_redirect_route(original_route)
+    # Get the controller name from the original path
+    controller_name = Rails.application.routes.recognize_path(original_route)[:controller]
+
+    # Return the user to the "index" action of the original controller they were on,
+    # unless it's in the exception list.
+    ROUTE_EXCEPTIONS.include?(controller_name) ? ROUTE_EXCEPTIONS[controller_name] : url_for(action: 'index', controller: controller_name)
+  end
+
+end
\ No newline at end of file
diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index 9ec0d98c3..39921c0b2 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -1,4 +1,5 @@
 class UsersController < ApplicationController
+  include Redirector
   skip_before_filter :is_logged_in, except: [:set_provider, :refresh_providers]
   skip_before_filter :setup_query
 
@@ -25,23 +26,8 @@ def set_provider
 
     set_provider_context_token
 
-    # Add any controller names we want to exclude here. For anything in this list,
-    # the user will be redirected to the Manage Metadata page
-    route_exceptions = %w(pages)
-
-    # Get the controller name from the original path
-    controller_name = Rails.application.routes.recognize_path(request.referer)[:controller]
-
-    # Return the user to the "index" action of the original controller they were on,
-    # unless it's in the exception list.
-    if route_exceptions.include?(controller_name)
-      session[:return_to] = manage_metadata_path
-    else
-      session[:return_to] = '/' + controller_name
-    end
-
     respond_to do |format|
-      format.html { redirect_to session[:return_to] }
+      format.html { redirect_to get_redirect_route(request.referer) }
       format.json { render json: nil, status: :ok }
     end
   end

From 63b254760a6ea9d0fbaf270807cfd29152fa3e22 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Thu, 23 Mar 2017 15:53:56 -0400
Subject: [PATCH 06/14] MMT-717: Clarifying comment about the exception list.

Former-commit-id: a472fe73770a56cb9b6a17dda5f8868a0f9ef7be
---
 app/concerns/redirector.rb | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/app/concerns/redirector.rb b/app/concerns/redirector.rb
index 438ae042f..c1c89fc1a 100644
--- a/app/concerns/redirector.rb
+++ b/app/concerns/redirector.rb
@@ -3,8 +3,9 @@ module Redirector
 
   routes = Rails.application.routes.url_helpers
 
-  # Add any controller names we want to exclude here. For anything in this map,
-  # the user will be redirected to the Manage Metadata page
+  # Add any redirects we want to customize here, e.g., if the "index" action
+  # for a given path is "/provider_orders", we will redirect to the '/orders'
+  # path.
   ROUTE_EXCEPTIONS = {
     'pages' => routes.manage_metadata_path,
     'provider_orders' => routes.orders_path

From d1140ee39129c7e3237cbc5f153ec51913aa1bf6 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Fri, 24 Mar 2017 18:20:57 -0400
Subject: [PATCH 07/14] MMT-717: Clarifying comment about the exception list.

Former-commit-id: 74eff0743d270e1b15f3bec35f08d31bdbcbbe7d
---
 .../change_current_provider.coffee            | 19 +++++++++++++++++++
 app/views/shared/_header.html.erb             |  3 ++-
 spec/fixtures/providers.json                  |  8 ++++++++
 3 files changed, 29 insertions(+), 1 deletion(-)
 create mode 100644 spec/fixtures/providers.json

diff --git a/app/assets/javascripts/change_current_provider.coffee b/app/assets/javascripts/change_current_provider.coffee
index dc21d5874..e0b0c6814 100644
--- a/app/assets/javascripts/change_current_provider.coffee
+++ b/app/assets/javascripts/change_current_provider.coffee
@@ -68,3 +68,22 @@ $(document).ready ->
       dataType: 'json'
       success: (data, status, xhr) ->
         $("##{actionLink}")[0].click()
+
+
+  $('#refresh_available_providers_link').on 'click', (element) ->
+    $.ajax(
+      url: '/refresh_providers'
+      method: 'get'
+      dataType: 'json'
+    ).done( (response) ->
+      $('#select_provider').empty()
+      $('#select_provider').append( $('<option>').text('Select Provider') );
+      $.each(response.items, (i,v) ->
+        $('#select_provider').append( $('<option>').val(v).text(v) );
+        $('span.refresh-providers.spinner').remove()
+        $('a.refresh-providers.spinner').show()
+
+    )).fail( (response) ->
+      console.error(response)
+    )
+
diff --git a/app/views/shared/_header.html.erb b/app/views/shared/_header.html.erb
index 861c56ccd..9bb1d9eff 100644
--- a/app/views/shared/_header.html.erb
+++ b/app/views/shared/_header.html.erb
@@ -78,7 +78,8 @@
       <% end %>
 
       <p>
-        <%= link_to 'Refresh your available providers', refresh_user_providers_path, class: 'refresh-providers spinner' %>
+        <%-# link_to 'Refresh your available providers', refresh_user_providers_path, id: 'refresh_available_providers_link', class: 'refresh-providers spinner' %>
+        <%= link_to 'Refresh your available providers', '#', id: 'refresh_available_providers_link', class: 'refresh-providers spinner' %>
       </p>
     </div>
   <% end %>
diff --git a/spec/fixtures/providers.json b/spec/fixtures/providers.json
new file mode 100644
index 000000000..7cd8bce59
--- /dev/null
+++ b/spec/fixtures/providers.json
@@ -0,0 +1,8 @@
+{
+  "hits": 1,
+  "took": 2,
+  "items": [
+    "MMT_1",
+    "MMT_2"
+  ]
+}
\ No newline at end of file

From 88ca7e5a9386955fcd2efee3024fc2229670b4c9 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Fri, 24 Mar 2017 19:46:24 -0400
Subject: [PATCH 08/14] MMT-717: Adding tests, fixing Coffeescript, changing
 refresh of providers to an AJAX call

Former-commit-id: da13f14c205a422374793684c83b72e766abdcd6
---
 .../change_current_provider.coffee            | 12 +++--
 app/controllers/users_controller.rb           |  4 +-
 app/views/shared/_header.html.erb             |  3 +-
 .../provider_context/provider_context_spec.rb | 51 +++++++++++++++++++
 4 files changed, 62 insertions(+), 8 deletions(-)

diff --git a/app/assets/javascripts/change_current_provider.coffee b/app/assets/javascripts/change_current_provider.coffee
index e0b0c6814..61ac88d8a 100644
--- a/app/assets/javascripts/change_current_provider.coffee
+++ b/app/assets/javascripts/change_current_provider.coffee
@@ -77,13 +77,15 @@ $(document).ready ->
       dataType: 'json'
     ).done( (response) ->
       $('#select_provider').empty()
-      $('#select_provider').append( $('<option>').text('Select Provider') );
+      $('#select_provider').append( $('<option>').text('Select Provider') )
+
       $.each(response.items, (i,v) ->
-        $('#select_provider').append( $('<option>').val(v).text(v) );
-        $('span.refresh-providers.spinner').remove()
-        $('a.refresh-providers.spinner').show()
+        $('#select_provider').append( $('<option>').val(v).text(v) )
+      )
 
-    )).fail( (response) ->
+      $('span.refresh-providers.spinner').remove()
+      $('a.refresh-providers.spinner').show()
+    ).fail( (response) ->
       console.error(response)
     )
 
diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index 39921c0b2..40866433c 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -36,6 +36,8 @@ def refresh_providers
     current_user.update(provider_id: nil)
     current_user.set_available_providers(token)
 
-    redirect_to manage_metadata_path
+    respond_to do |format|
+      format.json { render json: { items: current_user.available_providers } }
+    end
   end
 end
diff --git a/app/views/shared/_header.html.erb b/app/views/shared/_header.html.erb
index 9bb1d9eff..26cc8b8c8 100644
--- a/app/views/shared/_header.html.erb
+++ b/app/views/shared/_header.html.erb
@@ -78,8 +78,7 @@
       <% end %>
 
       <p>
-        <%-# link_to 'Refresh your available providers', refresh_user_providers_path, id: 'refresh_available_providers_link', class: 'refresh-providers spinner' %>
-        <%= link_to 'Refresh your available providers', '#', id: 'refresh_available_providers_link', class: 'refresh-providers spinner' %>
+        <%= link_to 'Refresh your available providers', 'javascript:void(0)', id: 'refresh_available_providers_link', class: 'refresh-providers spinner' %>
       </p>
     </div>
   <% end %>
diff --git a/spec/features/provider_context/provider_context_spec.rb b/spec/features/provider_context/provider_context_spec.rb
index 82c3d5c7e..3e23bd296 100644
--- a/spec/features/provider_context/provider_context_spec.rb
+++ b/spec/features/provider_context/provider_context_spec.rb
@@ -86,6 +86,57 @@
           expect(page).to have_content('MMT_2')
         end
       end
+
+      context 'if user is on the search orders result page' do
+        before do
+          visit orders_path
+
+          fill_in 'Order GUID', with: 'ABC'
+
+          VCR.use_cassette('provider_context/order_search', record: :none) do
+            within '.order-by-guid-form' do
+              click_on 'Submit'
+            end
+          end
+
+          within '#user-info' do
+            click_on 'Change Provider'
+          end
+
+          select 'MMT_1', from: 'select_provider'
+          wait_for_ajax
+        end
+
+        it 'redirects to the orders index page when switching provider context' do
+          expect(page).to have_current_path(orders_path, only_path: true)
+        end
+      end
+
+
+      context 'if user is on a groups detail page' do
+        before do
+          VCR.use_cassette('provider_context/groups_list', record: :none) do
+            visit groups_path
+          end
+
+          VCR.use_cassette('provider_context/groups_detail', record: :none) do
+            within '.groups-table' do
+              click_on 'Group 1'
+            end
+          end
+
+          within '#user-info' do
+            click_on 'Change Provider'
+          end
+
+          select 'MMT_1', from: 'select_provider'
+          wait_for_ajax
+        end
+
+        it 'redirects to the groups listing page when switching provider context' do
+          expect(page).to have_current_path(groups_path, only_path: true)
+        end
+      end
     end
   end
 

From 57fd33220d00aadf61bdb33b835e8459b4c393e5 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Fri, 24 Mar 2017 19:49:45 -0400
Subject: [PATCH 09/14] MMT-717: Adding VCR recordings.

Former-commit-id: abf033c83d80840232ea682054d3b9d64eb821d6
---
 spec/vcr/provider_context/order_search.yml    | 50 +++++++++++++++++++
 .../provider_context/view_service_entries.yml | 50 +++++++++++++++++++
 2 files changed, 100 insertions(+)
 create mode 100644 spec/vcr/provider_context/order_search.yml
 create mode 100644 spec/vcr/provider_context/view_service_entries.yml

diff --git a/spec/vcr/provider_context/order_search.yml b/spec/vcr/provider_context/order_search.yml
new file mode 100644
index 000000000..f37572912
--- /dev/null
+++ b/spec/vcr/provider_context/order_search.yml
@@ -0,0 +1,50 @@
+---
+http_interactions:
+- request:
+    method: post
+    uri: https://testbed.echo.nasa.gov/echo-v10/OrderManagementServicePortImpl
+    body:
+      encoding: UTF-8
+      string: <?xml version="1.0" encoding="UTF-8"?><SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"
+        xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"><SOAP-ENV:Header/><SOAP-ENV:Body><ns2:GetOrders
+        xmlns:ns2="http://echo.nasa.gov/echo/v10" xmlns:ns3="http://echo.nasa.gov/echo/v10/types"
+        xmlns:ns4="http://echo.nasa.gov/ingest/v10"><ns2:token>access_token</ns2:token><ns2:orderGuids><ns3:Item>ABC</ns3:Item></ns2:orderGuids></ns2:GetOrders><to_str/></SOAP-ENV:Body></SOAP-ENV:Envelope>
+    headers:
+      User-Agent:
+      - Faraday v0.8.11
+      Content-Type:
+      - text/xml
+  response:
+    status:
+      code: 500
+      message: 
+    headers:
+      x-secret-message:
+      - HELLO
+      strict-transport-security:
+      - max-age=31536000;
+      accept:
+      - text/xml, text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2
+      content-type:
+      - text/xml; charset=UTF-8
+      date:
+      - Fri, 24 Mar 2017 23:16:13 GMT
+      server:
+      - nginx/1.10.2
+      soapaction:
+      - '""'
+      content-length:
+      - '942'
+      connection:
+      - close
+    body:
+      encoding: UTF-8
+      string: <SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"><SOAP-ENV:Header/><SOAP-ENV:Body><SOAP-ENV:Fault><faultcode>SOAP-ENV:Client</faultcode><faultstring>Token
+        [access_token] does not exist</faultstring><detail><ns4:ItemNotFoundFault
+        xmlns:ns2="http://echo.nasa.gov/echo/v10" xmlns:ns3="http://echo.nasa.gov/echo/v10/types"
+        xmlns:ns4="http://echo.nasa.gov/echo/v10/faults"><ns4:ErrorCode>TokenNotFound</ns4:ErrorCode><ns4:OpsMessage
+        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:nil="true"/><ns4:SystemMessage>Token
+        [access_token] does not exist</ns4:SystemMessage><ns4:Timestamp>2017-03-24T23:14:49.111Z</ns4:Timestamp><ns4:ErrorInstanceId>DDD7D9F1-7EEE-3A6B-6334-E5CE733ED304</ns4:ErrorInstanceId><ns4:Arguments><ns4:Argument><ns4:Name>Token</ns4:Name><ns4:Value>access_token</ns4:Value></ns4:Argument></ns4:Arguments></ns4:ItemNotFoundFault></detail></SOAP-ENV:Fault></SOAP-ENV:Body></SOAP-ENV:Envelope>
+    http_version: 
+  recorded_at: Fri, 24 Mar 2017 23:14:49 GMT
+recorded_with: VCR 3.0.1
diff --git a/spec/vcr/provider_context/view_service_entries.yml b/spec/vcr/provider_context/view_service_entries.yml
new file mode 100644
index 000000000..f9046091a
--- /dev/null
+++ b/spec/vcr/provider_context/view_service_entries.yml
@@ -0,0 +1,50 @@
+---
+http_interactions:
+- request:
+    method: post
+    uri: https://testbed.echo.nasa.gov/echo-v10/ServiceManagementServicePortImpl
+    body:
+      encoding: UTF-8
+      string: <?xml version="1.0" encoding="UTF-8"?><SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"
+        xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"><SOAP-ENV:Header/><SOAP-ENV:Body><ns2:GetServiceEntriesByProvider
+        xmlns:ns2="http://echo.nasa.gov/echo/v10" xmlns:ns3="http://echo.nasa.gov/echo/v10/types"
+        xmlns:ns4="http://echo.nasa.gov/ingest/v10"><ns2:token>access_token</ns2:token><ns2:providerGuids><ns3:Item>07ACF84C-6360-0BC7-92A8-E95A958DDE12</ns3:Item></ns2:providerGuids></ns2:GetServiceEntriesByProvider><to_str/></SOAP-ENV:Body></SOAP-ENV:Envelope>
+    headers:
+      User-Agent:
+      - Faraday v0.8.11
+      Content-Type:
+      - text/xml
+  response:
+    status:
+      code: 500
+      message: 
+    headers:
+      x-secret-message:
+      - HELLO
+      strict-transport-security:
+      - max-age=31536000;
+      accept:
+      - text/xml, text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2
+      content-type:
+      - text/xml; charset=UTF-8
+      date:
+      - Fri, 24 Mar 2017 23:28:18 GMT
+      server:
+      - nginx/1.10.2
+      soapaction:
+      - '""'
+      content-length:
+      - '942'
+      connection:
+      - close
+    body:
+      encoding: UTF-8
+      string: <SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"><SOAP-ENV:Header/><SOAP-ENV:Body><SOAP-ENV:Fault><faultcode>SOAP-ENV:Client</faultcode><faultstring>Token
+        [access_token] does not exist</faultstring><detail><ns4:ItemNotFoundFault
+        xmlns:ns2="http://echo.nasa.gov/echo/v10" xmlns:ns3="http://echo.nasa.gov/echo/v10/types"
+        xmlns:ns4="http://echo.nasa.gov/echo/v10/faults"><ns4:ErrorCode>TokenNotFound</ns4:ErrorCode><ns4:OpsMessage
+        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:nil="true"/><ns4:SystemMessage>Token
+        [access_token] does not exist</ns4:SystemMessage><ns4:Timestamp>2017-03-24T23:27:40.737Z</ns4:Timestamp><ns4:ErrorInstanceId>32ADBFC1-44EF-6F40-4041-FB381BDC2B5A</ns4:ErrorInstanceId><ns4:Arguments><ns4:Argument><ns4:Name>Token</ns4:Name><ns4:Value>access_token</ns4:Value></ns4:Argument></ns4:Arguments></ns4:ItemNotFoundFault></detail></SOAP-ENV:Fault></SOAP-ENV:Body></SOAP-ENV:Envelope>
+    http_version: 
+  recorded_at: Fri, 24 Mar 2017 23:27:40 GMT
+recorded_with: VCR 3.0.1

From a3255a08d4f43e6e9475950a5b39ea2bfdb4e045 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Mon, 27 Mar 2017 16:04:12 -0400
Subject: [PATCH 10/14] MMT-717: Making parameter names more clear.

Former-commit-id: ec0e027d11b5a88ef2c283e94cc341d6c8647f95
---
 app/assets/javascripts/change_current_provider.coffee | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/assets/javascripts/change_current_provider.coffee b/app/assets/javascripts/change_current_provider.coffee
index 61ac88d8a..65049b740 100644
--- a/app/assets/javascripts/change_current_provider.coffee
+++ b/app/assets/javascripts/change_current_provider.coffee
@@ -79,8 +79,8 @@ $(document).ready ->
       $('#select_provider').empty()
       $('#select_provider').append( $('<option>').text('Select Provider') )
 
-      $.each(response.items, (i,v) ->
-        $('#select_provider').append( $('<option>').val(v).text(v) )
+      $.each(response.items, (index,value) ->
+        $('#select_provider').append( $('<option>').val(value).text(value) )
       )
 
       $('span.refresh-providers.spinner').remove()

From de2e3292ce8350348855337b6e69a0edad07bbe0 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Tue, 28 Mar 2017 14:57:47 -0400
Subject: [PATCH 11/14] MMT-717: Doing some general cleanup (removing unused
 JSON and yml files), fixing tests.

Former-commit-id: 3650b86458e1f6002c910b12d4a7f297c0e32488
---
 .../provider_context/provider_context_spec.rb | 19 +++----
 spec/fixtures/providers.json                  |  8 ---
 spec/vcr/provider_context/order_search.yml    | 32 ++++++------
 .../provider_context/view_service_entries.yml | 50 -------------------
 4 files changed, 20 insertions(+), 89 deletions(-)
 delete mode 100644 spec/fixtures/providers.json
 delete mode 100644 spec/vcr/provider_context/view_service_entries.yml

diff --git a/spec/features/provider_context/provider_context_spec.rb b/spec/features/provider_context/provider_context_spec.rb
index 3e23bd296..9f2d98a71 100644
--- a/spec/features/provider_context/provider_context_spec.rb
+++ b/spec/features/provider_context/provider_context_spec.rb
@@ -1,4 +1,4 @@
-# MMT-272
+# MMT-272, MMT-717
 
 require 'rails_helper'
 
@@ -112,18 +112,11 @@
         end
       end
 
-
-      context 'if user is on a groups detail page' do
+      context 'if user is on the permissions creation page' do
         before do
-          VCR.use_cassette('provider_context/groups_list', record: :none) do
-            visit groups_path
-          end
+          visit permissions_path
 
-          VCR.use_cassette('provider_context/groups_detail', record: :none) do
-            within '.groups-table' do
-              click_on 'Group 1'
-            end
-          end
+          click_on 'Create a Permission'
 
           within '#user-info' do
             click_on 'Change Provider'
@@ -133,8 +126,8 @@
           wait_for_ajax
         end
 
-        it 'redirects to the groups listing page when switching provider context' do
-          expect(page).to have_current_path(groups_path, only_path: true)
+        it 'redirects to the permissions index page when switching provider context' do
+          expect(page).to have_current_path(permissions_path, only_path: true)
         end
       end
     end
diff --git a/spec/fixtures/providers.json b/spec/fixtures/providers.json
deleted file mode 100644
index 7cd8bce59..000000000
--- a/spec/fixtures/providers.json
+++ /dev/null
@@ -1,8 +0,0 @@
-{
-  "hits": 1,
-  "took": 2,
-  "items": [
-    "MMT_1",
-    "MMT_2"
-  ]
-}
\ No newline at end of file
diff --git a/spec/vcr/provider_context/order_search.yml b/spec/vcr/provider_context/order_search.yml
index f37572912..edf85f67c 100644
--- a/spec/vcr/provider_context/order_search.yml
+++ b/spec/vcr/provider_context/order_search.yml
@@ -16,35 +16,31 @@ http_interactions:
       - text/xml
   response:
     status:
-      code: 500
+      code: 503
       message: 
     headers:
       x-secret-message:
       - HELLO
       strict-transport-security:
       - max-age=31536000;
-      accept:
-      - text/xml, text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2
+      cache-control:
+      - no-cache
       content-type:
-      - text/xml; charset=UTF-8
-      date:
-      - Fri, 24 Mar 2017 23:16:13 GMT
-      server:
-      - nginx/1.10.2
-      soapaction:
-      - '""'
+      - text/html
       content-length:
-      - '942'
+      - '107'
       connection:
       - close
+      date:
+      - Tue, 28 Mar 2017 12:58:07 GMT
+      server:
+      - lighttpd
     body:
       encoding: UTF-8
-      string: <SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"><SOAP-ENV:Header/><SOAP-ENV:Body><SOAP-ENV:Fault><faultcode>SOAP-ENV:Client</faultcode><faultstring>Token
-        [access_token] does not exist</faultstring><detail><ns4:ItemNotFoundFault
-        xmlns:ns2="http://echo.nasa.gov/echo/v10" xmlns:ns3="http://echo.nasa.gov/echo/v10/types"
-        xmlns:ns4="http://echo.nasa.gov/echo/v10/faults"><ns4:ErrorCode>TokenNotFound</ns4:ErrorCode><ns4:OpsMessage
-        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:nil="true"/><ns4:SystemMessage>Token
-        [access_token] does not exist</ns4:SystemMessage><ns4:Timestamp>2017-03-24T23:14:49.111Z</ns4:Timestamp><ns4:ErrorInstanceId>DDD7D9F1-7EEE-3A6B-6334-E5CE733ED304</ns4:ErrorInstanceId><ns4:Arguments><ns4:Argument><ns4:Name>Token</ns4:Name><ns4:Value>access_token</ns4:Value></ns4:Argument></ns4:Arguments></ns4:ItemNotFoundFault></detail></SOAP-ENV:Fault></SOAP-ENV:Body></SOAP-ENV:Envelope>
+      string: |
+        <html><body><h1>503 Service Unavailable</h1>
+        No server is available to handle this request.
+        </body></html>
     http_version: 
-  recorded_at: Fri, 24 Mar 2017 23:14:49 GMT
+  recorded_at: Tue, 28 Mar 2017 12:58:07 GMT
 recorded_with: VCR 3.0.1
diff --git a/spec/vcr/provider_context/view_service_entries.yml b/spec/vcr/provider_context/view_service_entries.yml
deleted file mode 100644
index f9046091a..000000000
--- a/spec/vcr/provider_context/view_service_entries.yml
+++ /dev/null
@@ -1,50 +0,0 @@
----
-http_interactions:
-- request:
-    method: post
-    uri: https://testbed.echo.nasa.gov/echo-v10/ServiceManagementServicePortImpl
-    body:
-      encoding: UTF-8
-      string: <?xml version="1.0" encoding="UTF-8"?><SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"
-        xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"><SOAP-ENV:Header/><SOAP-ENV:Body><ns2:GetServiceEntriesByProvider
-        xmlns:ns2="http://echo.nasa.gov/echo/v10" xmlns:ns3="http://echo.nasa.gov/echo/v10/types"
-        xmlns:ns4="http://echo.nasa.gov/ingest/v10"><ns2:token>access_token</ns2:token><ns2:providerGuids><ns3:Item>07ACF84C-6360-0BC7-92A8-E95A958DDE12</ns3:Item></ns2:providerGuids></ns2:GetServiceEntriesByProvider><to_str/></SOAP-ENV:Body></SOAP-ENV:Envelope>
-    headers:
-      User-Agent:
-      - Faraday v0.8.11
-      Content-Type:
-      - text/xml
-  response:
-    status:
-      code: 500
-      message: 
-    headers:
-      x-secret-message:
-      - HELLO
-      strict-transport-security:
-      - max-age=31536000;
-      accept:
-      - text/xml, text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2
-      content-type:
-      - text/xml; charset=UTF-8
-      date:
-      - Fri, 24 Mar 2017 23:28:18 GMT
-      server:
-      - nginx/1.10.2
-      soapaction:
-      - '""'
-      content-length:
-      - '942'
-      connection:
-      - close
-    body:
-      encoding: UTF-8
-      string: <SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"><SOAP-ENV:Header/><SOAP-ENV:Body><SOAP-ENV:Fault><faultcode>SOAP-ENV:Client</faultcode><faultstring>Token
-        [access_token] does not exist</faultstring><detail><ns4:ItemNotFoundFault
-        xmlns:ns2="http://echo.nasa.gov/echo/v10" xmlns:ns3="http://echo.nasa.gov/echo/v10/types"
-        xmlns:ns4="http://echo.nasa.gov/echo/v10/faults"><ns4:ErrorCode>TokenNotFound</ns4:ErrorCode><ns4:OpsMessage
-        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:nil="true"/><ns4:SystemMessage>Token
-        [access_token] does not exist</ns4:SystemMessage><ns4:Timestamp>2017-03-24T23:27:40.737Z</ns4:Timestamp><ns4:ErrorInstanceId>32ADBFC1-44EF-6F40-4041-FB381BDC2B5A</ns4:ErrorInstanceId><ns4:Arguments><ns4:Argument><ns4:Name>Token</ns4:Name><ns4:Value>access_token</ns4:Value></ns4:Argument></ns4:Arguments></ns4:ItemNotFoundFault></detail></SOAP-ENV:Fault></SOAP-ENV:Body></SOAP-ENV:Envelope>
-    http_version: 
-  recorded_at: Fri, 24 Mar 2017 23:27:40 GMT
-recorded_with: VCR 3.0.1

From 1ca90c6d3e28607356d6793e4a3781ab43d6d570 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Wed, 29 Mar 2017 12:24:52 -0400
Subject: [PATCH 12/14] MMT-717: Replacing recording (previous recording had an
 error message due to ECHO REST throwing a 503 error).

Former-commit-id: 24cba29b6dd2633e31a1a40aab21c329667f1a64
---
 spec/vcr/provider_context/order_search.yml | 32 ++++++++++++----------
 1 file changed, 18 insertions(+), 14 deletions(-)

diff --git a/spec/vcr/provider_context/order_search.yml b/spec/vcr/provider_context/order_search.yml
index edf85f67c..4b5ed0925 100644
--- a/spec/vcr/provider_context/order_search.yml
+++ b/spec/vcr/provider_context/order_search.yml
@@ -16,31 +16,35 @@ http_interactions:
       - text/xml
   response:
     status:
-      code: 503
+      code: 500
       message: 
     headers:
       x-secret-message:
       - HELLO
       strict-transport-security:
       - max-age=31536000;
-      cache-control:
-      - no-cache
+      accept:
+      - text/xml, text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2
       content-type:
-      - text/html
+      - text/xml; charset=UTF-8
+      date:
+      - Wed, 29 Mar 2017 16:19:36 GMT
+      server:
+      - nginx/1.10.2
+      soapaction:
+      - '""'
       content-length:
-      - '107'
+      - '941'
       connection:
       - close
-      date:
-      - Tue, 28 Mar 2017 12:58:07 GMT
-      server:
-      - lighttpd
     body:
       encoding: UTF-8
-      string: |
-        <html><body><h1>503 Service Unavailable</h1>
-        No server is available to handle this request.
-        </body></html>
+      string: <SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"><SOAP-ENV:Header/><SOAP-ENV:Body><SOAP-ENV:Fault><faultcode>SOAP-ENV:Client</faultcode><faultstring>Could
+        not find order with guid [ABC].</faultstring><detail><ns4:ItemNotFoundFault
+        xmlns:ns2="http://echo.nasa.gov/echo/v10" xmlns:ns3="http://echo.nasa.gov/echo/v10/types"
+        xmlns:ns4="http://echo.nasa.gov/echo/v10/faults"><ns4:ErrorCode>OrderNotFound</ns4:ErrorCode><ns4:OpsMessage
+        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:nil="true"/><ns4:SystemMessage>Could
+        not find order with guid [ABC].</ns4:SystemMessage><ns4:Timestamp>2017-03-29T16:19:08.714Z</ns4:Timestamp><ns4:ErrorInstanceId>221378BE-D155-F822-7195-7F12F6783A37</ns4:ErrorInstanceId><ns4:Arguments><ns4:Argument><ns4:Name>OrderGuid</ns4:Name><ns4:Value>ABC</ns4:Value></ns4:Argument></ns4:Arguments></ns4:ItemNotFoundFault></detail></SOAP-ENV:Fault></SOAP-ENV:Body></SOAP-ENV:Envelope>
     http_version: 
-  recorded_at: Tue, 28 Mar 2017 12:58:07 GMT
+  recorded_at: Wed, 29 Mar 2017 16:19:08 GMT
 recorded_with: VCR 3.0.1

From 3a1bf2eb8b665c084465de79a1f5421af82ae627 Mon Sep 17 00:00:00 2001
From: James Pino <jpino@innovim.com>
Date: Thu, 30 Mar 2017 07:57:56 -0400
Subject: [PATCH 13/14] MMT-717: Replacing existing test case with one that
 ensures the user is redirected back to the orders index action, removing
 unused VCR recording.

Former-commit-id: ee298b3b89e7be33b4d47d8ee6c33cd8ac5cfe79
---
 .../change_current_provider.coffee            |  2 +-
 app/views/shared/_header.html.erb             |  2 +-
 .../provider_context/provider_context_spec.rb | 16 +++---
 spec/vcr/provider_context/order_search.yml    | 50 -------------------
 4 files changed, 10 insertions(+), 60 deletions(-)
 delete mode 100644 spec/vcr/provider_context/order_search.yml

diff --git a/app/assets/javascripts/change_current_provider.coffee b/app/assets/javascripts/change_current_provider.coffee
index 65049b740..b020e02dd 100644
--- a/app/assets/javascripts/change_current_provider.coffee
+++ b/app/assets/javascripts/change_current_provider.coffee
@@ -70,7 +70,7 @@ $(document).ready ->
         $("##{actionLink}")[0].click()
 
 
-  $('#refresh_available_providers_link').on 'click', (element) ->
+  $('#refresh-available-providers-link').on 'click', (element) ->
     $.ajax(
       url: '/refresh_providers'
       method: 'get'
diff --git a/app/views/shared/_header.html.erb b/app/views/shared/_header.html.erb
index 26cc8b8c8..0bc3f1612 100644
--- a/app/views/shared/_header.html.erb
+++ b/app/views/shared/_header.html.erb
@@ -78,7 +78,7 @@
       <% end %>
 
       <p>
-        <%= link_to 'Refresh your available providers', 'javascript:void(0)', id: 'refresh_available_providers_link', class: 'refresh-providers spinner' %>
+        <%= link_to 'Refresh your available providers', 'javascript:void(0)', id: 'refresh-available-providers-link', class: 'refresh-providers spinner' %>
       </p>
     </div>
   <% end %>
diff --git a/spec/features/provider_context/provider_context_spec.rb b/spec/features/provider_context/provider_context_spec.rb
index 9f2d98a71..78630e35e 100644
--- a/spec/features/provider_context/provider_context_spec.rb
+++ b/spec/features/provider_context/provider_context_spec.rb
@@ -3,6 +3,8 @@
 require 'rails_helper'
 
 describe 'Provider context', js: true do
+  let(:order_guid) { 'FF330AD3-1A89-871C-AC94-B689A5C95723' }
+
   context 'when the user has multiple possible provider contexts' do
     before do
       login(providers: nil)
@@ -87,16 +89,14 @@
         end
       end
 
-      context 'if user is on the search orders result page' do
+      # This is the exception case (see redirector.rb)
+      context 'if user is on the provider order details page' do
         before do
-          visit orders_path
-
-          fill_in 'Order GUID', with: 'ABC'
+          # The order guid belongs to NSIDC_ECS
+          User.first.update(provider_id: 'NSIDC_ECS')
 
-          VCR.use_cassette('provider_context/order_search', record: :none) do
-            within '.order-by-guid-form' do
-              click_on 'Submit'
-            end
+          VCR.use_cassette('echo_soap/order_processing_service/provider_orders/terminal_order', record: :none) do
+            visit provider_order_path(order_guid)
           end
 
           within '#user-info' do
diff --git a/spec/vcr/provider_context/order_search.yml b/spec/vcr/provider_context/order_search.yml
deleted file mode 100644
index 4b5ed0925..000000000
--- a/spec/vcr/provider_context/order_search.yml
+++ /dev/null
@@ -1,50 +0,0 @@
----
-http_interactions:
-- request:
-    method: post
-    uri: https://testbed.echo.nasa.gov/echo-v10/OrderManagementServicePortImpl
-    body:
-      encoding: UTF-8
-      string: <?xml version="1.0" encoding="UTF-8"?><SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"
-        xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"><SOAP-ENV:Header/><SOAP-ENV:Body><ns2:GetOrders
-        xmlns:ns2="http://echo.nasa.gov/echo/v10" xmlns:ns3="http://echo.nasa.gov/echo/v10/types"
-        xmlns:ns4="http://echo.nasa.gov/ingest/v10"><ns2:token>access_token</ns2:token><ns2:orderGuids><ns3:Item>ABC</ns3:Item></ns2:orderGuids></ns2:GetOrders><to_str/></SOAP-ENV:Body></SOAP-ENV:Envelope>
-    headers:
-      User-Agent:
-      - Faraday v0.8.11
-      Content-Type:
-      - text/xml
-  response:
-    status:
-      code: 500
-      message: 
-    headers:
-      x-secret-message:
-      - HELLO
-      strict-transport-security:
-      - max-age=31536000;
-      accept:
-      - text/xml, text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2
-      content-type:
-      - text/xml; charset=UTF-8
-      date:
-      - Wed, 29 Mar 2017 16:19:36 GMT
-      server:
-      - nginx/1.10.2
-      soapaction:
-      - '""'
-      content-length:
-      - '941'
-      connection:
-      - close
-    body:
-      encoding: UTF-8
-      string: <SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"><SOAP-ENV:Header/><SOAP-ENV:Body><SOAP-ENV:Fault><faultcode>SOAP-ENV:Client</faultcode><faultstring>Could
-        not find order with guid [ABC].</faultstring><detail><ns4:ItemNotFoundFault
-        xmlns:ns2="http://echo.nasa.gov/echo/v10" xmlns:ns3="http://echo.nasa.gov/echo/v10/types"
-        xmlns:ns4="http://echo.nasa.gov/echo/v10/faults"><ns4:ErrorCode>OrderNotFound</ns4:ErrorCode><ns4:OpsMessage
-        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:nil="true"/><ns4:SystemMessage>Could
-        not find order with guid [ABC].</ns4:SystemMessage><ns4:Timestamp>2017-03-29T16:19:08.714Z</ns4:Timestamp><ns4:ErrorInstanceId>221378BE-D155-F822-7195-7F12F6783A37</ns4:ErrorInstanceId><ns4:Arguments><ns4:Argument><ns4:Name>OrderGuid</ns4:Name><ns4:Value>ABC</ns4:Value></ns4:Argument></ns4:Arguments></ns4:ItemNotFoundFault></detail></SOAP-ENV:Fault></SOAP-ENV:Body></SOAP-ENV:Envelope>
-    http_version: 
-  recorded_at: Wed, 29 Mar 2017 16:19:08 GMT
-recorded_with: VCR 3.0.1

From 5b53e666898b34a7281c1eb5b14eb6fc4842f508 Mon Sep 17 00:00:00 2001
From: Ryan Abbott <ryan@element84.com>
Date: Mon, 3 Apr 2017 16:23:31 -0400
Subject: [PATCH 14/14] MMT-717: Updates CoffeeScript syntax and adds redirect
 fallback

---
 .../change_current_provider.coffee            | 26 ++++++++-----------
 ...ctor.rb => provider_context_redirector.rb} | 14 ++++++----
 app/controllers/users_controller.rb           |  4 ++-
 .../provider_context/provider_context_spec.rb |  4 +--
 4 files changed, 25 insertions(+), 23 deletions(-)
 rename app/concerns/{redirector.rb => provider_context_redirector.rb} (69%)

diff --git a/app/assets/javascripts/change_current_provider.coffee b/app/assets/javascripts/change_current_provider.coffee
index b020e02dd..9ae1a80da 100644
--- a/app/assets/javascripts/change_current_provider.coffee
+++ b/app/assets/javascripts/change_current_provider.coffee
@@ -1,5 +1,4 @@
 $(document).ready ->
-
   $('.display-modal').leanModal
     top: 200
     overlay: 0.6
@@ -53,11 +52,12 @@ $(document).ready ->
       url: "/set_provider?provider_id=#{provider}"
       method: 'post'
       dataType: 'json'
+      
       success: (data, status, xhr) ->
         # Click the link that the user needs
         $(link)[0].click()
 
-  # change current provider from banner link
+  # Change current provider
   $('#change-current-provider-banner-link').on 'click', (element) ->
     provider = $(element.target).data('provider')
     actionLink = $(element.target).data('actionLink')
@@ -66,26 +66,22 @@ $(document).ready ->
       url: "/set_provider?provider_id=#{provider}"
       method: 'post'
       dataType: 'json'
+
       success: (data, status, xhr) ->
         $("##{actionLink}")[0].click()
 
-
   $('#refresh-available-providers-link').on 'click', (element) ->
-    $.ajax(
+    $.ajax
       url: '/refresh_providers'
       method: 'get'
       dataType: 'json'
-    ).done( (response) ->
-      $('#select_provider').empty()
-      $('#select_provider').append( $('<option>').text('Select Provider') )
 
-      $.each(response.items, (index,value) ->
-        $('#select_provider').append( $('<option>').val(value).text(value) )
-      )
+      success: (response) ->
+        $('#select_provider').empty()
+        $('#select_provider').append($('<option>').text('Select Provider'))
 
-      $('span.refresh-providers.spinner').remove()
-      $('a.refresh-providers.spinner').show()
-    ).fail( (response) ->
-      console.error(response)
-    )
+        $.each response.items, (index, value) ->
+          $('#select_provider').append($('<option>').val(value).text(value))
 
+        $('span.refresh-providers.spinner').remove()
+        $('a.refresh-providers.spinner').show()
\ No newline at end of file
diff --git a/app/concerns/redirector.rb b/app/concerns/provider_context_redirector.rb
similarity index 69%
rename from app/concerns/redirector.rb
rename to app/concerns/provider_context_redirector.rb
index c1c89fc1a..3d977fab3 100644
--- a/app/concerns/redirector.rb
+++ b/app/concerns/provider_context_redirector.rb
@@ -1,4 +1,5 @@
-module Redirector
+# :nodoc:
+module ProviderContextRedirector
   extend ActiveSupport::Concern
 
   routes = Rails.application.routes.url_helpers
@@ -7,9 +8,10 @@ module Redirector
   # for a given path is "/provider_orders", we will redirect to the '/orders'
   # path.
   ROUTE_EXCEPTIONS = {
-    'pages' => routes.manage_metadata_path,
+    'manage_cmr'      => routes.manage_cmr_path,
+    'pages'           => routes.manage_metadata_path,
     'provider_orders' => routes.orders_path
-  }
+  }.freeze
 
   def get_redirect_route(original_route)
     # Get the controller name from the original path
@@ -18,6 +20,8 @@ def get_redirect_route(original_route)
     # Return the user to the "index" action of the original controller they were on,
     # unless it's in the exception list.
     ROUTE_EXCEPTIONS.include?(controller_name) ? ROUTE_EXCEPTIONS[controller_name] : url_for(action: 'index', controller: controller_name)
+  rescue ActionController::UrlGenerationError
+    # If we missed any route exceptions fallback to manage metadata
+    Rails.application.routes.url_helpers.manage_metadata_path
   end
-
-end
\ No newline at end of file
+end
diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index 40866433c..76ea64f47 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -1,5 +1,7 @@
+# :nodoc:
 class UsersController < ApplicationController
-  include Redirector
+  include ProviderContextRedirector
+
   skip_before_filter :is_logged_in, except: [:set_provider, :refresh_providers]
   skip_before_filter :setup_query
 
diff --git a/spec/features/provider_context/provider_context_spec.rb b/spec/features/provider_context/provider_context_spec.rb
index 78630e35e..ee485a9dc 100644
--- a/spec/features/provider_context/provider_context_spec.rb
+++ b/spec/features/provider_context/provider_context_spec.rb
@@ -2,10 +2,10 @@
 
 require 'rails_helper'
 
-describe 'Provider context', js: true do
+describe 'Provider context', reset_provider: true, js: true do
   let(:order_guid) { 'FF330AD3-1A89-871C-AC94-B689A5C95723' }
 
-  context 'when the user has multiple possible provider contexts' do
+  context 'when the user has multiple providers' do
     before do
       login(providers: nil)
     end