fix: remove duplicated code

apps/emqx_authn/src/emqx_authn_schema.erl

@@ -115,23 +115,10 @@ fields("metrics_status_fields") ->
     ];
 fields("metrics") ->
     [
-        {"matched", mk(integer(), #{desc => ?DESC("matched")})},
+        {"ignore", mk(integer(), #{desc => ?DESC("failed")})}
-        {"success", mk(integer(), #{desc => ?DESC("success")})},
+    ] ++ common_field();
-        {"failed", mk(integer(), #{desc => ?DESC("failed")})},
-        {"ignore", mk(integer(), #{desc => ?DESC("failed")})},
-        {"rate", mk(float(), #{desc => ?DESC("rate")})},
-        {"rate_max", mk(float(), #{desc => ?DESC("rate_max")})},
-        {"rate_last5m", mk(float(), #{desc => ?DESC("rate_last5m")})}
-    ];
 fields("resource_metrics") ->
-    [
+    common_field();
-        {"matched", mk(integer(), #{desc => ?DESC("matched")})},
-        {"success", mk(integer(), #{desc => ?DESC("success")})},
-        {"failed", mk(integer(), #{desc => ?DESC("failed")})},
-        {"rate", mk(float(), #{desc => ?DESC("rate")})},
-        {"rate_max", mk(float(), #{desc => ?DESC("rate_max")})},
-        {"rate_last5m", mk(float(), #{desc => ?DESC("rate_last5m")})}
-    ];
 fields("node_metrics") ->
     [
         node_name(),
@@ -153,6 +140,16 @@ fields("node_error") ->
         {"error", mk(string(), #{desc => ?DESC("node_error")})}
     ].
 
+common_field() ->
+    [
+        {"matched", mk(integer(), #{desc => ?DESC("matched")})},
+        {"success", mk(integer(), #{desc => ?DESC("success")})},
+        {"failed", mk(integer(), #{desc => ?DESC("failed")})},
+        {"rate", mk(float(), #{desc => ?DESC("rate")})},
+        {"rate_max", mk(float(), #{desc => ?DESC("rate_max")})},
+        {"rate_last5m", mk(float(), #{desc => ?DESC("rate_last5m")})}
+    ].
+
 status() ->
     hoconsc:enum([connected, disconnected, connecting]).
 

apps/emqx_prometheus/test/emqx_prometheus_SUITE.erl

@@ -22,14 +22,14 @@
 -compile(export_all).
 
 -define(CLUSTER_RPC_SHARD, emqx_cluster_rpc_shard).
--define(CONF_DEFAULT,
+-define(CONF_DEFAULT, <<
-    <<"\n"
+    "\n"
     "prometheus {\n"
     "  push_gateway_server = \"http://127.0.0.1:9091\"\n"
     "  interval = \"1s\"\n"
     "  enable = true\n"
-    "}\n">>
+    "}\n"
-).
+>>).
 
 %%--------------------------------------------------------------------
 %% Setups