oemof / oemof-solph / 10473118052 / 1

82%

dev: 82%

push

github

Merge pull request #1080 from oemof/revision/replace-constraint-tests

Replaces lp diff constraint tests by granular unit tests for functionality.

Reasoning:
- The LP files depend on Pyomo internals, thus they change when Pyomo changes things.
- They include tests for stuff from multiple parts of the code. For example, if I change the constraint formulation for the Bus, every constraint test needs to be rewritten, not only the one for the Bus.
- They are hard to understand (for many) and failed tests are hard to fix. The latter is particularly because of the undefined order of lines, so that just looking at a diff will not tell the difference: Constraints might be in a different order but still the same.

These unit tests still use Pyomo but the results are independent from the back-end. They are rather easy to understand (rather trivial test examples), and even internal constraint formulations can be changed without breaking the test, so it really tests for the specified functionally not the implementation.

1325 of 1517 branches covered (87.34%)

Branch coverage included in aggregate %.

2471 of 2747 relevant lines covered (89.95%)

0.9 hits per line