It will become mandatory in a future version.

This always happens when writing to Kafka, as the Kafka writer sets it
to None at the same it it injects the 'tool' data.

This was not caught by tests because they use the in-mem writer;
which did not call unique_key() at all in swh-journal<=v0.5.1
(but future versions will).

This fixes the indexer debian package build.

It's simplifying reading and more consistent with other similar tests

... instead of OriginVisit.

OriginVisit model object no longer hold status information so the current
filtering happening on the journal client side could not work.

Related to T2814
Related to P882

The minimum configuration is provided either by the --config-file
or the --broker parameters

Related to T2814

swh.storage and swh.objstorage, as well as swh.indexer itself, have deprecated
using an explicit `args` in their factories for a while; we can drop them now.

vcversioner was already removed months ago.

test_origin_intrinsic_metadata_add__deadlock: use more values, to make the test less likely to unexpectedly pass.

postgresql kindly returns the results in the order the test expected... most of the time.

Commits in the last month made these endpoints more consistent with
the other ones (though not completely), so these tests don't need to
be skipped anymore.

By removing the False behavior, which we didn't use in practice,
and was removed from the indexers in the previous commit.

The main motivation is to make _add endpoints write to Kafka in a
future commit, as Kafka's semantics are closer to conflict_update=True
than conflict_update=False.

By removing the False/ignore-dups behavior, which we didn't use.

The main motivation is to make _add endpoints write to Kafka in a
future commit, as Kafka's semantics are closer to True/update-dups
than False/ignore-dups.

This was expected to be used in these two cases:

1. if we remove mappings or file detection from a metadata indexer
2. if an origin removes all its metadata files

but:

1. if we do so, then we should bump the indexer version, so the
   old metadata will be preserved anyway, as different indexer
   versions get different indexer_configuration_ids
2. this should be a rather rare even, and even if it happens, we
   might want to keep the old metadata anyway rather than
   nothing (even if it's outdated), for search purposes.

Additionally, this commit is motivated by:

* that's less issues to deal with when writing to Kafka (the journal
  writer currently doesn't support suppression; and we would also have
  to add support for deletion in all consumers)
* less code (~250 lines)

This should fix [1]

[1] https://jenkins.softwareheritage.org/job/DENV/job/tests/lastBuild/artifact/swh-indexer.log

According to the task schema, that parameter should not be null and defaults to
0. But current scheduler backend implementation expects a value nonetheless and
if it's null, that breaks the call.

This should fix [1]

[1]
https://jenkins.softwareheritage.org/job/DENV/job/tests/lastFailedBuild/artifact/swh-indexer.log

So the `swh db *` commands work expectedly.

Related to T2736

Related to T1410

They are all migrated to attr classes now.

also add typing to some test functions

also add typing to some test functions

For consistency with the main storage.

1. it was wrongfully annotated as '-> TResult' even though some indexers
   can return None
2. in a future commit, the fossology indexer will need to return multiple
   results.

When going through the RPC, it's turned into a list.

By adding a simple type annotation to the test functions' signature.

So the type of results can be statically checked, instead of needing to
assert it to please mypy.

This temporarily adds mess in the generic tests to support both rows and dicts,
but I'll remove it once I migrated all endpoints.

And fix the incorrect test `test_generate_content_get`, which was
passing for the wrong reasons.