Dana OkaforBackend engineer · Завдання: виправити подвійне списання · Claude Code · 2 год 40 хв
Перевірено Lemonsignal 89%Сильна відповідність
- Знайшла причину, перш ніж дозволити ШІ писати виправлення.
- Протестувала виправлення від ШІ, а не повірила йому на слово.
- Виправила другий сервіс тим, що створила для першого.
- Розуміння Сильно Попросила відтворити подвійне списання й простежила один платіж у коді до будь-якого виправлення. Репліка 2
- Постановка завдання Упевнено Дала логи й потрібні файли та сказала, чого не можна змінювати. Репліка 4
- Перевірка Сильно Запустила тести на першому виправленні від ШІ й відхилила його. Власне виправлення довела тестом. Репліки 12, 19
- Виправлення Сильно Пояснила, чому перше виправлення хибне і де шукати натомість. Репліка 13
- Приймання Упевнено Невелика зміна з тестом і примітка про те, чого вона не тестувала.
- Повторюваність Сильно Зберегла кроки як скіл і виправила ним сервіс повернень.
Навички для цієї роботи
- Пошук першопричиниВище планки
- Доведення виправлення тестомВище планки
- Код, який зможуть підтримувати іншіНа рівні планки
Що вона здала, у тому вигляді, в якому здала. Кожна позначка відповідає навичці з її результату.
Примітка до зміни Одне списання на замовлення, навіть коли шлюз повторює запит
- Причина: після тайм-ауту шлюзу повторний запит створював друге списання.
- Виправлення: кожне списання має ключ свого замовлення, тому повторний запит повертає перше списання, а не створює нове.
- Тест: повторний запит після тайм-ауту тепер списує один раз. Усі тести проходять (48).
- Не тестувалось: часткові списання, яких ми поки не використовуємо.
payments/charge.pyВиправлення
def create_charge(order, card):
- return gateway.charge(order.total, card)
+ key = f"order-{order.id}"
+ return gateway.charge(order.total, card,
+ idempotency_key=key)
tests/test_charge.pyПадає до виправлення, проходить після
+def test_retry_charges_once(gw, order, card):
+ gw.fail_next_with(Timeout)
+ create_charge(order, card)
+ create_charge(order, card) # the retry
+ assert gw.charges_for(order) == 1
refunds/refund.pyТаке саме виправлення в сервісі повернень
def create_refund(charge, amount):
- return gateway.refund(charge.id, amount)
+ key = f"refund-{charge.id}"
+ return gateway.refund(charge.id, amount,
+ idempotency_key=key)