Fix capturing new metric names (#191)

Author: NelsonVides

src/collectors/vm/prometheus_vm_system_info_collector.erl

@@ -39,7 +39,7 @@ Collects Erlang VM metrics using `erlang:system_info/1`.
 * `erlang_vm_port_limit`
   Type: gauge.
   The maximum number of simultaneously existing ports at the local node.
-* `erlang_vm_process_count`
+* `erlang_vm_processes`
   Type: gauge.
   The number of processes currently existing at the local node.
 * `erlang_vm_process_limit`
@@ -90,7 +90,7 @@ Options are the same as Item parameter values for `erlang:system_info/1`:
 * `logical_processors_online` for `erlang_vm_logical_processors_online`.
 * `ports` for `erlang_vm_ports`.
 * `port_limit` for `erlang_vm_port_limit`.
-* `process_count` for `erlang_vm_process_count`.
+* `processes` for `erlang_vm_processes`.
 * `process_limit` for `erlang_vm_process_limit`.
 * `schedulers` for `erlang_vm_schedulers`.
 * `schedulers_online` for `erlang_vm_schedulers_online`.
@@ -204,6 +204,12 @@ collect_metrics(allocators) ->
     collect_allocator_metrics();
 collect_metrics(wordsize_bytes) ->
     erlang:system_info(wordsize);
+collect_metrics(atoms) ->
+    erlang:system_info(atom_count);
+collect_metrics(ports) ->
+    erlang:system_info(port_count);
+collect_metrics(processes) ->
+    erlang:system_info(process_count);
 collect_metrics(Name) ->
     try
         case erlang:system_info(Name) of

test/eunit/collectors/vm/prometheus_vm_system_info_collector_tests.erl

@@ -14,26 +14,26 @@ test_default_metrics(_) ->
     prometheus_registry:register_collector(prometheus_vm_system_info_collector),
     Metrics = prometheus_text_format:format(),
     [
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_dirty_cpu_schedulers")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_dirty_cpu_schedulers_online")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_dirty_io_schedulers")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_ets_limit")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_logical_processors")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_logical_processors_available")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_logical_processors_online")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_ports")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_port_limit")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_processes")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_process_limit")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_schedulers")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_schedulers_online")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_smp_support")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_threads")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_thread_pool_size")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_time_correction")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_wordsize_bytes")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_atoms")),
-        ?_assertMatch({match, _}, re:run(Metrics, "erlang_vm_atom_limit"))
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_dirty_cpu_schedulers")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_dirty_cpu_schedulers_online")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_dirty_io_schedulers")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_ets_limit")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_logical_processors")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_logical_processors_available")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_logical_processors_online")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_ports")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_port_limit")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_processes")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_process_limit")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_schedulers")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_schedulers_online")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_smp_support")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_threads")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_thread_pool_size")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_time_correction")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_wordsize_bytes")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_atoms")),
+        ?_assertMatch({match, _}, re:run(Metrics, "\nerlang_vm_atom_limit"))
     ].
 
 test_all_metrics(_) ->