IV
я бы начал с изменения метода
Спасибо, не нужно. Это чистая синтетика, чтобы понять принцип
Size: a a a
2020 May 17
AS
Спасибо, не нужно. Это чистая синтетика, чтобы понять принцип
это бы существенно упростило тестирование
IV
это бы существенно упростило тестирование
Тогда можно попробовать. Пойдем от простого к сложному. Каким образом?
AS
Тогда можно попробовать. Пойдем от простого к сложному. Каким образом?
для начала этот метод подменяет геттер. это никуда не годится.
далее. метод меняет поток с компаниями.
надо сделать чтобы companies был результирующим потоком из входных компаний, и потока selected
Что-то вроде selectedCompanies = combineLatest(companies, selected).pipe(combineFunction)
далее. метод меняет поток с компаниями.
надо сделать чтобы companies был результирующим потоком из входных компаний, и потока selected
Что-то вроде selectedCompanies = combineLatest(companies, selected).pipe(combineFunction)
AS
и был бы метод принимающий на вход два потока, чистая функция, которую легко протестировать
AS
что касается тестирования потоков, то не обязательно marbles, зачастую просто достаточно сформировать входные потоки, подписаться на результат и проверить что получили.
Тем более что всякие комбинации еще и синхронно сработают.
Тем более что всякие комбинации еще и синхронно сработают.
IV
для начала этот метод подменяет геттер. это никуда не годится.
далее. метод меняет поток с компаниями.
надо сделать чтобы companies был результирующим потоком из входных компаний, и потока selected
Что-то вроде selectedCompanies = combineLatest(companies, selected).pipe(combineFunction)
далее. метод меняет поток с компаниями.
надо сделать чтобы companies был результирующим потоком из входных компаний, и потока selected
Что-то вроде selectedCompanies = combineLatest(companies, selected).pipe(combineFunction)
Не очень понял про подмену геттера. Каким образом он геттер подменяет?
AS
Не очень понял про подмену геттера. Каким образом он геттер подменяет?
у вас companies это приватный геттер, присмотритесь
IV
там есть и геттер и сеттер
AS
а тьфу. глючу уже под вечер :)
AS
там есть и геттер и сеттер
в общем это тоже надо выкинуть :)
IV
для начала этот метод подменяет геттер. это никуда не годится.
далее. метод меняет поток с компаниями.
надо сделать чтобы companies был результирующим потоком из входных компаний, и потока selected
Что-то вроде selectedCompanies = combineLatest(companies, selected).pipe(combineFunction)
далее. метод меняет поток с компаниями.
надо сделать чтобы companies был результирующим потоком из входных компаний, и потока selected
Что-то вроде selectedCompanies = combineLatest(companies, selected).pipe(combineFunction)
И я не очень понял про companies. Я подписываюсь на selected$ и unselected$. Будет удобнее, если я сброшу ссылку на stackblitz?
AS
IV
Не очень понятно что это. Открыл ссылку, работает
AS
Не очень понятно что это. Открыл ссылку, работает
так было, но сейчас заработало
IV
так было, но сейчас заработало
Только URL не тот
IV
так было, но сейчас заработало