zulip/web/tests/message_edit.test.js

367 lines
12 KiB
JavaScript
Raw Normal View History

"use strict";
const {strict: assert} = require("assert");
const {mock_esm, zrequire} = require("./lib/namespace");
const {run_test} = require("./lib/test");
const {current_user, realm} = require("./lib/zpage_params");
realm.realm_move_messages_within_stream_limit_seconds = 259200;
const message_edit = zrequire("message_edit");
const people = zrequire("people");
const is_content_editable = message_edit.is_content_editable;
const settings_data = mock_esm("../src/settings_data");
run_test("is_content_editable", () => {
// You can't edit a null message
assert.equal(is_content_editable(null), false);
// You can't edit a message you didn't send
assert.equal(
is_content_editable({
sent_by_me: false,
}),
false,
);
// Failed request are currently not editable (though we want to
// change this back).
assert.equal(
is_content_editable({
sent_by_me: true,
failed_request: true,
}),
false,
);
// Locally echoed messages are not editable, since the message hasn't
// finished being sent yet.
assert.equal(
is_content_editable({
sent_by_me: true,
locally_echoed: true,
}),
false,
);
// For the rest of these tests, we only consider messages sent by the
// user, and that were successfully sent (i.e. no failed_request or local_id)
const message = {
sent_by_me: true,
};
realm.realm_allow_message_editing = false;
assert.equal(is_content_editable(message), false);
realm.realm_allow_message_editing = true;
// Limit of 0 means no time limit on editing messages
realm.realm_message_content_edit_limit_seconds = null;
assert.equal(is_content_editable(message), true);
realm.realm_message_content_edit_limit_seconds = 10;
const now = new Date();
const current_timestamp = now / 1000;
message.timestamp = current_timestamp - 60;
// Have 55+10 > 60 seconds from message.timestamp to edit the message; we're good!
assert.equal(is_content_editable(message, 55), true);
// It's been 60 > 45+10 since message.timestamp. When realm_allow_message_editing
// is true, we can edit the topic if there is one.
assert.equal(is_content_editable(message, 45), false);
// Right now, we prevent users from editing widgets.
message.submessages = ["/poll"];
assert.equal(is_content_editable(message, 55), false);
delete message.submessages;
message.type = "private";
assert.equal(is_content_editable(message, 45), false);
assert.equal(is_content_editable(message, 55), true);
// If we don't pass a second argument, treat it as 0
assert.equal(is_content_editable(message), false);
});
run_test("is_topic_editable", ({override}) => {
const now = new Date();
const current_timestamp = now / 1000;
const message = {
sent_by_me: true,
message_edit: Do not allow moving locally echoed or failed messages. We should not show UI for moving locally echoed or failed messages. For the edit/move icon in message controls - - After 2451002, we showed "Move message" icon for locally echoed messages when moving messages was allowed as per org permissions. This was clearly a bug and this commit fixes it and we instead show "View source" icon. For "m" hotkey - - After 2451002, pressing "m" opened "Move messages" modal for locally echoed messages when moving messages was allowed as per org permissions. This was clearly a bug and this commit fixes it and pressing "m" does nothing. For pencil icon in recipient header - - Previously, the pencil icon was showed even if first message below the recipient bar was locally echoed and the topic edit failed silently with locally echoed or failed messages again showing under old topic after reload. This commit fixes it to hide the icon in such cases. The original behavior was that server returned an error with not so appropriate response. - Note that we can only check the first message below the header since it will be inefficient to check all the messages whether each of them is successfully sent. And in such cases when any of the later messages is locally echoed, the topic edit succeeds but the locally echoed or failed messages are shown under old topic after reload. - The behavior is same for the "checkmark" icon for resolving topics in the header. For three-dot message menu - - We don't need to do anything here since the three-dot icon is not shown for locally echoed or failed messages. For the "Move topic" option in topic popover in left sidebar - - We cannot do anything here since it will be inefficient to check all messages in the topic whether they are locally echoed or not everytime we open the popover. - If the first message in topic is locally echoed or a failed message, then the id of first message of the topic returned from server is undefined and we see an error in client while trying to get message from that id. And if one or multiple of the later messages is locally echoed, then only successful and server-acked messages are moved. Note that this cannot handle all cases like a user can use a successfully sent message to access the topic edit UI and then choose "change_all" and "change_later" propagate modes. In such cases the locally echoed and failed messages will remain in the old topic only.
2022-10-20 16:29:57 +02:00
locally_echoed: true,
type: "stream",
};
realm.realm_allow_message_editing = true;
override(settings_data, "user_can_move_messages_to_another_topic", () => true);
current_user.is_admin = true;
message_edit: Do not allow moving locally echoed or failed messages. We should not show UI for moving locally echoed or failed messages. For the edit/move icon in message controls - - After 2451002, we showed "Move message" icon for locally echoed messages when moving messages was allowed as per org permissions. This was clearly a bug and this commit fixes it and we instead show "View source" icon. For "m" hotkey - - After 2451002, pressing "m" opened "Move messages" modal for locally echoed messages when moving messages was allowed as per org permissions. This was clearly a bug and this commit fixes it and pressing "m" does nothing. For pencil icon in recipient header - - Previously, the pencil icon was showed even if first message below the recipient bar was locally echoed and the topic edit failed silently with locally echoed or failed messages again showing under old topic after reload. This commit fixes it to hide the icon in such cases. The original behavior was that server returned an error with not so appropriate response. - Note that we can only check the first message below the header since it will be inefficient to check all the messages whether each of them is successfully sent. And in such cases when any of the later messages is locally echoed, the topic edit succeeds but the locally echoed or failed messages are shown under old topic after reload. - The behavior is same for the "checkmark" icon for resolving topics in the header. For three-dot message menu - - We don't need to do anything here since the three-dot icon is not shown for locally echoed or failed messages. For the "Move topic" option in topic popover in left sidebar - - We cannot do anything here since it will be inefficient to check all messages in the topic whether they are locally echoed or not everytime we open the popover. - If the first message in topic is locally echoed or a failed message, then the id of first message of the topic returned from server is undefined and we see an error in client while trying to get message from that id. And if one or multiple of the later messages is locally echoed, then only successful and server-acked messages are moved. Note that this cannot handle all cases like a user can use a successfully sent message to access the topic edit UI and then choose "change_all" and "change_later" propagate modes. In such cases the locally echoed and failed messages will remain in the old topic only.
2022-10-20 16:29:57 +02:00
assert.equal(message_edit.is_topic_editable(message), false);
message.locally_echoed = false;
message.failed_request = true;
assert.equal(message_edit.is_topic_editable(message), false);
message.failed_request = false;
assert.equal(message_edit.is_topic_editable(message), true);
message.sent_by_me = false;
assert.equal(message_edit.is_topic_editable(message), true);
override(settings_data, "user_can_move_messages_to_another_topic", () => false);
assert.equal(message_edit.is_topic_editable(message), false);
current_user.is_admin = false;
assert.equal(message_edit.is_topic_editable(message), false);
message.topic = "translated: (no topic)";
assert.equal(message_edit.is_topic_editable(message), false);
message.topic = "test topic";
override(settings_data, "user_can_move_messages_to_another_topic", () => false);
assert.equal(message_edit.is_topic_editable(message), false);
realm.realm_move_messages_within_stream_limit_seconds = 259200;
message.timestamp = current_timestamp - 60;
override(settings_data, "user_can_move_messages_to_another_topic", () => true);
assert.equal(message_edit.is_topic_editable(message), true);
message.timestamp = current_timestamp - 600000;
assert.equal(message_edit.is_topic_editable(message), false);
current_user.is_moderator = true;
assert.equal(message_edit.is_topic_editable(message), true);
realm.realm_allow_message_editing = false;
assert.equal(message_edit.is_topic_editable(message), true);
});
run_test("is_stream_editable", ({override}) => {
const now = new Date();
const current_timestamp = now / 1000;
const message = {
sent_by_me: true,
locally_echoed: true,
type: "stream",
};
realm.realm_allow_message_editing = true;
override(settings_data, "user_can_move_messages_between_streams", () => true);
current_user.is_admin = true;
assert.equal(message_edit.is_stream_editable(message), false);
message.locally_echoed = false;
message.failed_request = true;
assert.equal(message_edit.is_stream_editable(message), false);
message.failed_request = false;
assert.equal(message_edit.is_stream_editable(message), true);
message.sent_by_me = false;
assert.equal(message_edit.is_stream_editable(message), true);
override(settings_data, "user_can_move_messages_between_streams", () => false);
assert.equal(message_edit.is_stream_editable(message), false);
current_user.is_admin = false;
assert.equal(message_edit.is_stream_editable(message), false);
realm.realm_move_messages_between_streams_limit_seconds = 259200;
message.timestamp = current_timestamp - 60;
override(settings_data, "user_can_move_messages_between_streams", () => true);
assert.equal(message_edit.is_stream_editable(message), true);
message.timestamp = current_timestamp - 600000;
assert.equal(message_edit.is_stream_editable(message), false);
current_user.is_moderator = true;
assert.equal(message_edit.is_stream_editable(message), true);
});
run_test("get_deletability", ({override}) => {
current_user.is_admin = true;
override(settings_data, "user_can_delete_own_message", () => false);
realm.realm_message_content_delete_limit_seconds = null;
const test_user = {
user_id: 1,
full_name: "Test user",
email: "test@zulip.com",
};
people.add_active_user(test_user);
const bot_user = {
user_id: 2,
full_name: "Test bot user",
email: "test-bot@zulip.com",
bot_owner_id: 1,
};
people.add_active_user(bot_user);
const message = {
sent_by_me: false,
locally_echoed: true,
sender_id: 1,
};
// Admin can always delete any message
assert.equal(message_edit.get_deletability(message), true);
// Non-admin can't delete message sent by others
current_user.is_admin = false;
assert.equal(message_edit.get_deletability(message), false);
// Locally echoed messages are not deletable
message.sent_by_me = true;
assert.equal(message_edit.get_deletability(message), false);
message.locally_echoed = false;
assert.equal(message_edit.get_deletability(message), false);
override(settings_data, "user_can_delete_own_message", () => true);
assert.equal(message_edit.get_deletability(message), true);
message.sent_by_me = false;
assert.equal(message_edit.get_deletability(message), false);
message.sent_by_me = true;
let now = new Date();
let current_timestamp = now / 1000;
message.timestamp = current_timestamp - 5;
realm.realm_message_content_delete_limit_seconds = 10;
assert.equal(message_edit.get_deletability(message), true);
message.timestamp = current_timestamp - 60;
assert.equal(message_edit.get_deletability(message), false);
message.sender_id = 2;
message.sent_by_me = false;
people.initialize_current_user(test_user.user_id);
realm.realm_message_content_delete_limit_seconds = null;
override(settings_data, "user_can_delete_own_message", () => true);
assert.equal(message_edit.get_deletability(message), true);
override(settings_data, "user_can_delete_own_message", () => false);
assert.equal(message_edit.get_deletability(message), false);
now = new Date();
current_timestamp = now / 1000;
realm.realm_message_content_delete_limit_seconds = 10;
message.timestamp = current_timestamp - 60;
override(settings_data, "user_can_delete_own_message", () => true);
assert.equal(message_edit.get_deletability(message), false);
});
run_test("stream_and_topic_exist_in_edit_history", () => {
// A message with no edit history should always return false;
// the message's current stream_id and topic are not compared
// to the stream_id and topic parameters.
const message_no_edits = {
stream_id: 1,
topic: "topic match",
};
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(message_no_edits, 2, "no match"),
false,
);
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(message_no_edits, 1, "topic match"),
false,
);
// A non-stream message (object has no stream_id or topic)
// with content edit history, should return false.
const private_message = {
edit_history: [{prev_content: "content edit to direct message"}],
};
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(private_message, 1, "topic match"),
false,
);
// A stream message with only content edits should return false,
// even if the message's current stream_id and topic are a match.
const message_content_edit = {
stream_id: 1,
topic: "topic match",
edit_history: [{prev_content: "content edit"}],
};
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(message_content_edit, 1, "topic match"),
false,
);
const message_stream_edit = {
stream_id: 6,
topic: "topic match",
edit_history: [{stream: 6, prev_stream: 1}],
};
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(message_stream_edit, 2, "topic match"),
false,
);
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(message_stream_edit, 1, "topic match"),
true,
);
const message_topic_edit = {
stream_id: 1,
topic: "final topic",
edit_history: [{topic: "final topic", prev_topic: "topic match"}],
};
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(message_topic_edit, 1, "no match"),
false,
);
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(message_topic_edit, 1, "topic match"),
true,
);
const message_many_edits = {
stream_id: 6,
topic: "final topic",
edit_history: [
{stream: 6, prev_stream: 5},
{prev_content: "content only edit"},
{topic: "final topic", prev_topic: "topic match"},
{stream: 5, prev_stream: 1},
],
};
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(message_many_edits, 1, "no match"),
false,
);
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(message_many_edits, 2, "topic match"),
false,
);
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(message_many_edits, 1, "topic match"),
true,
);
// When the topic and stream_id exist in the message's edit history
// individually, but not together in a historical state, it should return false.
const message_no_historical_match = {
stream_id: 6,
topic: "final topic",
edit_history: [
{stream: 6, prev_stream: 1}, // stream matches, topic does not
{stream: 1, prev_stream: 5}, // neither match
{topic: "final topic", prev_topic: "topic match"}, // topic matches, stream does not
],
};
assert.equal(
message_edit.stream_and_topic_exist_in_edit_history(
message_no_historical_match,
1,
"topic match",
),
false,
);
});