[Pkg-javascript-devel] Bug#997579: node-vue-style-loader: FTBFS: dh_auto_test: error: /bin/sh -ex debian/tests/pkg-js/test returned exit code 1

[Lucas Nussbaum]

Source: node-vue-style-loader
Version: 4.1.2-3
Severity: serious
Justification: FTBFS
Tags: bookworm sid ftbfs
User: lucas at debian.org
Usertags: ftbfs-20211023 ftbfs-bookworm

Hi,

During a rebuild of all packages in sid, your package failed to build
on amd64.


Relevant part (hopefully):
> dpkg-buildpackage
> -----------------
> 
> Command: dpkg-buildpackage -us -uc -sa -rfakeroot
> dpkg-buildpackage: info: source package node-vue-style-loader
> dpkg-buildpackage: info: source version 4.1.2-3
> dpkg-buildpackage: info: source distribution unstable
> dpkg-buildpackage: info: source changed by Jelmer Vernooij <jelmer at debian.org>
>  dpkg-source --before-build .
> dpkg-buildpackage: info: host architecture amd64
>  debian/rules clean
> dh clean
>    dh_auto_clean --buildsystem=nodejs
> 	rm -rf ./node_modules/.cache
>    dh_clean
>  dpkg-source -b .
> dpkg-source: info: using source format '3.0 (quilt)'
> dpkg-source: info: building node-vue-style-loader using existing ./node-vue-style-loader_4.1.2.orig.tar.gz
> dpkg-source: info: using patch list from debian/patches/series
> dpkg-source: info: building node-vue-style-loader in node-vue-style-loader_4.1.2-3.debian.tar.xz
> dpkg-source: info: building node-vue-style-loader in node-vue-style-loader_4.1.2-3.dsc
>  debian/rules binary
> dh binary
>    dh_update_autotools_config
>    dh_autoreconf
>    dh_auto_configure --buildsystem=nodejs
>    dh_auto_build --buildsystem=nodejs
> No build command found, searching known files
>    dh_auto_test --buildsystem=nodejs
> 	mkdir -p node_modules
> 	ln -s ../. node_modules/vue-style-loader
> 	/bin/sh -ex debian/tests/pkg-js/test
> + test  != 
> + jest --ci --testRegex test/test.js
> FAIL test/test.js
>   ✕ addStylesClient (dev) (2 ms)
>   ✕ addStylesClient (prod)
>   ✕ addStylesClient (dev + ssr)
>   ✕ addStylesClient (prod + ssr)
>   ✓ addStylesServer (dev) (2 ms)
>   ✓ addStylesServer (prod) (1 ms)
> 
>   ● addStylesClient (dev)
> 
>     The error below may be caused by using the wrong test environment, see https://jestjs.io/docs/configuration#testenvironment-string.
>     Consider using the "jsdom" test environment.
> 
>     ReferenceError: document is not defined
> 
>       118 | function addStyle (obj /* StyleObjectPart */) {
>       119 |   var update, remove
>     > 120 |   var styleElement = document.querySelector('style[' + ssrIdKey + '~="' + obj.id + '"]')
>           |                      ^
>       121 | 
>       122 |   if (styleElement) {
>       123 |     if (isProduction) {
> 
>       at addStyle (lib/addStylesClient.js:120:22)
>       at addStylesToDom (lib/addStylesClient.js:104:20)
>       at addStylesClient (lib/addStylesClient.js:58:3)
>       at Object.<anonymous> (test/test.js:12:18)
> 
>   ● addStylesClient (prod)
> 
>     The error below may be caused by using the wrong test environment, see https://jestjs.io/docs/configuration#testenvironment-string.
>     Consider using the "jsdom" test environment.
> 
>     ReferenceError: document is not defined
> 
>       118 | function addStyle (obj /* StyleObjectPart */) {
>       119 |   var update, remove
>     > 120 |   var styleElement = document.querySelector('style[' + ssrIdKey + '~="' + obj.id + '"]')
>           |                      ^
>       121 | 
>       122 |   if (styleElement) {
>       123 |     if (isProduction) {
> 
>       at addStyle (lib/addStylesClient.js:120:22)
>       at addStylesToDom (lib/addStylesClient.js:104:20)
>       at addStylesClient (lib/addStylesClient.js:58:3)
>       at Object.<anonymous> (test/test.js:22:18)
> 
>   ● addStylesClient (dev + ssr)
> 
>     The error below may be caused by using the wrong test environment, see https://jestjs.io/docs/configuration#testenvironment-string.
>     Consider using the "jsdom" test environment.
> 
>     ReferenceError: document is not defined
> 
>       77 | function mockSSRTags (list, parentId) {
>       78 |   list.forEach((item, i) => {
>     > 79 |     const style = document.createElement('style')
>          |                   ^
>       80 |     style.setAttribute('data-vue-ssr-id', `${parentId}:${i}`)
>       81 |     style.textContent = item[1]
>       82 |     if (item[2]) {
> 
>       at forEach (test/test.js:79:19)
>           at Array.forEach (<anonymous>)
>       at mockSSRTags (test/test.js:78:8)
>       at Object.<anonymous> (test/test.js:32:3)
> 
>   ● addStylesClient (prod + ssr)
> 
>     The error below may be caused by using the wrong test environment, see https://jestjs.io/docs/configuration#testenvironment-string.
>     Consider using the "jsdom" test environment.
> 
>     ReferenceError: document is not defined
> 
>       88 | 
>       89 | function mockProdSSRTags (list, parentId) {
>     > 90 |   const style = document.createElement('style')
>          |                 ^
>       91 |   style.setAttribute('data-vue-ssr-id', list.map((item, i) => `${parentId}:${i}`).join(' '))
>       92 |   style.textContent = list.map(item => item[1]).join('\n')
>       93 |   document.head.appendChild(style)
> 
>       at mockProdSSRTags (test/test.js:90:17)
>       at Object.<anonymous> (test/test.js:40:3)
> 
> Test Suites: 1 failed, 1 total
> Tests:       4 failed, 2 passed, 6 total
> Snapshots:   0 total
> Time:        0.82 s
> Ran all test suites.
> dh_auto_test: error: /bin/sh -ex debian/tests/pkg-js/test returned exit code 1


The full build log is available from:
http://qa-logs.debian.net/2021/10/23/node-vue-style-loader_4.1.2-3_unstable.log

A list of current common problems and possible solutions is available at
http://wiki.debian.org/qa.debian.org/FTBFS . You're welcome to contribute!

If you reassign this bug to another package, please marking it as 'affects'-ing
this package. See https://www.debian.org/Bugs/server-control#affects

If you fail to reproduce this, please provide a build log and diff it with mine
so that we can identify if something relevant changed in the meantime.