test: added an edge case for LatticeReactionSystem

[sivasathyaseeelan]

I have added an edge cases to test LatticeReactionSystem which were not tested before.

Checklist

• Appropriate tests were added
• Any code changes were done in a way that does not break public API
• All documentation related to code changes were updated
• The new code follows the
  contributor guidelines, in particular the SciML Style Guide and
  COLPRAC.
• Any new documentation only uses public API

Additional context

Add any other context about the problem here.

[TorkelE]

Just checking, exactly what is this meant to test? That you cannot create LatticeReactionSystems with random inputs? Seems a bit excessive. And if we want to test it, is there a specific reason for creating a new structure?
(no direct problem, just trying to understand what the actual test intention is)

[sivasathyaseeelan]

I am trying to test, if we input any invalid type instead of AbstractSpatialReaction then it should throws an argument error

[TorkelE]

Sounds good. I'd probably call it InvalidSpatialReactionType, or maybe just use a reaction.

Also, in the test already run include("../spatial_test_networks.jl"), which loads various networks and lattices. I'd use some simple examples from there instead. That way, new networks and alttices do not have to be created.

[sivasathyaseeelan]

Yeah make sense. I will modify the test!

[sivasathyaseeelan]

@TorkelE I have modified the test as per your above comments. Could you please review it?

[TorkelE]

Looks good, thanks a lot :)

[isaacsas]

This needs to have master merged into it to get the doc build error fix.

@TorkelE feel free to merge when you are happy.

[isaacsas]

(I guess we can ignore the doc failure since this is just updating a test.)

[TorkelE]

I have merged master into this, happy to take from here @sivasathyaseeelan (thanks again for the PR :) )