This is related to the previous commit

nsids are random identifiers, usually used to ease merging
between documents, and can trivially be used for fingerprinting.

- gentoo and debian with bubblewrap are not allowed to fail anymore
- don't run coverage on debian without bubblewrap

georg authored 5 years ago and jvoisin committed 5 years ago

It seems, there is a bug somewhere if the test suite is invoked as
'root', and bubblewrap is available.

Relates #106

georg authored 5 years ago and jvoisin committed 5 years ago

As such, hopefully, it's not really used widely anymore. If so, this
note isn't really relevant.

georg authored 5 years ago and jvoisin committed 5 years ago

Also, make the note generic, to omit the need to update it "constantly".

Closes #76

This shouldn't make a big difference in the CLI/extension
usage, but might improve the performances of long-running
instances, or people misusing the API.

atenart authored 5 years ago and jvoisin committed 5 years ago

Rework the dependencies definition to include a 'required' flags, which
is passed by the check_dependencies helper to the callers, so that they
can distinguish between required and optional dependencies.

This help in two ways:
- The unit test for the dependencies was now failing when an optional
  one was missing, due to a previous rework.
- Mat2's --check-dependencies was referring to "required dependencies"
  and was misleading for the user as some of them could be optional.

Signed-off-by: Antoine Tenart <antoine.tenart@ack.tf>

atenart authored 5 years ago and jvoisin committed 5 years ago

Remove the try/except logic when calling check_dependencies, as it
cannot throw the exception anymore (it's caught already in the
function).

Signed-off-by: Antoine Tenart <antoine.tenart@ack.tf>

This should ensure that if we decide to implement
some threading in the cli, a faulty file
won't break everything.

This one was spotted by @fuzzy