Skip to content

Commit

Permalink
Fixes for rubocops in metric capture
Browse files Browse the repository at this point in the history
  • Loading branch information
blomquisg committed Mar 20, 2017
1 parent 8b24751 commit 7bd3090
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion app/models/metric/ci_mixin/capture.rb
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ def queue_name_for_metrics_collection
ems.metrics_collector_queue_name
end

def split_capture_intervals(interval_name, start_time, end_time, threshold=1.day)
def split_capture_intervals(interval_name, start_time, end_time, threshold = 1.day)
raise _("Start time must be earlier than End time") if start_time > end_time
# Create an array of ordered pairs from start_time and end_time so that each ordered pair is contained
# within the threshold. Then, reverse it so the newest ordered pair is first:
Expand Down
2 changes: 1 addition & 1 deletion spec/models/metric/ci_mixin/capture_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ def test_perf_capture_queue(time_since_last_perf_capture, total_queue_items, ver
expect(q_end_time).to be_same_time_as interval_end_time
interval_end_time = interval_start_time
# if the collection threshold is ever parameterized, then this 1.day will have to change
interval_start_time = interval_start_time - 1.day
interval_start_time -= 1.day
end
end
end
Expand Down

0 comments on commit 7bd3090

Please sign in to comment.