mirror of
https://github.com/zulip/zulip.git
synced 2026-06-24 21:08:25 +08:00
Now whenever we initiate sending a message, we save / update its draft, which is deleted on a successful send. Earlier, we did this only for locally echoed messages. Hence a non locally echoed message's draft would remain, if created in the timeframe between initiating send and receiving the same message from the server, which can be significant for slow connections.
335 lines
9.4 KiB
JavaScript
335 lines
9.4 KiB
JavaScript
"use strict";
|
|
|
|
const {strict: assert} = require("assert");
|
|
|
|
const MockDate = require("mockdate");
|
|
|
|
const {mock_esm, zrequire} = require("./lib/namespace");
|
|
const {make_stub} = require("./lib/stub");
|
|
const {run_test, noop} = require("./lib/test");
|
|
const {current_user} = require("./lib/zpage_params");
|
|
|
|
const compose_notifications = mock_esm("../src/compose_notifications");
|
|
const markdown = mock_esm("../src/markdown");
|
|
const message_lists = mock_esm("../src/message_lists");
|
|
|
|
let disparities = [];
|
|
|
|
mock_esm("../src/message_live_update", {
|
|
update_message_in_all_views() {},
|
|
});
|
|
|
|
mock_esm("../src/sent_messages", {
|
|
mark_disparity(local_id) {
|
|
disparities.push(local_id);
|
|
},
|
|
report_event_received() {},
|
|
});
|
|
|
|
const message_store = mock_esm("../src/message_store", {
|
|
get: () => ({failed_request: true}),
|
|
|
|
update_booleans() {},
|
|
|
|
set_message_booleans() {},
|
|
});
|
|
|
|
message_lists.current = {
|
|
view: {
|
|
rerender_messages: noop,
|
|
change_message_id: noop,
|
|
},
|
|
change_message_id: noop,
|
|
};
|
|
message_lists.home = {
|
|
view: {
|
|
rerender_messages: noop,
|
|
change_message_id: noop,
|
|
},
|
|
change_message_id: noop,
|
|
};
|
|
message_lists.all_rendered_message_lists = () => [message_lists.home, message_lists.current];
|
|
|
|
const echo = zrequire("echo");
|
|
const people = zrequire("people");
|
|
const stream_data = zrequire("stream_data");
|
|
|
|
const general_sub = {
|
|
stream_id: 101,
|
|
name: "general",
|
|
subscribed: true,
|
|
};
|
|
stream_data.add_sub(general_sub);
|
|
|
|
run_test("process_from_server for un-echoed messages", () => {
|
|
const waiting_for_ack = new Map();
|
|
const server_messages = [
|
|
{
|
|
local_id: "100.1",
|
|
},
|
|
];
|
|
echo._patch_waiting_for_ack(waiting_for_ack);
|
|
const non_echo_messages = echo.process_from_server(server_messages);
|
|
assert.deepEqual(non_echo_messages, server_messages);
|
|
});
|
|
|
|
run_test("process_from_server for differently rendered messages", ({override}) => {
|
|
let messages_to_rerender = [];
|
|
|
|
override(message_lists.home.view, "rerender_messages", (msgs) => {
|
|
messages_to_rerender = msgs;
|
|
});
|
|
|
|
// Test that we update all the booleans and the content of the message
|
|
// in local echo.
|
|
const old_value = "old_value";
|
|
const new_value = "new_value";
|
|
const waiting_for_ack = new Map([
|
|
[
|
|
"100.1",
|
|
{
|
|
content: "<p>A client rendered message</p>",
|
|
timestamp: old_value,
|
|
is_me_message: old_value,
|
|
submessages: old_value,
|
|
topic_links: old_value,
|
|
},
|
|
],
|
|
]);
|
|
const server_messages = [
|
|
{
|
|
local_id: "100.1",
|
|
content: "<p>A server rendered message</p>",
|
|
timestamp: new_value,
|
|
is_me_message: new_value,
|
|
submessages: new_value,
|
|
topic_links: new_value,
|
|
},
|
|
];
|
|
echo._patch_waiting_for_ack(waiting_for_ack);
|
|
disparities = [];
|
|
const non_echo_messages = echo.process_from_server(server_messages);
|
|
assert.deepEqual(non_echo_messages, []);
|
|
assert.equal(disparities.length, 1);
|
|
assert.deepEqual(messages_to_rerender, [
|
|
{
|
|
content: server_messages[0].content,
|
|
timestamp: new_value,
|
|
is_me_message: new_value,
|
|
submessages: new_value,
|
|
topic_links: new_value,
|
|
},
|
|
]);
|
|
});
|
|
|
|
run_test("build_display_recipient", () => {
|
|
current_user.user_id = 123;
|
|
|
|
const params = {};
|
|
params.realm_users = [
|
|
{
|
|
user_id: 123,
|
|
full_name: "Iago",
|
|
email: "[email protected]",
|
|
},
|
|
{
|
|
email: "[email protected]",
|
|
full_name: "Cordelia",
|
|
user_id: 21,
|
|
},
|
|
];
|
|
params.realm_non_active_users = [];
|
|
params.cross_realm_bots = [];
|
|
people.initialize(current_user.user_id, params);
|
|
|
|
let message = {
|
|
type: "stream",
|
|
stream_id: general_sub.stream_id,
|
|
sender_email: "[email protected]",
|
|
sender_full_name: "Iago",
|
|
sender_id: 123,
|
|
};
|
|
let display_recipient = echo.build_display_recipient(message);
|
|
assert.equal(display_recipient, "general");
|
|
|
|
message = {
|
|
type: "private",
|
|
private_message_recipient: "[email protected],[email protected]",
|
|
sender_email: "[email protected]",
|
|
sender_full_name: "Iago",
|
|
sender_id: 123,
|
|
};
|
|
display_recipient = echo.build_display_recipient(message);
|
|
assert.equal(display_recipient.length, 3);
|
|
|
|
let iago = display_recipient.find((recipient) => recipient.email === "[email protected]");
|
|
assert.equal(iago.full_name, "Iago");
|
|
assert.equal(iago.id, 123);
|
|
|
|
const cordelia = display_recipient.find(
|
|
(recipient) => recipient.email === "[email protected]",
|
|
);
|
|
assert.equal(cordelia.full_name, "Cordelia");
|
|
assert.equal(cordelia.id, 21);
|
|
|
|
const hamlet = display_recipient.find((recipient) => recipient.email === "[email protected]");
|
|
assert.equal(hamlet.full_name, "[email protected]");
|
|
assert.equal(hamlet.id, undefined);
|
|
assert.equal(hamlet.unknown_local_echo_user, true);
|
|
|
|
message = {
|
|
type: "private",
|
|
private_message_recipient: "[email protected]",
|
|
sender_email: "[email protected]",
|
|
sender_full_name: "Iago",
|
|
sender_id: 123,
|
|
};
|
|
display_recipient = echo.build_display_recipient(message);
|
|
|
|
assert.equal(display_recipient.length, 1);
|
|
iago = display_recipient.find((recipient) => recipient.email === "[email protected]");
|
|
assert.equal(iago.full_name, "Iago");
|
|
assert.equal(iago.id, 123);
|
|
});
|
|
|
|
run_test("update_message_lists", () => {
|
|
message_lists.home.view = {};
|
|
|
|
const stub = make_stub();
|
|
const view_stub = make_stub();
|
|
|
|
message_lists.home.change_message_id = stub.f;
|
|
message_lists.home.view.change_message_id = view_stub.f;
|
|
|
|
echo.update_message_lists({old_id: 401, new_id: 402});
|
|
|
|
assert.equal(stub.num_calls, 1);
|
|
const args = stub.get_args("old", "new");
|
|
assert.equal(args.old, 401);
|
|
assert.equal(args.new, 402);
|
|
|
|
assert.equal(view_stub.num_calls, 1);
|
|
const view_args = view_stub.get_args("old", "new");
|
|
assert.equal(view_args.old, 401);
|
|
assert.equal(view_args.new, 402);
|
|
});
|
|
|
|
run_test("insert_local_message streams", ({override}) => {
|
|
const fake_now = 555;
|
|
MockDate.set(new Date(fake_now * 1000));
|
|
|
|
const local_id_float = 101.01;
|
|
|
|
let render_called = false;
|
|
let get_topic_links_called = false;
|
|
let insert_message_called = false;
|
|
|
|
override(markdown, "render", () => {
|
|
render_called = true;
|
|
});
|
|
|
|
override(markdown, "get_topic_links", () => {
|
|
get_topic_links_called = true;
|
|
});
|
|
|
|
const insert_new_messages = ([message]) => {
|
|
assert.equal(message.display_recipient, "general");
|
|
assert.equal(message.timestamp, fake_now);
|
|
assert.equal(message.sender_email, "[email protected]");
|
|
assert.equal(message.sender_full_name, "Iago");
|
|
assert.equal(message.sender_id, 123);
|
|
insert_message_called = true;
|
|
};
|
|
|
|
const message_request = {
|
|
type: "stream",
|
|
stream_id: general_sub.stream_id,
|
|
topic: "important note",
|
|
sender_email: "[email protected]",
|
|
sender_full_name: "Iago",
|
|
sender_id: 123,
|
|
};
|
|
echo.insert_local_message(message_request, local_id_float, insert_new_messages);
|
|
|
|
assert.ok(render_called);
|
|
assert.ok(get_topic_links_called);
|
|
assert.ok(insert_message_called);
|
|
});
|
|
|
|
run_test("insert_local_message direct message", ({override}) => {
|
|
const local_id_float = 102.01;
|
|
|
|
current_user.user_id = 123;
|
|
|
|
const params = {};
|
|
params.realm_users = [
|
|
{
|
|
user_id: 123,
|
|
full_name: "Iago",
|
|
email: "[email protected]",
|
|
},
|
|
];
|
|
params.realm_non_active_users = [];
|
|
params.cross_realm_bots = [];
|
|
people.initialize(current_user.user_id, params);
|
|
|
|
let render_called = false;
|
|
let insert_message_called = false;
|
|
|
|
const insert_new_messages = ([message]) => {
|
|
assert.equal(message.display_recipient.length, 3);
|
|
insert_message_called = true;
|
|
};
|
|
|
|
override(markdown, "render", () => {
|
|
render_called = true;
|
|
});
|
|
|
|
const message_request = {
|
|
private_message_recipient: "[email protected],[email protected]",
|
|
type: "private",
|
|
sender_email: "[email protected]",
|
|
sender_full_name: "Iago",
|
|
sender_id: 123,
|
|
};
|
|
echo.insert_local_message(message_request, local_id_float, insert_new_messages);
|
|
assert.ok(render_called);
|
|
assert.ok(insert_message_called);
|
|
});
|
|
|
|
run_test("test reify_message_id", ({override}) => {
|
|
const local_id_float = 103.01;
|
|
|
|
override(markdown, "render", noop);
|
|
|
|
const message_request = {
|
|
type: "stream",
|
|
stream_id: general_sub.stream_id,
|
|
sender_email: "[email protected]",
|
|
sender_full_name: "Iago",
|
|
sender_id: 123,
|
|
draft_id: 100,
|
|
};
|
|
echo.insert_local_message(message_request, local_id_float, noop);
|
|
|
|
let message_store_reify_called = false;
|
|
let notifications_reify_called = false;
|
|
|
|
override(message_store, "reify_message_id", () => {
|
|
message_store_reify_called = true;
|
|
});
|
|
|
|
override(compose_notifications, "reify_message_id", () => {
|
|
notifications_reify_called = true;
|
|
});
|
|
|
|
echo.reify_message_id(local_id_float.toString(), 110);
|
|
|
|
assert.ok(message_store_reify_called);
|
|
assert.ok(notifications_reify_called);
|
|
});
|
|
|
|
run_test("reset MockDate", () => {
|
|
MockDate.reset();
|
|
});
|