liderako / eth_contracts

0 stars 1 forks source link

test-case exchange #6

Closed liderako closed 6 years ago

liderako commented 6 years ago

Main case

  1. Проверить Ввод и выввод денег. (Ethereum). Done Найдено событие
  2. Проверить Ввод и выввод токенов. Done Найдено событие
  3. Создать и Отменить заявку. Done All ok.
  4. Проверить что заявка не будет выполняться в трейде.
  5. Создать заявку и выполныть ее сразу с помощью трейд.
  6. Создать заявку и выполныть ее за несколько раз с помощью трейд. Done all ok
  7. Добиться того чтобы на счету feeAccount были деньги. Done all ok
  8. Вывести деньги со счета feeAccount. Done all ok
  9. Изменить админа. Done All ok
  10. Изменить feeAccount. Done All ok
  11. Изменить feeTake. Done All ok
  12. Включить окончание принятие заявок. Done all ok

    Error case

  13. Создать заявку в которой один токен меняется на себя самого только за меньше ценну. Проверить поведение.
  14. Создать заявку в которой эфир меняется на эфир за меньшую цену. Проверить поведение.
  15. Попробовать завести ноль токенов или эфира. Done All ok
  16. Попробовать вывести ноль токенов или эфира. Done all ok
  17. Попробовать вывести эфир с помощью функции withdrawToken. Done all ok
  18. Попробовать поднять цену feeTake. Done All ok
  19. Попробовать вывести чужие деньги. Done Как это вообще возможно?
  20. Вывести деньги когда у юзера их в бирже нету а в самой бирже есть. (Эфир и токены) Done all ok
liderako commented 6 years ago
  1. Если юзер заводит эфир и у него нет этих денег транзакция теперь просто висит в списке транзакций. И всегда 0 из 12 блоков.
  2. Завести ноль эфира нельзя.

    9 . Изменить Админку никто кроме Админа не может. Проверенно. m11/e6. Изменить feeTake можно только вниз. Вызвать функцию не может никто кроме Админа.

liderako commented 6 years ago

10 . Был протестирован функционал изменение feeAccount. Никто кроме Админа это сделать не может даже сам фееАккаунт. Админ успешно меняет сам фееАккаунт.

liderako commented 6 years ago

2 . Если не вызвать функцию approve в смарт-контракте токена который ты хочешь завести на биржу с параметрами ( адресс биржы, количество токенов которые ты хочешь завести на биржу) вызвать метод depositToken не получиться. Обнаруженна ошибка с вывводом токенов. Возможно это связанно с тем что теперь функция вызивает не transfer а transferFrom.

Ошибка с transferFrom исправлена

liderako commented 6 years ago

3 . Заявка создается нормально. Деньги перемещаются в поле balanceOrder. Мейкер их не может вывести. Мейкер не может выставить в заявке что он хочет отдать 4 эфира за что-то если у него нету этих 4 эфироф на бирже. При отмене заявки деньги перемещаются обратно к мейкеру. И он их может вывести.

liderako commented 6 years ago

Нужно пофиксить event Trade. AmountSell не коректно отображается

Done

liderako commented 6 years ago

Протестировал торговлю. Как бы деньги перечисляются все как надо. И комиссию получаем. И люди торгуют. Но тестирование одного кейса заняло минут 30-40. Чтобы все проверить. Очень неудобно смотреть что должно было получиться. С информацией которая надходит из событий и с просмотром балансов.

liderako commented 6 years ago

Протестировал торговлю между токенами. Все работает корректно. Деньги перечисляются feeAccount. Деньги перечисляются между адрессами.