Skip to content

chore(*): fix gemini tests with postcss-calc errors #202

Merged
merged 3 commits into from
Aug 7, 2017

Conversation

tx44
Copy link
Contributor

@tx44 tx44 commented Aug 7, 2017

Временный фикс проблем с gemini тестами и postcss-calc.

Мотивация и контекст

Проблемы из-за shim реализации Array.prototype.find во внутренних зависимостях (collections -> q-io -> gemini-coverage -> gemini):
montagejs/collections#178

Попробовал n способов. Запрет конфигурирования метода через Object.defineProperty() вызывает потом TypeError — не подходит. Переопределить после shim и до обработки стилей postcss представляется труднодоступным, нужно тогда тоже форкать, только сам gemini. Опускать версию postcss-calc в пресетах или здесь — заставит опускать версии других postcss модулей. Поэтому пока так. Если есть лучше идеи — предлагайте.

Заодно кстати поправились скриншоты.

@tx44 tx44 merged commit a862157 into master Aug 7, 2017
@tx44 tx44 deleted the chore/fix-gemini-tests branch August 7, 2017 15:29
@tx44
Copy link
Contributor Author

tx44 commented Nov 6, 2017

Поправка, ломка была в дублирующем модуле, установленном во внутренних node_modules у gemini-optipng:
collections ^0.2.2 -> q-io ^1.3.12 -> gemini-optipng ^1.0.0 -> gemini

Ломающий find shim:
https://github.com/montagejs/collections/blob/v0.2.2/shim-array.js#L125

collections, который ставится на первый уровень не патчит Array.prototype:
montagejs/collections#178 (comment)

collections >=2.0.1 <3.0.0 -> q-io ^2.0.6 -> gemini-coverage ^1.0.3 -> gemini

@tx44
Copy link
Contributor Author

tx44 commented Feb 12, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants