feat: add faker to variable #169
Conversation
|
@Cdayz займусь этим |
|
@Cdayz концепт готов. |
|
|
||
| f := faker.NewFaker() | ||
|
|
||
| return f.Faker(key) |
There was a problem hiding this comment.
Мне не нравится, что вот тут может быть паника https://github.com/neotoolkit/faker/blob/main/faker.go#L348
По хорошему лучше чтобы была ошибка и ее можно было нормально отобразить в логе, а не сломать весь процесс выполнения тестов.
There was a problem hiding this comment.
И история с псевдо-callable обьектами внутри шаблона наверное так не должна обрабатываться.
Это выглядит как супер неочевидный код.
У нас внутри есть задача на замену неполного шаблонизатора в gonkey на полноценный text/template, что по идее должно будет решать подобные проблемы проще.
@vitkarpenko FYI
There was a problem hiding this comment.
Заменю панику на обработку ошибки
There was a problem hiding this comment.
Предлагаю тогда скооперироваться. Я дополню библиотеку нужными данными и сценариями. И когда перейдете на text/template - внедрим все. Сейчас некоторые вещи конечно не очень очевидные, поэтому для концепта пришлось сделать подобное. В шаблонах конечно было бы проще, просто передать фейкер как функцию в шаблон
|
@sashamelentyev а почему использовать именно вашу библиоткеку? Чем https://github.com/brianvoe/gofakeit плоха? |
|
@leorush функционал тот же. Здесь решать уже вам что использовать |
Ребят, привет!
Было бы здорово иметь функционал генерации случайных данных для тестов. Предлагаю внедрить faker.
В переменных описываем faker и какие данные нам нужны, и на каждый запуск - будут прилетать случайные данные.
Концепт в PR