bterkuileandClaude Sonnet 5
90ab7c74b1
Finishes the two gaps that mattered most for real use: Invoices#create
(POST invoices) + Invoices#quote (POST invoices/{id}/quote, returning
the new Quote model with the actual payable bolt11 — an invoice alone
has nothing payable until quoted) and Webhook.valid_signature?
(HMAC-SHA256 over the raw body via X-Webhook-Signature, per Strike's own
documented reference implementation, timing-safe compare). Also finishes
Invoices#find (was a stub with a leftover binding.pry) and adds
WebhookEvent to parse the {id, eventType, data: {entityId, changes}}
payload shape — deliberately doesn't expose changed values, since
Strike's webhooks never carry them; a real value only ever comes from
re-fetching the resource.
Client gains #post/#patch (only #get existed before), needed for both
of the above.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0199wudpuAtkhouHKSEGxzpa
38 lines
1.1 KiB
Ruby
38 lines
1.1 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
RSpec.describe StrikeMe::API::WebhookEvent do
|
|
subject(:event) do
|
|
described_class.new(
|
|
'id' => '245e40d8-f197-411c-8f20-a326d08da402',
|
|
'eventType' => 'invoice.updated',
|
|
'webhookVersion' => 'v1',
|
|
'data' => { 'entityId' => '602cc206-6b4a-42dd-bc89-9a534bb04034', 'changes' => ['state'] },
|
|
'created' => '2026-09-15T12:00:00+00:00',
|
|
'deliverySuccess' => true
|
|
)
|
|
end
|
|
|
|
it 'exposes entity_id and changes from the nested data object' do
|
|
expect(event.entity_id).to eq '602cc206-6b4a-42dd-bc89-9a534bb04034'
|
|
expect(event.changes).to eq ['state']
|
|
end
|
|
|
|
it 'aliases the camelCase attributes to snake_case' do
|
|
expect(event.event_type).to eq 'invoice.updated'
|
|
expect(event.webhook_version).to eq 'v1'
|
|
expect(event.delivery_success).to eq true
|
|
end
|
|
|
|
it 'parses created as a real Time' do
|
|
expect(event.created).to be_a Time
|
|
end
|
|
|
|
it 'identifies invoice events' do
|
|
expect(event).to be_invoice_event
|
|
end
|
|
|
|
it 'does not carry the actual changed values — only that state changed, not to what' do
|
|
expect(event.attributes.keys).not_to include('amount', 'state')
|
|
end
|
|
end
|