Skip to content

Feature add graphviz for tests

James Kennington requested to merge feature-add-graphviz-for-tests into main

Changes

Adds:

  • graphviz as an optional dependency to "test" dependency group sgn
  • enable testing in CI with graphviz installed
  • test behavior of ImportError in case of no graphviz installed
  • achieve true 100% coverage by line count (reflected in result of CI build on this MR)

Motivation

Previous MR !23 (merged) added unit test infrastructure for 97% percent of lines (excluding the graphviz import logic); however, some of those tests were skipped in the CI build do to missing optional dependencies. This MR achieves full, 100% test coverage by adding those deps to CI build and testing import logic.

Edited by James Kennington

Merge request reports