[Pkg-javascript-devel] Bug#1009558: node-convert-source-map: FTBFS: ERROR: Coverage for statements (98.55%) does not meet global threshold (100%)

Lucas Nussbaum lucas at debian.org
Tue Apr 12 20:18:31 BST 2022


Source: node-convert-source-map
Version: 1.8.0+~1.5.2-1
Severity: serious
Justification: FTBFS
Tags: bookworm sid ftbfs
User: lucas at debian.org
Usertags: ftbfs-20220412 ftbfs-bookworm

Hi,

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


Relevant part (hopefully):
>  debian/rules binary
> dh binary
>    dh_update_autotools_config
>    dh_autoreconf
>    dh_auto_configure --buildsystem=nodejs
> Link node_modules/@types/convert-source-map -> ../../types-convert-source-map
>    dh_auto_build --buildsystem=nodejs
> No build command found, searching known files
> No build command found, searching known files
>    dh_auto_test --buildsystem=nodejs
> 	ln -s ../. node_modules/convert-source-map
> 	/bin/sh -ex debian/tests/pkg-js/test
> + tap -R spec test/comment-regex.js test/convert-source-map.js test/map-file-comment.js
> 
> (node:3380862) DeprecationWarning: deepEqual() is deprecated, use same() instead
> (Use `node --trace-deprecation ...` to show where the warning was created)
> test/comment-regex.js
>   comment regex old spec - @
> 
>     ✓ matches //@
> 
>     ✓ matches //@  with charset
> 
>     ✓ matches //@  with charset
> 
>     ✓ matches   //@
> 
>     ✓ matches   //@  with charset
> 
>     ✓ matches   //@  with charset
> 
>     ✓ matches   //@
> 
>     ✓ matches   //@  with charset
> 
>     ✓ matches   //@  with charset
> 
>     ✓ matches //@
> 
>     ✓ matches //@  with charset
> 
>     ✓ matches //@  with charset
> 
>     ✓ matches /*@
> 
>     ✓ matches /*@  with charset
> 
>     ✓ matches /*@  with charset
> 
>     ✓ matches   /*@
> 
>     ✓ matches   /*@  with charset
> 
>     ✓ matches   /*@  with charset
> 
>     ✓ matches   /*@
> 
>     ✓ matches   /*@  with charset
> 
>     ✓ matches   /*@  with charset
> 
>     ✓ matches /*@
> 
>     ✓ matches /*@  with charset
> 
>     ✓ matches /*@  with charset
> 
>     ✓ should not match  @// @
> 
>     ✓ should not match  @/* @
> 
>   comment regex new spec - #
> 
>     ✓ matches   //\#
> 
>     ✓ matches   //\#  with charset
> 
>     ✓ matches   //\#  with charset
> 
>     ✓ matches   //\#
> 
>     ✓ matches   //\#  with charset
> 
>     ✓ matches   //\#  with charset
> 
>     ✓ matches //\#
> 
>     ✓ matches //\#  with charset
> 
>     ✓ matches //\#  with charset
> 
>     ✓ matches /*\#
> 
>     ✓ matches /*\#  with charset
> 
>     ✓ matches /*\#  with charset
> 
>     ✓ matches   /*\#
> 
>     ✓ matches   /*\#  with charset
> 
>     ✓ matches   /*\#  with charset
> 
>     ✓ matches   /*\#
> 
>     ✓ matches   /*\#  with charset
> 
>     ✓ matches   /*\#  with charset
> 
>     ✓ matches /*\#
> 
>     ✓ matches /*\#  with charset
> 
>     ✓ matches /*\#  with charset
> 
>     ✓ should not match  \#// \#
> 
>     ✓ should not match  \#/* \#
> 
>   mapFileComment regex old spec - @
> 
> (node:3380861) DeprecationWarning: deepEqual() is deprecated, use same() instead
> (Use `node --trace-deprecation ...` to show where the warning was created)
>     ✓ matches //@  ::
> 
>     ✓ matches   //@  ::
> (node:3380861) DeprecationWarning: similar() is deprecated, use match() instead
> 
>     ✓ matches   //@  ::
> 
>     ✓ matches ///@  ::
> 
>     ✓ matches ;//@  ::
> 
>     ✓ matches return//@  ::
> 
>     ✓ does not match  @// @ ::
> 
>     ✓ does not match var sm = `//@  :: `
> 
>     ✓ does not match var sm = "//@  :: "
> 
>     ✓ does not match var sm = '//@  :: '
> 
>     ✓ does not match var sm = ' //@  :: '
> 
>   mapFileComment regex new spec - #
> 
>     ✓ matches //\#  ::
> 
>     ✓ matches   //\#  ::
> 
>     ✓ matches   //\#  ::
> 
>     ✓ matches ///\#  ::
> 
>     ✓ matches ;//\#  ::
> 
>     ✓ matches return//\#  ::
> 
>     ✓ does not match  \#// \# ::
> 
>     ✓ does not match var sm = `//\#  :: `
> 
>     ✓ does not match var sm = "//\#  :: "
> 
>     ✓ does not match var sm = '//\#  :: '
> 
>     ✓ does not match var sm = ' //\#  :: '
> 
>   mapFileComment regex /* */ old spec - @
> 
>     ✓ matches /*@  :: */
> 
>     ✓ matches   /*@  ::   */
> 
>     ✓ matches   /*@  ::    */
> 
>     ✓ matches leading string/*@  :: */
> 
>     ✓ matches /*@  ::    */
> 
>     ✓ does not match /*@  ::  */ */
> 
>     ✓ does not match /*@  ::  */ more text
> 
>   mapFileComment regex /* */ new spec - #
> 
>     ✓ matches /*\#  :: */
> 
>     ✓ matches   /*\#  ::   */
> 
>     ✓ matches   /*\#  ::    */
> 
>     ✓ matches leading string/*\#  :: */
> 
>     ✓ matches /*\#  ::    */
> 
>     ✓ does not match /*\#  ::  */ */
> 
>     ✓ does not match /*\#  ::  */ more text
> 
> test/convert-source-map.js
>   different formats
> 
>     ✓ comment -> comment
> 
>     ✓ comment -> base64
> 
>     ✓ comment -> json
> 
>     ✓ comment -> object
> 
>     ✓ base64 -> base64
> 
>     ✓ base64 -> comment
> 
>     ✓ base64 -> json
> 
>     ✓ base64 -> object
> 
>     ✓ json -> json
> 
>     ✓ json -> base64
> 
>     ✓ json -> comment
> 
>     ✓ json -> object
> 
>   to object returns a copy
> 
>     ✓ setting property on returned object does not affect original
> 
>   to multi-line map
> 
>     ✓ should match pattern provided
> 
>   to map file comment
> 
>     ✓ should be equal
> 
>     ✓ should be equal
> 
>   from source
> 
>     ✓ no comment returns null
> 
>     ✓ beginning of last line
> 
>     ✓ indented of last line
> 
>     ✓ indented on last non empty line
> 
>     ✓ in the middle of code
> 
>     ✓ finds last map in source
> 
>   from source with a large source
> 
>     ✓ no comment returns null
> 
>     ✓ beginning of last line
> 
>     ✓ indented of last line
> 
>     ✓ indented on last non empty line
> 
>     ✓ in the middle of code
> 
>     ✓ finds last map in source
> 
>   remove comments
> 
>     ✓ from last line
> 
>     ✓ from the middle of code
> 
>     ✓ multiple comments from the middle of code
> 
>   remove map file comments
> 
>     ✓ // style filemap comment
> 
>     ✓ /* */ style filemap comment
> 
>   pretty json
> 
>     ✓ pretty prints json when space is given
> 
>   adding properties
> 
>     ✓ includes added property
> 
>   adding properties, existing property
> 
>     ✓ the error message includes the property name
> 
>   setting properties
> 
>     ✓ includes new property and changes existing properties
> 
>   getting properties
> 
>     ✓ gets version
> 
>     ✓ gets sources
> 
>   return null fromSource when largeSource is true
> 
>     ✓ return value should be null
> 
>   commentRegex returns new RegExp on each get
> 
>     ✓ has an updated lastIndex
> 
>     ✓ a fresh RegExp has lastIndex of 0
> 
>   mapFileCommentRegex returns new RegExp on each get
> 
>     ✓ has an updated lastIndex
> 
>     ✓ a fresh RegExp has lastIndex of 0
> 
> test/map-file-comment.js
>   resolving a "/*# sourceMappingURL=map-file-comment.css.map*/" style comment inside a given css content
> 
>     ✓ resolves paths of original sources
> 
>     ✓ includes filename of generated file
> 
>     ✓ includes mappings
> 
>   resolving a "//# sourceMappingURL=map-file-comment.css.map" style comment inside a given css content
> 
>     ✓ resolves paths of original sources
> 
>     ✓ includes filename of generated file
> 
>     ✓ includes mappings
> 
>   resolving a /*# sourceMappingURL=data:application/json;base64,... */ style comment inside a given css content
> 
>     ✓ resolves paths of original sources
> 
>     ✓ includes filename of generated file
> 
>     ✓ includes mappings
> 
> 
>   138 passing (853.707ms)
> ERROR: Coverage for lines (98.43%) does not meet global threshold (100%)
> ERROR: Coverage for branches (90.62%) does not meet global threshold (100%)
> ERROR: Coverage for statements (98.55%) does not meet global threshold (100%)
> ----------|---------|----------|---------|---------|-------------------
> File      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
> ----------|---------|----------|---------|---------|-------------------
> All files |   98.55 |    90.62 |     100 |   98.43 |                   
>  index.js |   98.55 |    90.62 |     100 |   98.43 | 40                
> ----------|---------|----------|---------|---------|-------------------
> 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/2022/04/12/node-convert-source-map_1.8.0+~1.5.2-1_unstable.log

All bugs filed during this archive rebuild are listed at:
https://bugs.debian.org/cgi-bin/pkgreport.cgi?tag=ftbfs-20220412;users=lucas@debian.org
or:
https://udd.debian.org/bugs/?release=na&merged=ign&fnewerval=7&flastmodval=7&fusertag=only&fusertagtag=ftbfs-20220412&fusertaguser=lucas@debian.org&allbugs=1&cseverity=1&ctags=1&caffected=1#results

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.



More information about the Pkg-javascript-devel mailing list