diff options
| author | Sage Ross <ragesoss@gmail.com> | 2021-12-31 00:27:48 +0300 |
|---|---|---|
| committer | Benjamin Neff <benjamin@coding4coffee.ch> | 2022-06-19 20:29:57 +0300 |
| commit | 688245c0de784efbb60c61f83a037728a06c5f03 (patch) | |
| tree | 42c455067117ca5f96650147bedaf89c2120bf3c /features | |
| parent | 245bc1a05fb281ee046785a2e6ea763ac2693536 (diff) | |
Remove unnecessary `execute_script` call in feature spec
This call to `execute_script` would directly call jQuery's `sortable` function on an element to make it sortable. However, this isn't necessary; the test can still drag the "Cat People" aspect to the desired position without it.
Diffstat (limited to 'features')
| -rw-r--r-- | features/step_definitions/aspects_steps.rb | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/features/step_definitions/aspects_steps.rb b/features/step_definitions/aspects_steps.rb index 4a5c9e577..5be963ddb 100644 --- a/features/step_definitions/aspects_steps.rb +++ b/features/step_definitions/aspects_steps.rb @@ -119,8 +119,6 @@ When /^(.*) in the aspect creation modal$/ do |action| end When /^I drag "([^"]*)" (up|down)$/ do |aspect_name, direction| - expect(page).to have_js_defined("$('body').sortable") - page.execute_script("$('#aspect_nav .list-group').sortable('option', 'tolerance', 'pointer');") aspect_id = @me.aspects.where(name: aspect_name).first.id aspect = find(:xpath, "//div[@id='aspect_nav']/ul/a[@data-aspect-id='#{aspect_id}']") target = direction == "up" ? aspect.all(:xpath, "./preceding-sibling::a").last : |