Skip to content
Snippets Groups Projects

Add unit tests for imported products

Merged Karel van Klink requested to merge feature/add-tests-for-imported-products into develop
All threads resolved!

Added unit tests for the imported_* products: both the create_imported_* and import_* workflows are now tested, and the fixtures have been updated so these kinds of products can now also be used in other tests.

Test coverage up from ~80% to 92% :tada:

Merge request reports

Pipeline #86761 passed

Pipeline passed for 74019763 on feature/add-tests-for-imported-products

Approved by

Merged by Karel van KlinkKarel van Klink 11 months ago (May 10, 2024 12:19pm UTC)

Merge details

  • Changes merged into develop with 74019763.
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #86762 passed

Pipeline passed for 74019763 on develop

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • 9d5c941e - Move unit test workflow inputs to pytest fixtures

    Compare with previous version

  • Karel van Klink resolved all threads

    resolved all threads

  • @mohammad.torkashvand Should be all good now :slight_smile:

  • Test coverage up from ~80% to 92% :tada:

  • Mohammad Torkashvand approved this merge request

    approved this merge request

  • added 8 commits

    • 9d5c941e...637cb0bb - 2 commits from branch develop
    • b7a8332f - Update fixtures to allow for un-imported varieties
    • 586f1a25 - Add imported site unit tests
    • d0b08050 - Add unit tests for importing routers
    • 924c0dd1 - Add unit tests for importing Iptrunks
    • 4cfe6542 - Add unit tests for importing office routers and super pop switches
    • 74019763 - Move unit test workflow inputs to pytest fixtures

    Compare with previous version

  • Karel van Klink enabled an automatic merge when the pipeline for 74019763 succeeds

    enabled an automatic merge when the pipeline for 74019763 succeeds

  • Please register or sign in to reply
    Loading