GitLab

In python3, lxml.etree.tostring returns bytes but we expect text in
tests. By passing six.text_type as encoding parameter to this function,
it returns a unicode string.

Avoids FutureWarning: The behavior of this method will change in future
versions. Use specific 'len(elem)' or 'elem is not None' test instead.

This is an enum in the original xsd while we don't output possible values
currently, so in the mean time make it mandatory, it's still a bit better than
relying on the original schema for that, and it makes sense from a business POV
to force specification of description level of an archive unit.

A lot of tests have to be updated, some of them because output value is now
different, others because we've to introduce a scheme+concept to handle the
validation, but they were not expecting that.

Closes #16687907

it's not required anymore and doesn't apply to transfer request
(To be confirmed by Baptiste)

* make it mandatory ;

* update migration script removing id to copy existing value to annotation
  before removal ;

* regenerate test data file since this change adds exported annotations.

Along the way, add missing escaping when displaying user_annotation, by using
printable_value.

Related to #16070476

They have been introduced at a first glance to handle references during profile
generation as well as display of 'first class' element in the UI. The main problem is
that they clash when using the 'import archive unit' feature, since one may not
import the same unit twice without changing the id first. Another problem is
that their purpose is not clear to end-user.

This changeset remove the attribute, use eid instead during profile generation
and fix tests. UI and proper migration will be handled in later csets.

Related #16070476

Closes #11447493