Problem: overlap option is not set to jobs

Solution: add test case and add overlap option in normalize step Fixes quantum-elixir#71
c-rack · Jan 28, 2016 · 3be8a15 · 3be8a15
1 parent 510529f
commit 3be8a15
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/lib/quantum/normalizer.ex b/lib/quantum/normalizer.ex
@@ -110,6 +110,7 @@ defmodule Quantum.Normalizer do
       schedule: extract(:schedule, opts),
       task: extract(:task, opts),
       args: extract(:args, opts, []),
+      overlap: extract(:overlap, opts, Application.get_env(:quantum, :default_overlap, true)),
       nodes: :nodes |> extract(opts, [node()]) |> atomize
     }
   end

diff --git a/test/normalizer_test.exs b/test/normalizer_test.exs
@@ -16,6 +16,7 @@ defmodule Quantum.NormalizerTest do
       schedule: "@weekly",
       task: "MyModule.my_method",
       args: [1, 2, 3],
+      overlap: false,
       nodes: [:atom@node, "string@node"]
     ]}
 
@@ -24,6 +25,7 @@ defmodule Quantum.NormalizerTest do
       schedule: "@weekly",
       task: {"MyModule", "my_method"},
       args: [1, 2, 3],
+      overlap: false,
       nodes: [:atom@node, :string@node]
     }}
   end