Cleaning DaVinci tests and add new DaVinciExamples

Cleaning the DaVinci qmtests removing dependencies between DaVinciTests and DaVinciExamples. All the tests that provide examples for running DaVinci are now moved into DaVinciExamples while DaVinciTests is meant to have only unit tests.
The environment variable $DAVINCIROOT is reintroduced to access the Phys/DaVinci/options/DaVinciDB-Example.yaml database. This is just a temporary solution since in the end all the metadata infos of the input files used in the qmtests will be merged in the general TestFileDB, so the tests will not depend anymore on the Phys/DaVinci package.

Addresses #20 (closed) and provide some more examples for https://gitlab.cern.ch/lhcb-dpa/project/-/issues/102 and an example for running DaVinci with gaudirun.py required by #23 (closed).

added DPA-WP3 testing labels

assigned to @dfazzini

changed the description

changed the description

Happy New Year @dfazzini!

FYI I linked this to issue #20 (closed) as I believe it addresses it and even closes it. But please let me/us know if there is more to it.

added 1 commit

• 37ed8f42 - fix failing tests

Compare with previous version

added 1 commit

• 54b834b7 - fix python Unicode string conversion error

Compare with previous version

mentioned in issue #23 (closed)

changed title from Draft: Cleaning DaVinci tests to Draft: Cleaning DaVinci tests and add new DaVinciExamples

changed the description

I presume this is just shuffling files around, in which case the only thing that counts is that the tests pass.

Such kind of MR are generators of merge conflicts, so it should me merged asap to avoid annoying everyone else.

approved this merge request

assigned to @pkoppenb

approved this merge request

@dfazzini, you now have 2 approvals. Un-WIP if happy as well so that we can trigger a CI test asap.

You still have issues with Unicode characters so probably better to run the copyright, etc. locally "file by file" to see which is the culprit ... Nothing obvious by eye.

added 1 commit

• 41d8c992 - final changes

Compare with previous version

added 1 commit

• 20e9d1df - Fixed formatting

Compare with previous version

added 1 commit

• 901f827d - fix unicode error

Compare with previous version

added 1 commit

• c731a796 - remove ref file temporary

Compare with previous version

added 1 commit

• d9a32dd4 - Fixed formatting

Compare with previous version

added 1 commit

• 480c3131 - restore ref file

Compare with previous version

marked this merge request as ready

/ci-test --merge

resolved all threads

added ci-test-triggered label

• [2022-01-06 18:42] Validation started with lhcb-master-mr#3393

changed the description

assigned to @erodrigu

resolved all threads

approved this merge request

merged

mentioned in commit 3f475434

mentioned in merge request !618 (merged)

mentioned in merge request !639 (merged)