[Python-apps-team] Bug#909996: vdirsyncer: FTBFS (TypeError: 'NoneType' object is not iterable)

[Santiago Vila]

Package: src:vdirsyncer
Version: 0.16.2-4
Severity: serious
Tags: ftbfs

Dear maintainer:

I tried to build this package in buster but it failed:

--------------------------------------------------------------------------------
[...]
 debian/rules build-indep
dh build-indep --with python3,sphinxdoc --buildsystem=pybuild
   dh_update_autotools_config -i -O--buildsystem=pybuild
   dh_autoreconf -i -O--buildsystem=pybuild
   dh_auto_configure -i -O--buildsystem=pybuild
I: pybuild base:217: python3.7 setup.py config 
running config
/usr/lib/python3/dist-packages/setuptools_scm/version.py:191: UserWarning: meta invoked without explicit configuration, will use defaults where required.
  "meta invoked without explicit configuration,"
I: pybuild base:217: python3.6 setup.py config 
running config
/usr/lib/python3/dist-packages/setuptools_scm/version.py:191: UserWarning: meta invoked without explicit configuration, will use defaults where required.
  "meta invoked without explicit configuration,"
   debian/rules override_dh_auto_build

[... snipped ...]

tests/storage/test_singlefile.py ....................................... [ 49%]
.............ss.ss                                                       [ 53%]
tests/storage/dav/test_caldav.py sssssssssssssssssssssssssssssssssssssss [ 62%]
sssssssssssssss                                                          [ 65%]
tests/storage/dav/test_carddav.py ssssssssssssssssssssssssss             [ 71%]
tests/storage/dav/test_main.py .                                         [ 71%]
tests/system/cli/test_config.py .........                                [ 73%]
tests/system/cli/test_discover.py .....                                  [ 74%]
tests/system/cli/test_fetchparams.py ..........                          [ 76%]
tests/system/cli/test_repair.py .....                                    [ 77%]
tests/system/cli/test_sync.py .................                          [ 81%]
tests/system/cli/test_utils.py ..                                        [ 82%]
tests/system/utils/test_main.py .sssF                                    [ 83%]
tests/unit/test_exceptions.py .                                          [ 83%]
tests/unit/test_metasync.py .........                                    [ 85%]
tests/unit/test_repair.py ..........                                     [ 87%]
tests/unit/test_sync.py ...............................                  [ 94%]
tests/unit/cli/test_config.py .                                          [ 94%]
tests/unit/cli/test_discover.py ......                                   [ 96%]
tests/unit/utils/test_vobject.py ..................                      [100%]

=================================== FAILURES ===================================
______________ TestFilesystemStorage.test_metadata_normalization _______________
tests/storage/__init__.py:301: in test_metadata_normalization
    st.none(),
E   hypothesis.errors.FailedHealthCheck: It looks like your strategy is filtering out a lot of data. Health check found 50 filtered examples but only 3 good ones. This will make your tests much slower, and also will probably distort the data generation quite a lot. You should adapt your strategy to filter less. This can also be caused by a low max_leaves parameter in recursive() calls
E   See https://hypothesis.readthedocs.io/en/latest/healthchecks.html for more information about this. If you want to disable just this health check, add HealthCheck.filter_too_much to the suppress_health_check settings for this test.
---------------------------------- Hypothesis ----------------------------------
You can add @seed(296577508032444947572819497154329054860) to this test or run pytest with --hypothesis-seed=296577508032444947572819497154329054860 to reproduce this failure.
________________ TestMemoryStorage.test_metadata_normalization _________________
tests/storage/__init__.py:301: in test_metadata_normalization
    st.none(),
E   hypothesis.errors.FailedHealthCheck: It looks like your strategy is filtering out a lot of data. Health check found 50 filtered examples but only 5 good ones. This will make your tests much slower, and also will probably distort the data generation quite a lot. You should adapt your strategy to filter less. This can also be caused by a low max_leaves parameter in recursive() calls
E   See https://hypothesis.readthedocs.io/en/latest/healthchecks.html for more information about this. If you want to disable just this health check, add HealthCheck.filter_too_much to the suppress_health_check settings for this test.
---------------------------------- Hypothesis ----------------------------------
You can add @seed(135387776198974078022358545371536932831) to this test or run pytest with --hypothesis-seed=135387776198974078022358545371536932831 to reproduce this failure.
_________________________ test_open_graphical_browser __________________________
tests/system/utils/test_main.py:71: in test_open_graphical_browser
    iter(webbrowser._tryorder)
E   TypeError: 'NoneType' object is not iterable
============== 3 failed, 329 passed, 118 skipped in 59.71 seconds ==============
E: pybuild pybuild:338: test: plugin distutils failed with: exit code=1: cd /<<PKGBUILDDIR>>/.pybuild/cpython3_3.7_vdirsyncer/build; python3.7 -m pytest tests
dh_auto_test: pybuild --test --test-pytest -i python{version} -p "3.7 3.6" returned exit code 13
make[1]: *** [debian/rules:23: override_dh_auto_test] Error 25
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
make: *** [debian/rules:7: build-indep] Error 2
dpkg-buildpackage: error: debian/rules build-indep subprocess returned exit status 2
--------------------------------------------------------------------------------

The build was made in my autobuilder with "dpkg-buildpackage -A"
but it also fails here:

https://tests.reproducible-builds.org/debian/rb-pkg/unstable/amd64/vdirsyncer.html

where you can get a full build log if you need it.

If this is really a bug in one of the build-depends, please use reassign and affects,
so that this is still visible in the BTS web page for this package.

Thanks.