Options to ignore the App + Assets in generic tests.

[ggwpez]

Adds IgnoreApp and IgnoreAssets options to make the [channel] GenericBackendTest work with the polkadot backend.

Closes #170

[matthiasgeihs]

This looks fine, but I don't know if this is actually needed. Instead of specifying options, you can setup state1 and state2 such that they are equal on the fields that you don't want to test them.

[ggwpez]

This looks fine, but I don't know if this is actually needed. Instead of specifying options, you can setup state1 and state2 such that they are equal on the fields that you don't want to test them.

I will try it without for now and see if it works.

PS: It does not work since there is only one asset in the Polkadot backend.

[ggwpez]

Confirmed to work with perun-network/perun-polkadot-backend#3, resolving draft status.

[matthiasgeihs]

Looks good, I was hoping we would get away without another change to the core.

I also experimented a bit with a different approach: simply checking whether the Assets are equal or not. And in this case ignoring them. This also works. A reoccurring problem seems to be that the function buildModifiedStates is supposed to only output modified states, but this may not be true depending on the inputs given. Here is my single-line fix: perun-network@556b481

I don't know. Your approach might be cleaner. But I also think conceptionally there is something wrong because if a function is called buildModifiedStates it should always output modified states (or error), and not just if the inputs are prepared in a specific way.

[matthiasgeihs]

@ggwpez Please see my review above. I am fine with your version, but I have also an alternative suggestion. If you prefer your approach, feel free to merge, but I think you need to rebase first anyways.