Merge pull request #4827 from Flaburgan/fix-aspect-renaming

Fix aspect renaming after a regression caused by jquery upgrade

app/assets/javascripts/aspect-edit-pane.js

@@ -17,7 +17,7 @@ function updatePageAspectName( an_id, new_name) {
 }
 
 $(document).ready(function() {
-  $('#aspect_name_title').on('click', '#rename_aspect_link', function(){
+  $(document).on('click', '#rename_aspect_link', function(){
     toggleAspectTitle();
   });
 

features/desktop/manages_aspects.feature

@@ -53,6 +53,17 @@ Feature: User manages contacts
 
     When I uncheck the first contact list button
     Then I should have 0 contacts in "Cat People"
+    
+  Scenario: Renaming an aspect
+    Given I am signed in
+    And I have an aspect called "Cat People"
+    When I am on the contacts page
+    And I follow "Cat People"
+    And I follow "Manage" within "#aspect_controls"
+    And I follow "rename"
+    And I fill in "aspect_name" with "Unicorn People"
+    And I press "update"
+    Then I should see "Unicorn People" within "#aspect_name_title"
 
   Scenario: infinite scroll on contacts index
     Given I am signed in