Remove SchedulerAgent#check! which name is confusing
Akinori MUSHA
3 fichiers modifiés avec 1 ajouts et 12 suppressions
+ 0
- 4
app/models/agents/scheduler_agent.rb
@@ -87,10 +87,6 @@ module Agents |
||
| 87 | 87 |
true |
| 88 | 88 |
end |
| 89 | 89 |
|
| 90 |
- def check! |
|
| 91 |
- control! |
|
| 92 |
- end |
|
| 93 |
- |
|
| 94 | 90 |
def validate_options |
| 95 | 91 |
if (spec = options['schedule']).present? |
| 96 | 92 |
begin |
+ 1
- 1
lib/huginn_scheduler.rb
@@ -61,7 +61,7 @@ class Rufus::Scheduler |
||
| 61 | 61 |
job.scheduler_agent_id = agent_id |
| 62 | 62 |
|
| 63 | 63 |
if scheduler_agent = job.scheduler_agent |
| 64 |
- scheduler_agent.check! |
|
| 64 |
+ scheduler_agent.control! |
|
| 65 | 65 |
else |
| 66 | 66 |
puts "Unscheduling SchedulerAgent##{job.scheduler_agent_id} (disabled or deleted)"
|
| 67 | 67 |
job.unschedule |
+ 0
- 7
spec/models/agents/scheduler_agent_spec.rb
@@ -84,11 +84,4 @@ describe Agents::SchedulerAgent do |
||
| 84 | 84 |
expect(agent.memory['scheduled_at']).to be_nil |
| 85 | 85 |
end |
| 86 | 86 |
end |
| 87 |
- |
|
| 88 |
- describe "check!" do |
|
| 89 |
- it "should control targets" do |
|
| 90 |
- stub(agent).control!.once { nil }
|
|
| 91 |
- agent.check! |
|
| 92 |
- end |
|
| 93 |
- end |
|
| 94 | 87 |
end |