Mirror
/
zulip
mirror of https://github.com/zulip/zulip.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

node_tests: Deduplicate test_sync_realm_settings in settings_org tests. 

There was a lot of duplicate code in test_sync_realm_settings where
each value in common_policy_values was being tested for different
policies (create_stream_policy and invite_to_stream_policy).

This commit deduplicates is by using a for-loop for testing all
common_policy values and extracting the code as a function to test
different policies.
This commit is contained in:
sahil839 2021-05-24 12:42:53 +05:30 committed by Tim Abbott
parent a271308ac3
commit 8846df6b53
1 changed files with 19 additions and 138 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

157
frontend_tests/node_tests/settings_org.js
View File

@ -446,149 +446,30 @@ function test_sync_realm_settings() {
settings_org.sync_realm_settings("invalid_settings_property"); settings_org.sync_realm_settings("invalid_settings_property");
} }
{ function test_common_policy(property_name) {
/* const property_elem = $(`#id_realm_${CSS.escape(property_name)}`);
Test that when create stream policy is set to "full members" that the dropdown
is set to the correct value.
*/
const property_elem = $("#id_realm_create_stream_policy");
property_elem.length = 1; property_elem.length = 1;
property_elem.attr("id", "id_realm_create_stream_policy"); property_elem.attr("id", `id_realm_${CSS.escape(property_name)}`);
page_params.realm_create_stream_policy = 3; /* Each policy is initialized to 'by_members' and then all the values are tested
in the following order - by_admins_only, by_moderators_only, by_full_members,
by_members. */
settings_org.sync_realm_settings("create_stream_policy"); page_params[`realm_${property_name}`] =
assert.equal( settings_config.common_policy_values.by_members.code;
$("#id_realm_create_stream_policy").val(), property_elem.val(settings_config.common_policy_values.by_members.code);
settings_config.common_policy_values.by_full_members.code,
); for (const policy_value of Array.from(
Object.values(settings_config.common_policy_values),
)) {
page_params[`realm_${property_name}`] = policy_value.code;
settings_org.sync_realm_settings(property_name);
assert.equal(property_elem.val(), policy_value.code);
}
} }
{ test_common_policy("create_stream_policy");
/* test_common_policy("invite_to_stream_policy");
Test that when create stream policy is set to "by members" that the dropdown
is set to the correct value.
*/
const property_elem = $("#id_realm_create_stream_policy");
property_elem.length = 1;
property_elem.attr("id", "id_realm_create_stream_policy");
page_params.realm_create_stream_policy = 1;
settings_org.sync_realm_settings("create_stream_policy");
assert.equal(
$("#id_realm_create_stream_policy").val(),
settings_config.common_policy_values.by_members.code,
);
}
{
/*
Test that when create stream policy is set to "by admins only" that the dropdown
is set to the correct value.
*/
const property_elem = $("#id_realm_create_stream_policy");
property_elem.length = 1;
property_elem.attr("id", "id_realm_create_stream_policy");
page_params.realm_create_stream_policy = 2;
settings_org.sync_realm_settings("create_stream_policy");
assert.equal(
$("#id_realm_create_stream_policy").val(),
settings_config.common_policy_values.by_admins_only.code,
);
}
{
/*
Test that when create stream policy is set to "by moderators only" that the dropdown
is set to the correct value.
*/
const property_elem = $("#id_realm_create_stream_policy");
property_elem.length = 1;
property_elem.attr("id", "id_realm_create_stream_policy");
page_params.realm_create_stream_policy = 4;
settings_org.sync_realm_settings("create_stream_policy");
assert.equal(
$("#id_realm_create_stream_policy").val(),
settings_config.common_policy_values.by_moderators_only.code,
);
}
{
/*
Test that when invite to stream policy is set to "full members" that the dropdown
is set to the correct value.
*/
const property_elem = $("#id_realm_invite_to_stream_policy");
property_elem.length = 1;
property_elem.attr("id", "id_realm_invite_to_stream_policy");
page_params.realm_invite_to_stream_policy = 3;
settings_org.sync_realm_settings("invite_to_stream_policy");
assert.equal(
$("#id_realm_invite_to_stream_policy").val(),
settings_config.common_policy_values.by_full_members.code,
);
}
{
/*
Test that when invite to stream policy is set to "by members" that the dropdown
is set to the correct value.
*/
const property_elem = $("#id_realm_invite_to_stream_policy");
property_elem.length = 1;
property_elem.attr("id", "id_realm_invite_to_stream_policy");
page_params.realm_invite_to_stream_policy = 1;
settings_org.sync_realm_settings("invite_to_stream_policy");
assert.equal(
$("#id_realm_invite_to_stream_policy").val(),
settings_config.common_policy_values.by_members.code,
);
}
{
/*
Test that when invite to stream policy is set to "by admins only" that the dropdown
is set to the correct value.
*/
const property_elem = $("#id_realm_invite_to_stream_policy");
property_elem.length = 1;
property_elem.attr("id", "id_realm_invite_to_stream_policy");
page_params.realm_invite_to_stream_policy = 2;
settings_org.sync_realm_settings("invite_to_stream_policy");
assert.equal(
$("#id_realm_invite_to_stream_policy").val(),
settings_config.common_policy_values.by_admins_only.code,
);
}
{
/*
Test that when invite to stream policy is set to "by moderators only" that the dropdown
is set to the correct value.
*/
const property_elem = $("#id_realm_invite_to_stream_policy");
property_elem.length = 1;
property_elem.attr("id", "id_realm_invite_to_stream_policy");
page_params.realm_invite_to_stream_policy = 4;
settings_org.sync_realm_settings("invite_to_stream_policy");
assert.equal(
$("#id_realm_invite_to_stream_policy").val(),
settings_config.common_policy_values.by_moderators_only.code,
);
}
{ {
/* Test message content edit limit minutes sync */ /* Test message content edit limit minutes sync */