search

fjuniorr / flowmapper

Mappings between elementary flows
MIT License
0 stars 1 forks source link

Suffixes can got both ways #47

Closed cmutel closed 6 months ago

cmutel commented 6 months ago

Needs tests!

codecov-commenter commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f404781) 72.22% compared to head (4c91a89) 72.22%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #47 +/- ## ======================================= Coverage 72.22% 72.22% ======================================= Files 10 10 Lines 342 342 ======================================= Hits 247 247 Misses 95 95 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

fjuniorr commented 6 months ago

I was trying to add tests with flows that were actually matched with the change but all of the 51 new mappings were from the condition f"{t.name}, {suffix}" == s.name with suffix equals ion.

The new mappings are all from agribalyse-3.1.1-biosphere.