Add integration test for removing a series from user's collection

[mukeshk]

Addressed to #45

[mystamps-bot]

	1 Message
📖	@mukeshk thank you for the PR! All quality checks have been passed! Next step is to wait when @php-coder will review this code

Generated by 🚫 Danger

[php-coder]

test(collection/remove-series): add integration test for removing series from user's collection

Fixed #45

For future: it's better to keep use the convention that we already has -- "Fix #XXX"

[php-coder]

As far I remember we have decided to use series with id=3 We can't use a series with id=2 because it may affect collection/add-series/logic.robot test.

[mukeshk]

There was a comment which said to change the changeset comment to add series#1 and #2 into seriesowner collection. I thought we need to used existing series.

Now I created a series which gets id=3 and I have used it in the test case.

[php-coder]

@mukeshk You're right. I've checked and see that we can't use series#1 because it used by the test for searching in a collection. We can't use series#2 because we use it in the test for adding a series to collection. So, yes, the only way is to create yet another series to use in this test. I wish you told me that before so I understood it earlier ;) (it's a joke of course and I should be more careful).

[php-coder]

See my comments.

[php-coder]

Thank you! I'll merge it later. Also I'm going to add some improvements to the migration :)

[php-coder]

@mukeshk Thank you for the contribution! it was 10th PR! 🥇 See https://github.com/php-coder/mystamps/wiki/team-members