BigW Consortium Gitlab

resolvable_note_spec.rb 8.51 KB
Newer Older
1 2
require 'spec_helper'

3
describe Note, ResolvableNote do
4
  let(:project) { create(:project, :repository) }
Douwe Maan committed
5 6
  let(:merge_request) { create(:merge_request, source_project: project) }
  subject { create(:discussion_note_on_merge_request, noteable: merge_request, project: project) }
7

Douwe Maan committed
8
  context 'resolvability scopes' do
Douwe Maan committed
9 10 11 12 13 14
    let!(:note1) { create(:note, project: project) }
    let!(:note2) { create(:diff_note_on_commit, project: project) }
    let!(:note3) { create(:diff_note_on_merge_request, :resolved, noteable: merge_request, project: project) }
    let!(:note4) { create(:discussion_note_on_merge_request, noteable: merge_request, project: project) }
    let!(:note5) { create(:discussion_note_on_issue, project: project) }
    let!(:note6) { create(:discussion_note_on_merge_request, :system, noteable: merge_request, project: project) }
Douwe Maan committed
15 16 17 18 19 20

    describe '.potentially_resolvable' do
      it 'includes diff and discussion notes on merge requests' do
        expect(Note.potentially_resolvable).to match_array([note3, note4, note6])
      end
    end
21

Douwe Maan committed
22 23 24 25 26 27 28 29 30 31 32
    describe '.resolvable' do
      it 'includes non-system diff and discussion notes on merge requests' do
        expect(Note.resolvable).to match_array([note3, note4])
      end
    end

    describe '.resolved' do
      it 'includes resolved non-system diff and discussion notes on merge requests' do
        expect(Note.resolved).to match_array([note3])
      end
    end
33

Douwe Maan committed
34 35 36 37 38
    describe '.unresolved' do
      it 'includes non-resolved non-system diff and discussion notes on merge requests' do
        expect(Note.unresolved).to match_array([note4])
      end
    end
39 40 41 42
  end

  describe ".resolve!" do
    let(:current_user) { create(:user) }
Douwe Maan committed
43 44 45
    let!(:commit_note) { create(:diff_note_on_commit, project: project) }
    let!(:resolved_note) { create(:discussion_note_on_merge_request, :resolved, noteable: merge_request, project: project) }
    let!(:unresolved_note) { create(:discussion_note_on_merge_request, noteable: merge_request, project: project) }
46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63

    before do
      described_class.resolve!(current_user)

      commit_note.reload
      resolved_note.reload
      unresolved_note.reload
    end

    it 'resolves only the resolvable, not yet resolved notes' do
      expect(commit_note.resolved_at).to be_nil
      expect(resolved_note.resolved_by).not_to eq(current_user)
      expect(unresolved_note.resolved_at).not_to be_nil
      expect(unresolved_note.resolved_by).to eq(current_user)
    end
  end

  describe ".unresolve!" do
Douwe Maan committed
64
    let!(:resolved_note) { create(:discussion_note_on_merge_request, :resolved, noteable: merge_request, project: project) }
65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80

    before do
      described_class.unresolve!

      resolved_note.reload
    end

    it 'unresolves the resolved notes' do
      expect(resolved_note.resolved_by).to be_nil
      expect(resolved_note.resolved_at).to be_nil
    end
  end

  describe '#resolvable?' do
    context "when potentially resolvable" do
      before do
Douwe Maan committed
81
        allow(subject).to receive(:potentially_resolvable?).and_return(true)
82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102
      end

      context "when a system note" do
        before do
          subject.system = true
        end

        it "returns false" do
          expect(subject.resolvable?).to be false
        end
      end

      context "when a regular note" do
        it "returns true" do
          expect(subject.resolvable?).to be true
        end
      end
    end

    context "when not potentially resolvable" do
      before do
Douwe Maan committed
103
        allow(subject).to receive(:potentially_resolvable?).and_return(false)
104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150
      end

      it "returns false" do
        expect(subject.resolvable?).to be false
      end
    end
  end

  describe "#to_be_resolved?" do
    context "when not resolvable" do
      before do
        allow(subject).to receive(:resolvable?).and_return(false)
      end

      it "returns false" do
        expect(subject.to_be_resolved?).to be false
      end
    end

    context "when resolvable" do
      before do
        allow(subject).to receive(:resolvable?).and_return(true)
      end

      context "when resolved" do
        before do
          allow(subject).to receive(:resolved?).and_return(true)
        end

        it "returns false" do
          expect(subject.to_be_resolved?).to be false
        end
      end

      context "when not resolved" do
        before do
          allow(subject).to receive(:resolved?).and_return(false)
        end

        it "returns true" do
          expect(subject.to_be_resolved?).to be true
        end
      end
    end
  end

  describe "#resolved?" do
Douwe Maan committed
151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181
    let(:current_user) { create(:user) }

    context 'when not resolvable' do
      before do
        subject.resolve!(current_user)

        allow(subject).to receive(:resolvable?).and_return(false)
      end

      it 'returns false' do
        expect(subject.resolved?).to be_falsey
      end
    end

    context 'when resolvable' do
      context 'when the note has been resolved' do
        before do
          subject.resolve!(current_user)
        end

        it 'returns true' do
          expect(subject.resolved?).to be_truthy
        end
      end

      context 'when the note has not been resolved' do
        it 'returns false' do
          expect(subject.resolved?).to be_falsey
        end
      end
    end
182 183 184 185 186 187 188 189 190 191
  end

  describe "#resolve!" do
    let(:current_user) { create(:user) }

    context "when not resolvable" do
      before do
        allow(subject).to receive(:resolvable?).and_return(false)
      end

192 193
      it "returns false" do
        expect(subject.resolve!(current_user)).to be_falsey
194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226
      end

      it "doesn't set resolved_at" do
        subject.resolve!(current_user)

        expect(subject.resolved_at).to be_nil
      end

      it "doesn't set resolved_by" do
        subject.resolve!(current_user)

        expect(subject.resolved_by).to be_nil
      end

      it "doesn't mark as resolved" do
        subject.resolve!(current_user)

        expect(subject.resolved?).to be false
      end
    end

    context "when resolvable" do
      before do
        allow(subject).to receive(:resolvable?).and_return(true)
      end

      context "when already resolved" do
        let(:user) { create(:user) }

        before do
          subject.resolve!(user)
        end

227 228
        it "returns false" do
          expect(subject.resolve!(current_user)).to be_falsey
229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281
        end

        it "doesn't change resolved_at" do
          expect(subject.resolved_at).not_to be_nil

          expect { subject.resolve!(current_user) }.not_to change { subject.resolved_at }
        end

        it "doesn't change resolved_by" do
          expect(subject.resolved_by).to eq(user)

          expect { subject.resolve!(current_user) }.not_to change { subject.resolved_by }
        end

        it "doesn't change resolved status" do
          expect(subject.resolved?).to be true

          expect { subject.resolve!(current_user) }.not_to change { subject.resolved? }
        end
      end

      context "when not yet resolved" do
        it "returns true" do
          expect(subject.resolve!(current_user)).to be true
        end

        it "sets resolved_at" do
          subject.resolve!(current_user)

          expect(subject.resolved_at).not_to be_nil
        end

        it "sets resolved_by" do
          subject.resolve!(current_user)

          expect(subject.resolved_by).to eq(current_user)
        end

        it "marks as resolved" do
          subject.resolve!(current_user)

          expect(subject.resolved?).to be true
        end
      end
    end
  end

  describe "#unresolve!" do
    context "when not resolvable" do
      before do
        allow(subject).to receive(:resolvable?).and_return(false)
      end

282 283
      it "returns false" do
        expect(subject.unresolve!).to be_falsey
284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322
      end
    end

    context "when resolvable" do
      before do
        allow(subject).to receive(:resolvable?).and_return(true)
      end

      context "when resolved" do
        let(:user) { create(:user) }

        before do
          subject.resolve!(user)
        end

        it "returns true" do
          expect(subject.unresolve!).to be true
        end

        it "unsets resolved_at" do
          subject.unresolve!

          expect(subject.resolved_at).to be_nil
        end

        it "unsets resolved_by" do
          subject.unresolve!

          expect(subject.resolved_by).to be_nil
        end

        it "unmarks as resolved" do
          subject.unresolve!

          expect(subject.resolved?).to be false
        end
      end

      context "when not resolved" do
323 324
        it "returns false" do
          expect(subject.unresolve!).to be_falsey
325 326 327 328 329
        end
      end
    end
  end
end