feat: abstract DB Adapter

[AVVS]

No description provided.

[codecov-io]

Current coverage is 88.45% (diff: 90.21%)

Merging #112 into master will decrease coverage by 0.65%

@@             master       #112   diff @@
==========================================
  Files            36         38     +2   
  Lines           744        823    +79   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            663        728    +65   
- Misses           81         95    +14   
  Partials          0          0          

Powered by Codecov. Last update 6b0a659...98b5770

[AVVS]

/home/runner/ms-users/src/actions/register.js
  12:23  error  Unable to resolve path to module '../db/adapter'  import/no-unresolved

/home/runner/ms-users/src/model/storages/redisstorage.js
  42:25  error  'captchaConfig' is defined but never used           no-unused-vars
  43:9   error  'deleteInactiveAccounts' is defined but never used  no-unused-vars

---

ну да, всему виной actions/register, а он пока на операционном столе

[AVVS]

@stainwoortsel относительно ошибок ремаппинг мы будем делать не каждый раз после каждого метода, а 1 раз во время ответа, есть возможность добавить функцию onComplete(err, data, actionName, actions), где err - ошибка, data - когда собственно ее нет, actionName - название события, actions - чаще всего undefined, а когда в AMQP QOS включен, то там ack/reject/retry

https://github.com/makeomatic/mservice/blob/master/src/plugins/amqp.js#L84-L94

[AVVS]

можно сократить и просто писать

exports.User = ...
exports.Attempts = ...

[ghost]

То есть Attempts и Tokens в классы не оборачивать?
Можно, но этж типа абстракция ) мало ли чего мы захотим вписать в методы по дефолту
Плюс оно обеспечивает требование по наличию в "стороже" нужных методов на этапе (типа) компиляции.
...ну не компиляции, но ошибка сразу найдется -- я к чему ) конечно, тесты -- дело святое, но так на ошибку наткнешься сразу и носом, еще на уровне линтера

[AVVS]

да не, я имею ввиду не нужно писать module.exports.SMTH =
пиши просто exports.SMTH =

-module.exports.User = new UserModel(storage.User);
+exports.User = new UserModel(storage.User);

[ghost]

onComplete -- это, я так понимаю, в настройках defaults.js, в разделе amqp?

[AVVS]

Все верно. Либо можем добавить хуки специальные в mservice по аналогии с hapi.js - там неплохо сделано, но это уже позже

[ghost]

да не, я имею ввиду не нужно писать module.exports.SMTH =
пиши просто exports.SMTH =

а... ок )

[ghost]

Так.. вижу branch conflict, но ветки я пока не мёржу.
Расскажу в общих чертах об изменениях:

1. да, класс с регистрацией сделал
2. перенес гугл-капчу в отдельную утилиту, тот код капчи что зависит от редиски -- в адаптере, в Utils. Гугл передается методу как функция-next) Utils.checkCaptcha.call(this, username, captcha, verifyGoogleCaptcha)`
3. Помимо register были правки в send-email, он оставался единственным где был redis, не уверен что сделал его чисто

[AVVS]

ну да, так тесты и пройдут)