2019-04-30 08:27:42 +08:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2015-10-11 17:41:23 +08:00
|
|
|
require 'rails_helper'
|
2013-02-06 03:16:51 +08:00
|
|
|
|
|
|
|
describe DraftController do
|
|
|
|
it 'requires you to be logged in' do
|
2018-06-07 12:24:20 +08:00
|
|
|
post "/draft"
|
2018-01-12 11:15:10 +08:00
|
|
|
expect(response.status).to eq(403)
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'saves a draft on update' do
|
2018-06-07 12:24:20 +08:00
|
|
|
user = sign_in(Fabricate(:user))
|
2018-11-15 00:47:59 +08:00
|
|
|
|
|
|
|
post "/draft.json", params: {
|
|
|
|
draft_key: 'xyz',
|
|
|
|
data: { my: "data" }.to_json,
|
|
|
|
sequence: 0
|
|
|
|
}
|
|
|
|
|
2018-06-07 12:24:20 +08:00
|
|
|
expect(response.status).to eq(200)
|
2018-11-15 00:47:59 +08:00
|
|
|
expect(Draft.get(user, 'xyz', 0)).to eq(%q({"my":"data"}))
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
2018-11-14 19:56:25 +08:00
|
|
|
it 'checks for an conflict on update' do
|
|
|
|
user = sign_in(Fabricate(:user))
|
|
|
|
post = Fabricate(:post, user: user)
|
|
|
|
|
|
|
|
post "/draft.json", params: {
|
|
|
|
draft_key: "topic",
|
|
|
|
sequence: 0,
|
2018-11-15 00:47:59 +08:00
|
|
|
data: {
|
|
|
|
postId: post.id,
|
2018-11-15 10:14:07 +08:00
|
|
|
originalText: post.raw,
|
|
|
|
action: "edit"
|
2018-11-15 00:47:59 +08:00
|
|
|
}.to_json
|
2018-11-14 19:56:25 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
expect(JSON.parse(response.body)['conflict_user']).to eq(nil)
|
|
|
|
|
|
|
|
post "/draft.json", params: {
|
|
|
|
draft_key: "topic",
|
|
|
|
sequence: 0,
|
2018-11-15 00:47:59 +08:00
|
|
|
data: {
|
|
|
|
postId: post.id,
|
2018-11-15 10:14:07 +08:00
|
|
|
originalText: "something else",
|
|
|
|
action: "edit"
|
2018-11-15 00:47:59 +08:00
|
|
|
}.to_json
|
2018-11-14 19:56:25 +08:00
|
|
|
}
|
|
|
|
|
2018-11-15 00:47:59 +08:00
|
|
|
json = JSON.parse(response.body)
|
|
|
|
|
|
|
|
expect(json['conflict_user']['id']).to eq(post.last_editor.id)
|
|
|
|
expect(json['conflict_user']).to include('avatar_template')
|
2018-11-14 19:56:25 +08:00
|
|
|
end
|
|
|
|
|
2013-02-26 00:42:20 +08:00
|
|
|
it 'destroys drafts when required' do
|
2018-06-07 12:24:20 +08:00
|
|
|
user = sign_in(Fabricate(:user))
|
2013-02-06 03:16:51 +08:00
|
|
|
Draft.set(user, 'xxx', 0, 'hi')
|
2018-06-07 12:24:20 +08:00
|
|
|
delete "/draft.json", params: { draft_key: 'xxx', sequence: 0 }
|
|
|
|
expect(response.status).to eq(200)
|
2015-01-10 01:04:02 +08:00
|
|
|
expect(Draft.get(user, 'xxx', 0)).to eq(nil)
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
2019-10-31 14:15:41 +08:00
|
|
|
|
|
|
|
it 'cant trivially resolve conflicts without interaction' do
|
|
|
|
|
|
|
|
user = sign_in(Fabricate(:user))
|
|
|
|
|
|
|
|
DraftSequence.next!(user, "abc")
|
|
|
|
|
|
|
|
post "/draft.json", params: {
|
|
|
|
draft_key: "abc",
|
|
|
|
sequence: 0,
|
|
|
|
data: { a: "test" }.to_json,
|
|
|
|
owner: "abcdefg"
|
|
|
|
}
|
|
|
|
|
|
|
|
expect(response.status).to eq(200)
|
|
|
|
json = JSON.parse(response.body)
|
|
|
|
expect(json["draft_sequence"]).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'has a clean protocol for ownership handover' do
|
|
|
|
user = sign_in(Fabricate(:user))
|
|
|
|
|
|
|
|
post "/draft.json", params: {
|
|
|
|
draft_key: "abc",
|
|
|
|
sequence: 0,
|
|
|
|
data: { a: "test" }.to_json,
|
|
|
|
owner: "abcdefg"
|
|
|
|
}
|
|
|
|
|
|
|
|
expect(response.status).to eq(200)
|
|
|
|
|
|
|
|
json = JSON.parse(response.body)
|
|
|
|
expect(json["draft_sequence"]).to eq(0)
|
|
|
|
|
|
|
|
post "/draft.json", params: {
|
|
|
|
draft_key: "abc",
|
|
|
|
sequence: 0,
|
|
|
|
data: { b: "test" }.to_json,
|
|
|
|
owner: "hijklmnop"
|
|
|
|
}
|
|
|
|
|
|
|
|
expect(response.status).to eq(200)
|
|
|
|
json = JSON.parse(response.body)
|
|
|
|
expect(json["draft_sequence"]).to eq(1)
|
|
|
|
|
|
|
|
expect(DraftSequence.current(user, "abc")).to eq(1)
|
|
|
|
|
|
|
|
post "/draft.json", params: {
|
|
|
|
draft_key: "abc",
|
|
|
|
sequence: 1,
|
|
|
|
data: { c: "test" }.to_json,
|
|
|
|
owner: "hijklmnop"
|
|
|
|
}
|
|
|
|
|
|
|
|
expect(response.status).to eq(200)
|
|
|
|
json = JSON.parse(response.body)
|
|
|
|
expect(json["draft_sequence"]).to eq(1)
|
|
|
|
|
|
|
|
post "/draft.json", params: {
|
|
|
|
draft_key: "abc",
|
|
|
|
sequence: 1,
|
|
|
|
data: { c: "test" }.to_json,
|
|
|
|
owner: "abc"
|
|
|
|
}
|
|
|
|
|
|
|
|
expect(response.status).to eq(200)
|
|
|
|
json = JSON.parse(response.body)
|
|
|
|
expect(json["draft_sequence"]).to eq(2)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'raises an error for out-of-sequence draft setting' do
|
|
|
|
|
|
|
|
user = sign_in(Fabricate(:user))
|
|
|
|
seq = DraftSequence.next!(user, "abc")
|
|
|
|
Draft.set(user, "abc", seq, { b: "test" }.to_json)
|
|
|
|
|
|
|
|
post "/draft.json", params: {
|
|
|
|
draft_key: "abc",
|
|
|
|
sequence: seq - 1,
|
|
|
|
data: { a: "test" }.to_json
|
|
|
|
}
|
|
|
|
|
|
|
|
expect(response.status).to eq(409)
|
|
|
|
|
|
|
|
post "/draft.json", params: {
|
|
|
|
draft_key: "abc",
|
|
|
|
sequence: seq + 1,
|
|
|
|
data: { a: "test" }.to_json
|
|
|
|
}
|
|
|
|
|
|
|
|
expect(response.status).to eq(409)
|
|
|
|
|
|
|
|
end
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|