Closed DudaGod closed 7 years ago
/cc @eGavr
Вроде 🆗
но опиши проблему в подробнее в описании, почему такое решение, почему сломалось и т д + еще надо парочку ок-ов заработать
cc @sipayRT @j0tunn
Постарался описать как можно подробнее.
ok описание просто 🔥
/ok
Описание ...
Проблема в следующем: При прогоне тестов выполненными помечаются только браузеры в стейтах. Сами же стейты и сюйты висят в состоянии pending.
После того как начал откатываться назад по коммитам, нашел в чем же проблема. При внедрении eslint в gemini-gui он мне активно подсказывал, что нужно исправить. Вот мы с ним вдвоем и исправили =)
Код до того, как мы с
моим корешомeslint-ом поработали:Код после:
Вроде бы мы все правильно сделали. Но оказывается, что browserId в определенных случаях равен
undefined
, а в другихnull
. А как все знаютnull == undefined
и они не равны ничему другому.А теперь подробнее про эти случаи:
browserId === undefined
:browserId === null
:browserId
считывается из атрибута указанного в текущей dom ноде. Данный атрибут указан только для тестов (по браузерам) внутри стейтов.При использовании исправленного кода:
логика не меняется и обрабатываются случаи как с
null
так и сundefined
.