allow to exclude uncles from average block time calculation

apps/explorer/config/config.exs

@@ -12,7 +12,9 @@ config :explorer,
   token_functions_reader_max_retries: 3,
   allowed_evm_versions:
     System.get_env("ALLOWED_EVM_VERSIONS") ||
-      "homestead,tangerineWhistle,spuriousDragon,byzantium,constantinople,petersburg"
+      "homestead,tangerineWhistle,spuriousDragon,byzantium,constantinople,petersburg",
+  include_uncles_in_average_block_time:
+    if(System.get_env("UNCLES_IN_AVERAGE_BLOCK_TIME") == "false", do: false, else: true)
 
 config :explorer, Explorer.Counters.AverageBlockTime, enabled: true
 

apps/explorer/lib/explorer/counters/average_block_time.ex

@@ -70,8 +70,17 @@ defmodule Explorer.Counters.AverageBlockTime do
         select: {block.number, block.timestamp}
       )
 
+    query =
+      if Application.get_env(:explorer, :include_uncles_in_average_block_time) do
+        timestamps_query
+      else
+        from(block in timestamps_query,
+          where: block.consensus == true
+        )
+      end
+
     timestamps =
-      timestamps_query
+      query
       |> Repo.all()
       |> Enum.sort_by(fn {_, timestamp} -> timestamp end, &>=/2)
       |> Enum.map(fn {number, timestamp} ->

apps/explorer/mix.exs

@@ -92,7 +92,6 @@ defmodule Explorer.Mixfile do
       {:math, "~> 0.3.0"},
       {:mock, "~> 0.3.0", only: [:test], runtime: false},
       {:mox, "~> 0.4", only: [:test]},
-      {:nimble_csv, "~> 0.6.0"},
       {:poison, "~> 3.1"},
       {:nimble_csv, "~> 0.6.0"},
       {:postgrex, ">= 0.0.0"},

apps/explorer/test/explorer/counters/average_block_time_test.exs

@@ -11,6 +11,8 @@ defmodule Explorer.Counters.AverageBlockTimeTest do
     start_supervised!(AverageBlockTime)
     Application.put_env(:explorer, AverageBlockTime, enabled: true)
 
+    Application.put_env(:explorer, :include_uncles_in_average_block_time, true)
+
     on_exit(fn ->
       Application.put_env(:explorer, AverageBlockTime, enabled: false)
     end)
@@ -43,6 +45,36 @@ defmodule Explorer.Counters.AverageBlockTimeTest do
       assert AverageBlockTime.average_block_time() == Timex.Duration.parse!("PT3S")
     end
 
+    test "excludes uncles if include_uncles_in_average_block_time is set to false" do
+      block_number = 99_999_999
+      Application.put_env(:explorer, :include_uncles_in_average_block_time, false)
+
+      first_timestamp = Timex.now()
+
+      insert(:block, number: block_number, consensus: true, timestamp: Timex.shift(first_timestamp, seconds: 3))
+      insert(:block, number: block_number, consensus: false, timestamp: Timex.shift(first_timestamp, seconds: 4))
+      insert(:block, number: block_number + 1, consensus: true, timestamp: Timex.shift(first_timestamp, seconds: 5))
+
+      AverageBlockTime.refresh()
+
+      assert AverageBlockTime.average_block_time() == Timex.Duration.parse!("PT2S")
+    end
+
+    test "excludes uncles if include_uncles_in_average_block_time is set to true" do
+      block_number = 99_999_999
+      Application.put_env(:explorer, :include_uncles_in_average_block_time, true)
+
+      first_timestamp = Timex.now()
+
+      insert(:block, number: block_number, consensus: true, timestamp: Timex.shift(first_timestamp, seconds: 3))
+      insert(:block, number: block_number, consensus: false, timestamp: Timex.shift(first_timestamp, seconds: 4))
+      insert(:block, number: block_number + 1, consensus: true, timestamp: Timex.shift(first_timestamp, seconds: 5))
+
+      AverageBlockTime.refresh()
+
+      assert AverageBlockTime.average_block_time() == Timex.Duration.parse!("PT1S")
+    end
+
     test "when there are no uncles sorts by block number" do
       block_number = 99_999_999