1 files changed, 23 insertions, 11 deletions

diff --git a/spec/services/issues/reorder_service_spec.rb b/spec/services/issues/reorder_service_spec.rb
index b6ad488a48c..78b937a1caf 100644
--- a/spec/services/issues/reorder_service_spec.rb
+++ b/spec/services/issues/reorder_service_spec.rb
@@ -3,9 +3,9 @@
 require 'spec_helper'
 
 RSpec.describe Issues::ReorderService do
-  let_it_be(:user)    { create(:user) }
-  let_it_be(:project) { create(:project) }
+  let_it_be(:user)    { create_default(:user) }
   let_it_be(:group)   { create(:group) }
+  let_it_be(:project, reload: true) { create(:project, namespace: group) }
 
   shared_examples 'issues reorder service' do
     context 'when reordering issues' do
@@ -14,7 +14,7 @@ RSpec.describe Issues::ReorderService do
       end
 
       it 'returns false with both invalid params' do
-        params = { move_after_id: nil, move_before_id: 1 }
+        params = { move_after_id: nil, move_before_id: non_existing_record_id }
 
         expect(service(params).execute(issue1)).to be_falsey
       end
@@ -27,27 +27,39 @@ RSpec.describe Issues::ReorderService do
         expect(issue1.relative_position)
           .to be_between(issue2.relative_position, issue3.relative_position)
       end
+
+      it 'sorts issues if only given one neighbour, on the left' do
+        params = { move_before_id: issue3.id }
+
+        service(params).execute(issue1)
+
+        expect(issue1.relative_position).to be > issue3.relative_position
+      end
+
+      it 'sorts issues if only given one neighbour, on the right' do
+        params = { move_after_id: issue1.id }
+
+        service(params).execute(issue3)
+
+        expect(issue3.relative_position).to be < issue1.relative_position
+      end
     end
   end
 
   describe '#execute' do
-    let(:issue1) { create(:issue, project: project, relative_position: 10) }
-    let(:issue2) { create(:issue, project: project, relative_position: 20) }
-    let(:issue3) { create(:issue, project: project, relative_position: 30) }
+    let_it_be(:issue1, reload: true) { create(:issue, project: project, relative_position: 10) }
+    let_it_be(:issue2) { create(:issue, project: project, relative_position: 20) }
+    let_it_be(:issue3, reload: true) { create(:issue, project: project, relative_position: 30) }
 
     context 'when ordering issues in a project' do
-      let(:parent) { project }
-
       before do
-        parent.add_developer(user)
+        project.add_developer(user)
       end
 
       it_behaves_like 'issues reorder service'
     end
 
     context 'when ordering issues in a group' do
-      let(:project) { create(:project, namespace: group) }
-
       before do
         group.add_developer(user)
       end