instrumentisto / medea

[Closed] Medea media server
Mozilla Public License 2.0
23 stars 3 forks source link

Refactor Jason error propagation #209

Closed alexlapa closed 3 years ago

alexlapa commented 3 years ago

Required for #198

Synopsis

Решили отказаться от абстрактного JasonError, который стриает тип возвращаемой ошибки, и смотреть в сторону создания кастомных эксепшенов под каждый случай.

В данный момент такая реализация осложнена тем, как мы прокидываем ошибки внутри Jason'а: в каждом модуле есть свой enum ModuleNameError, в котором лежат все возомжные варинаты ошибок из этого модуля и все функции возвращают Result<T, ModuleNameError>.

Solution

Максимально возможный (без обьемных переписываний) отказ от текущего подхода. Если фактический список ошибок методов A и B отличаются друг от друга, то они должны иметь разные Err.

Checklist

alexlapa commented 3 years ago
  1. Тут ченжлог не трогаю, когда запилю новые ошибки для wasm платформы в ченжлоге все укажу.
  2. CI / Flutter tests падает, с вот такой вот ошибкой: Error: Timeout waiting for emulator to boot., т.е. до самих тестов дело не доходит. Локально тесты проходятся, поэтому предлагаю мержить. С CI буду отдельно разбираться.
alexlapa commented 3 years ago

FCM:

Refactor Jason error propagation (#209)

Additionally:
- upgrade Firefox to 89.0.1 version for E2E tests