Skip to content
Snippets Groups Projects

Import Fatras

Merged Moritz Kiehn requested to merge import_fatras into master

This imports the current master from acts-fatras into the Fatras directory. To keep this separate from the core plugins the include files are located in ActsFatras/ and all code is in the ActsFatras namespace. Tests are available in the Tests/UnitTests/Fatras directory.

The outstanding issues for Fatras will be moved to the core repository once this has been merge (contrary to previous plans of fixing all issues first).

Contains some small modifications to simplify adding unit/integration tests with additional libraries.

Preparation for #686.

Edited by Moritz Kiehn

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • I think I'm ok with this.

  • Another question: FATRAS unit tests coverage is not included. I think this is because we disable it in the coverage exclusion patterns. This is also the case with the plugins. Maybe we need to devise a way how cmake can communicate which plugins are enabled, and those plugins and their tests are then taken into account in the coverage calculation.

    Hm, maybe it isn't.

    Edited by Paul Gessinger
  • In any case, the CI should build FATRAS.

  • Author Contributor

    I will enable it in the CI build.

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