да это ж нпм, захотят - подвинут, если на уровне стандарта будет

всё будет плохо тогда

вот url parser синхронизировали между нодой и браузером уже хорошо

а уже выпустили?

это ж брейкинг ченж

в 8 будет видимо

это ж брейкинг ченж

Отдельным свойством.

Не бреакинг.

semver-minor

https://github.com/nodejs/node/pull/7448

require('url').URL теперь даёт такой же интерфейс, как URL в браузере

А раньше его не было

И да, это в 7.0.0 вошло.

а, да

забыл что это св-во

#вопрос Господа, подскажите как быть с такой неприятностью? // foo.js import Lib from 'Lib/browser' function foo () { //кодище } export default foo // foo.test.js import foo from './foo' describe( 'foo', () => { // тесты валятся с ошибкой «ReferenceError: window is not defined» } )

Надо как-то подменять библиотеку версией для NodeJS на время тестирования

Надо как-то подменять библиотеку версией для NodeJS на время тестирования

Твой код?

Если да, то переписать

Мой

Как именно?

Проверять окружение?

Прокидывать window аргументом

ну а лучше то, что от window надо

эээ куда? В библиотеку?

а если она от dom зависит, то наверное jsdom

эээ куда? В библиотеку?

Да, что у тебя из windows используется?

Да, что у тебя из windows используется?

У меня в коде ничего. У меня подключена библиотека Lib и у неё есть вариант сборки для браузера и вариант для ноды. Тестируемый модуль написан для работы в браузере и подключает браузерную версию, которая падает при попытке тестировать код нодой.

Что логично

Мне надо на время теста подменять зависимость моего модуля

У меня в коде ничего. У меня подключена библиотека Lib и у неё есть вариант сборки для браузера и вариант для ноды. Тестируемый модуль написан для работы в браузере и подключает браузерную версию, которая падает при попытке тестировать код нодой.

Подключай в ноду ножовую версию

А как?

Я не знаю

Ты же говоришь что есть две версии

Ну да

Мне надо при тесте подменять их

Каким-то образом

Require('lib/dist/node-version.js')

А для тестов в браузере реквайрить браузерную версию

Ты же говоришь что есть две версии

Есть две версии

Если совсем ничего не получится, то попробуй это https://github.com/jhnns/rewire/blob/master/README.md

На одной экспресс точёный На другой коа дроченый

У меня в коде ничего. У меня подключена библиотека Lib и у неё есть вариант сборки для браузера и вариант для ноды. Тестируемый модуль написан для работы в браузере и подключает браузерную версию, которая падает при попытке тестировать код нодой.

Если трестируемый модуль для браузера, то не тестируй его в ноде

Какой смысл?

Если трестируемый модуль для браузера, то не тестируй его в ноде

а где тестировать?

В браузере может быть?

эмм, плохо себе представляю это

Посмотри testem

Для начала

эмм, плохо себе представляю это

Посмотришь?

Посмотришь?

хм, прикольная штука

спасибо большое

phantomjs, selenium

Если трестируемый модуль для браузера, то не тестируй его в ноде

Лол

Лол

Мм?

как-то оно сложно вяжется с Mocha + Chai + Babel

уф

Babel не причем

Chai тоже

Проблема обычно только в тест-раннерах

Мм?

Просто прозвучало будто про f5 :)

Jsdom + мок зависимость не решит проблему?

ктонибудь дружил webpack с mocha, у меня в вебпаке много алиасов, хочу unit тесты написать

Jsdom + мок зависимость не решит проблему?

Да мне не нужен jsdom

это оверкилл какой-то

ктонибудь дружил webpack с mocha, у меня в вебпаке много алиасов, хочу unit тесты написать

Самое простое - откажись от алиасов и пиши обычные юнит тесты

Зачем тут вебпак?

ERROR: S client not available

это оверкилл какой-то

С ним быстрее ?

Да мне не нужен jsdom

А вообще тебе нужно google how to run tests in the browser

phantomjs, selenium

..

Если jsdom оверкил, тогда не знаю как назвать phantomjs)

А вообще тебе нужно google how to run tests in the browser

Ну пока ты не сказал, мне даже в голову не приходил этот подход

До сих пор все отлично внутри вебшторма тестил просто

Внутри вебшторма?

Ну через npm test

я, пожалуй, удалю свой недописанный коммент про ci/cd..

Ну пока ты не сказал, мне даже в голову не приходил этот подход

Ну у тебя strongly wired to specific windows environment tests

Какой смысл их тестить в ноде не пойму

До сих пор все отлично внутри вебшторма тестил просто

С юнит тестами все норм

тут надо понимать, что есть оверкилл по потраченному времени, есть по скорости прохождения/сборки, и тд

главное правильно приоритезировать оверкиллы

С юнит тестами все норм

Кстати щас понял что там не обязательно браузерную версию либы брать в зависимости. Да и вообще она не очень там нужна

Кстати щас понял что там не обязательно браузерную версию либы брать в зависимости. Да и вообще она не очень там нужна

Ура

Так-то это юнит-тест, там просто генерация id используется из библиотеки просто потому что она уже есть в проекте

Поясните за преимущества коа перед эксперссом?

В экспрессе нельзя сделать конвейерную обработку запросов потому что express отбрасывает значения middleware-функции и вот такое без патча экспресса сделать не получится //log requests middlewares.push(async(next, req, res) => { console.log('request start:', req.method, req.url); var response = await next(); console.log('request send:', response); res.end(JSON.stringify(response)); console.log('request end:', req.method, req.url); }); //catch errors middlewares.push(async(req, res, next) => { try { return await next(); } catch (e) { console.log('error:', e); res.status(400); return {error: e.message}; } }); middlerawe.push(staticServer) //connect to database middlewares.push(async(next, req, res) => { req.db = await new DB(schema); req.data = {}; try { return await next(); } catch (err) { await req.db.disconnect(); throw err; } }); middlerawe.push(cookieParser) middlewares.push(getCurrentUserOrCreateGuest) middlerawe.push(bodyParser) //map request info to crud operation middlewares.push(async(next, req, res) => { var [table, id] = req.url.split('/').slice(1); var dbActions = { "GET": 'get', "POST": 'add', "PUT": 'update', "DELETE": 'delete' }; req.data.action = dbActions[req.method]; req.data.table = table; req.data.id = id; return await next(); }); //process request in database middlewares.push(async(next, req, res) => { var {table, id, action} = req.data; if(!table) throw new Error('no table'); var index = req.query.index; var body = req.body; if (action == 'get') { return await req.db.get(table, id, {index}) } if (action == 'add') { return await req.db.add(table, body) } if (action == 'update') { return await req.db.update(table, id, body) } if (action == 'delete') { return await req.db.delete(table, id) } });

В экспрессе нельзя сделать конвейерную обработку запросов потому что express отбрасывает значения middleware-функции и вот такое без патча экспресса сделать не получится //log requests middlewares.push(async(next, req, res) => { console.log('request start:', req.method, req.url); var response = await next(); console.log('request send:', response); res.end(JSON.stringify(response)); console.log('request end:', req.method, req.url); }); //catch errors middlewares.push(async(req, res, next) => { try { return await next(); } catch (e) { console.log('error:', e); res.status(400); return {error: e.message}; } }); middlerawe.push(staticServer) //connect to database middlewares.push(async(next, req, res) => { req.db = await new DB(schema); req.data = {}; try { return await next(); } catch (err) { await req.db.disconnect(); throw err; } }); middlerawe.push(cookieParser) middlewares.push(getCurrentUserOrCreateGuest) middlerawe.push(bodyParser) //map request info to crud operation middlewares.push(async(next, req, res) => { var [table, id] = req.url.split('/').slice(1); var dbActions = { "GET": 'get', "POST": 'add', "PUT": 'update', "DELETE": 'delete' }; req.data.action = dbActions[req.method]; req.data.table = table; req.data.id = id; return await next(); }); //process request in database middlewares.push(async(next, req, res) => { var {table, id, action} = req.data; if(!table) throw new Error('no table'); var index = req.query.index; var body = req.body; if (action == 'get') { return await req.db.get(table, id, {index}) } if (action == 'add') { return await req.db.add(table, body) } if (action == 'update') { return await req.db.update(table, id, body) } if (action == 'delete') { return await req.db.delete(table, id) } });

Я если честно не понял тебя, но спасибо, буду иметь в виду

Я если честно не понял тебя, но спасибо, буду иметь в виду

когда в экспрессе добавляем миддлварку мы не можем получить значение следующей миддлварки после того как она выполнится. И соотвественно не сможем подписаться на промис который вернет миддлварка потому что express его отбросил app.use(function(req,res,next){ console.log('start'); var result = next(); //всегда будет undefined даже если миддлварка вернет какое-то значение; result.then(()=>{ console.log('end'); }) }) app.use(function(req,res,next){ return new Promise(res=>{ setTimeout(res, 1000); }) })

В мидлварках можно в req пихать что угодно

когда в экспрессе добавляем миддлварку мы не можем получить значение следующей миддлварки после того как она выполнится. И соотвественно не сможем подписаться на промис который вернет миддлварка потому что express его отбросил app.use(function(req,res,next){ console.log('start'); var result = next(); //всегда будет undefined даже если миддлварка вернет какое-то значение; result.then(()=>{ console.log('end'); }) }) app.use(function(req,res,next){ return new Promise(res=>{ setTimeout(res, 1000); }) })

Ага, теперь я понял почему меня работа с экспрессом всегда немножко раздражала

Можно хоть промисы, хоть значения через нее пропихивать, как через контекст.

Прикольно. А на сколько это сейчас стабильно?

req.prom = new Promise(res=>{ setTimeout(res, 1000); }) в следующей после next() просто делаешь req.prom.then(...)

Да, да, экспресс - это очень удобно

req это по сути контекст запроса.

req.prom = new Promise(res=>{ setTimeout(res, 1000); }) в следующей после next() просто делаешь req.prom.then(...)

ок, с чистыми промисами это сделать можно, а как ты теперь заюзаешь async-await ? middlewares.push(async(next, req, res) => { console.log('request start:', req.method, req.url); var response = await next(); console.log('request send:', response); res.end(JSON.stringify(response)); console.log('request end:', req.method, req.url); }); //catch errors middlewares.push(async(req, res, next) => { try { return await next(); } catch (e) { console.log('error:', e); res.status(400); return {error: e.message}; } });

А экспресс с асинками уже работает? Я просто год его уже не видел.

не работает, как раз по причине того что отбрасывает значение миддлварки и await next() выдаст ошибку потому что next() вернул undefined а не промис

но если нужно то можно заюзать мой патч - https://github.com/expressjs/express/pull/3138 там всего лишь нужно в нескольких строках добавить слово return

ок, с чистыми промисами это сделать можно, а как ты теперь заюзаешь async-await ? middlewares.push(async(next, req, res) => { console.log('request start:', req.method, req.url); var response = await next(); console.log('request send:', response); res.end(JSON.stringify(response)); console.log('request end:', req.method, req.url); }); //catch errors middlewares.push(async(req, res, next) => { try { return await next(); } catch (e) { console.log('error:', e); res.status(400); return {error: e.message}; } });

Async/await просто сахар над промисами