Skip to content
Snippets Groups Projects

Introduce Pytest-based tests for the DaVinci configuration

Merged Eduardo Rodrigues requested to merge eduardo-introduce-pytest into master

Work towards a DaVinci test suite, see https://gitlab.cern.ch/lhcb-dpa/project/-/issues/179.

Edited by Eduardo Rodrigues

Merge request reports

Merge request pipeline #3333806 passed

Merge request pipeline passed for bd0b0563

Merged by Patrick KoppenburgPatrick Koppenburg 3 years ago (Dec 9, 2021 3:32pm UTC)

Loading

Pipeline #3337382 passed

Pipeline passed for d8a3bae0 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Eduardo Rodrigues resolved all threads

    resolved all threads

  • Eduardo Rodrigues
  • Eduardo Rodrigues mentioned in merge request !604 (merged)

    mentioned in merge request !604 (merged)

  • Eduardo Rodrigues changed title from WIP - Introduce Pytest-based test for the DaVinci configuration to WIP - Introduce Pytest-based tests for the DaVinci configuration

    changed title from WIP - Introduce Pytest-based test for the DaVinci configuration to WIP - Introduce Pytest-based tests for the DaVinci configuration

  • requested review from @pkoppenb and @dfazzini

  • added 1 commit

    • 84aec05b - Remove redundant algo tests in Phys/DaVinci/tests/

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Eduardo Rodrigues changed title from WIP - Introduce Pytest-based tests for the DaVinci configuration to Introduce Pytest-based tests for the DaVinci configuration

    changed title from WIP - Introduce Pytest-based tests for the DaVinci configuration to Introduce Pytest-based tests for the DaVinci configuration

  • added 1 commit

    • 0cb44855 - Start on unit tests for DaVinci.configOptions.list_data_options

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • This is ready for a CI test (a few trivial pytest tests added) if you really want to be sure. Then we can proceed with !604 (merged) :-).

  • Patrick Koppenburg approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading