The URL generation code is somewhat complex and I think it could do with unit testing.
The current JSON doesn't have any collisions so this prevents us from getting 100% coverage - line 57 isn't hit. Unit testing this proc would allow 100% test coverage.
This would also provide some documentation/examples of how it works.
I think we would need to remove the proc from within before. This has been discussed before and was also brought up by Louis in the formative feedback
The URL generation code is somewhat complex and I think it could do with unit testing.
The current JSON doesn't have any collisions so this prevents us from getting 100% coverage - line 57 isn't hit. Unit testing this proc would allow 100% test coverage.
This would also provide some documentation/examples of how it works.
I think we would need to remove the proc from within before. This has been discussed before and was also brought up by Louis in the formative feedback