add fusion product iterator test#99
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests.
... and 1 file with indirect coverage changes 🚀 New features to boost your workflow:
|
| end | ||
| end | ||
|
|
||
| @testsuite "Fusion product iterator" I -> begin |
There was a problem hiding this comment.
This might need a guard to only check this if the iteratorsize is haslength. In principle we always expect this to be true, but sometimes it can be convenient to have size unknown if the implementation is annoying.
As a separate question, maybe this can go together with the check on the dimensions just below here?
There was a problem hiding this comment.
This might need a guard to only check this if the iteratorsize is haslength. In principle we always expect this to be true, but sometimes it can be convenient to have size unknown if the implementation is annoying.
How will this then work for sector types which don't use SectorProductIterator? Do we just skip those?
Edit: I forgot other iterator types have an iterator size property
As a separate question, maybe this can go together with the check on the dimensions just below here?
I don't mind combining them
…ors.jl into bd/fusionproditerator
2 participants
Test whether the fusion product iterator was implemented correctly. Not sure if this should be extended to also check via N-symbols?