fullstack-development / react-redux-starter-kit

Modular starter kit for React+Redux+React Router projects.
https://demo.fullstack-development.com/
MIT License
91 stars 13 forks source link

demo app tests #2 #86

Closed chmnkh closed 5 years ago

chmnkh commented 5 years ago

тут тесты хэлперов

in19farkt commented 5 years ago

@chmnkh посмотри этот коммит https://github.com/fullstack-development/react-redux-starter-kit/pull/86/commits/36223587af5a1df7cafc309aa6b4930ed9f04b2c, немного изменил makeFeatureEntry. Я хз зачем там раньше было так сложно))

И еще в этой ветке сейчас тесты на PaginationControls крашатся (это не я, так и было :) ).

chmnkh commented 5 years ago

то что пагинация крашится это норм, последний закрытый пр это фиксит, в мастере все ок) а нафиг ты тест для мейкфичаентри удалил? :<

in19farkt commented 5 years ago

то что пагинация крашится это норм, последний закрытый пр это фиксит, в мастере все ок)

Ну дык надо мастера мержнуть сюда, негоже ветку с корявыми тестами мержить))

а нафиг ты тест для мейкфичаентри удалил? :<

А там теперь тестить нечего) Оно теперь работает как rule для jss

chmnkh commented 5 years ago

Ну дык надо мастера мержнуть сюда, негоже ветку с корявыми тестами мержить))

вмержил

А там теперь тестить нечего) Оно теперь работает как rule для jss

ну по крайней мере с тестом (как и в случае с rule) будет явно видно, что только для типизации написано и так задумано)

in19farkt commented 5 years ago

ну по крайней мере с тестом (как и в случае с rule) будет явно видно, что только для типизации написано и так задумано)

Ну да, тоже аргумент. Допиши, плиз, если ты сейчас в этой ветке)

chmnkh commented 5 years ago

да оно запушено уже))

я в дискашене даже отписал на эту тему)

upd: на самом деле там надо toEqual а не toBe, еще раз запушил