Created
April 19, 2018 18:25
-
-
Save JONBRWN/8c167a5e4bbf6b219890a4f31b1ef41b to your computer and use it in GitHub Desktop.
Updated LatencyStatsCalculator Test
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
diff --git a/monitoring_hub/apps/metrics_reporter/test/metrics_reporter/latency_stats_calculator_test.exs b/monitoring_hub/apps/metrics_reporter/test/metrics_reporter/latency_stats_calculator_test.exs | |
index d0baa7c..196f4bf 100644 | |
--- a/monitoring_hub/apps/metrics_reporter/test/metrics_reporter/latency_stats_calculator_test.exs | |
+++ b/monitoring_hub/apps/metrics_reporter/test/metrics_reporter/latency_stats_calculator_test.exs | |
@@ -13,7 +13,7 @@ defmodule MetricsReporter.LatencyStatsCalculatorTest do | |
"pipeline_key" => "test", | |
"latency_bins" => %{"1" => 50, "10" => 100} | |
} | |
- expected_latency_bins = get_all_bins() | |
+ expected_latency_bins = ["0", "10", "19", "20", "21", "22", "23", "24", "25", "26", "27", "30", "64"] | |
latency_bins_list = [latency_bins_msg_1, latency_bins_msg_2] | |
empty_stats = %{ | |
"50.0" => "0", | |
@@ -76,10 +76,8 @@ defmodule MetricsReporter.LatencyStatsCalculatorTest do | |
latency_bins_data = LatencyStatsCalculator.calculate_latency_percentage_bins_data(latency_bins_list, expected_latency_bins) | |
cumulative_latency_percentage_bins_data = LatencyStatsCalculator.calculate_cumulative_latency_percentage_bins_data(latency_bins_data) | |
latency_percentile_bin_stats = LatencyStatsCalculator.calculate_latency_percentile_bin_stats(cumulative_latency_percentage_bins_data) | |
- Enum.each latency_percentile_bin_stats, fn {percentile, bin} -> | |
+ Enum.each latency_percentile_bin_stats, fn {_percentile, bin} -> | |
cond do | |
- percentile == "50.0" -> | |
- assert bin == "2" | |
true -> | |
assert bin == "1024" | |
end |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment