test: add test_named_replicaset_alerts_when_replica_disconnects

This test checks scenario when replica disconnects from replicaset
in named vshard config. In this situation two alerts should be thrown:
`UNREACHABLE_REPLICA` and `UNREACHABLE_REPLICASET`. It is necessary to
add this test in order to reduce untested places in the codebase. Also
a function `info_assert_alert` is removed from `router_2_2_test` since
there is a more appropriate function in `asserts` module.

Follow-up tarantool#493

NO_DOC=test

Author: mrForza

test/luatest_helpers/asserts.lua

@@ -46,4 +46,13 @@ function asserts:assert_server_no_alerts(server)
     end)
 end
 
+function asserts:info_assert_alert(alerts, alert_name)
+    for _, alert in pairs(alerts) do
+        if alert[1] == alert_name then
+            return alert
+        end
+    end
+    t.fail(('There is no %s in alerts').format(alert_name))
+end
+
 return asserts

test/router-luatest/router_2_2_test.lua

@@ -3,6 +3,7 @@ local vtest = require('test.luatest_helpers.vtest')
 local vutil = require('vshard.util')
 local vconsts = require('vshard.consts')
 local vserver = require('test.luatest_helpers.server')
+local asserts = require('test.luatest_helpers.asserts')
 
 local g = t.group('router')
 local cfg_template = {
@@ -1044,14 +1045,6 @@ g.test_failed_calls_affect_priority = function()
     vtest.router_cfg(g.router, global_cfg)
 end
 
-local function info_find_alert(alerts, alert_name)
-    for _, v in pairs(alerts) do
-        if v[1] == alert_name then
-            return v
-        end
-    end
-end
-
 --
 -- gh-474: error during alert construction
 --
@@ -1069,7 +1062,7 @@ g.test_info_with_named_identification = function()
         ilt.assert(ok, 'no error')
         return result.alerts
     end)
-    t.assert(info_find_alert(alerts, 'MISSING_MASTER'),
+    t.assert(asserts:info_assert_alert(alerts, 'MISSING_MASTER'),
              'MISSING_MASTER alert is constructed')
 
     --
@@ -1085,7 +1078,7 @@ g.test_info_with_named_identification = function()
         ilt.assert(ok, 'no error')
         return result.alerts
     end)
-    local alert = info_find_alert(alerts, 'UNREACHABLE_MASTER')
+    local alert = asserts:info_assert_alert(alerts, 'UNREACHABLE_MASTER')
     t.assert(alert, 'UNREACHABLE_MASTER alert is constructed')
     t.assert_not_str_contains(alert[2], 'replicaset nil',
                               'alert contains valid replicaset id')

test/storage-luatest/persistent_names_test.lua renamed to test/storage-luatest/persistent_names_1_1_test.lua

@@ -0,0 +1,47 @@
+local t = require('luatest')
+local vtest = require('test.luatest_helpers.vtest')
+local vutil = require('vshard.util')
+local asserts = require('test.luatest_helpers.asserts')
+
+local test_group = t.group('storage')
+
+local cfg_template = {
+    sharding = {
+        repliacset_1 = {
+            replicas = {
+                replica_1_a = {
+                    master = true
+                },
+                replica_1_b = {},
+            },
+        },
+    },
+    bucket_count = 20,
+    identification_mode = 'name_as_key'
+}
+
+local global_cfg
+
+test_group.before_all(function(g)
+    t.run_only_if(vutil.feature.persistent_names)
+    global_cfg = vtest.config_new(cfg_template)
+
+    vtest.cluster_new(g, global_cfg)
+    vtest.cluster_bootstrap(g, global_cfg)
+    vtest.cluster_wait_vclock_all(g)
+    vtest.cluster_rebalancer_disable(g)
+end)
+
+test_group.after_all(function(g)
+    g.cluster:drop()
+end)
+
+test_group.test_named_replicaset_alerts_when_replica_disconnects = function(g)
+    g.replica_1_b:stop()
+    local alerts = g.replica_1_a:exec(function()
+        return ivshard.storage.info().alerts
+    end)
+    asserts:info_assert_alert(alerts, 'UNREACHABLE_REPLICA')
+    asserts:info_assert_alert(alerts, 'UNREACHABLE_REPLICASET')
+    g.replica_1_b:start()
+end