Skip to content

Commit

Permalink
Merge pull request #17 from cookpad/nekketsuuu-fix-body-hash
Browse files Browse the repository at this point in the history
fix spec: Use a hash for a positoinal argument rather than keywords
  • Loading branch information
nekketsuuu authored Jan 18, 2024
2 parents d109985 + af23823 commit bf54842
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 10 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@ jobs:
- '2.6'
- '2.7'
- '3.0'
- '3.1'
- '3.2'
- '3.3'
name: Run test with Ruby ${{ matrix.ruby }}
steps:
- uses: actions/checkout@v2
Expand Down
24 changes: 14 additions & 10 deletions spec/barbeque/client_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,13 @@
it 'enqueues a job to barbeque' do
expect(garage_client).to receive(:post).with(
'/v2/job_executions',
application: application,
job: job,
message: message,
queue: default_queue,
delay_seconds: nil,
{
application: application,
job: job,
message: message,
queue: default_queue,
delay_seconds: nil,
},
).and_return(result)
client.create_execution(job: job, message: message)
end
Expand All @@ -34,11 +36,13 @@
it 'enqueues with specified parameters' do
expect(garage_client).to receive(:post).with(
'/v2/job_executions',
application: application,
job: job,
message: message,
queue: queue,
delay_seconds: nil,
{
application: application,
job: job,
message: message,
queue: queue,
delay_seconds: nil,
},
).and_return(result)
client.create_execution(job: job, message: message, queue: queue)
end
Expand Down

0 comments on commit bf54842

Please sign in to comment.