zacck-zz commented 6 years ago

What's in this PR?

Added a test block for Github.Sync.installation event

Make sure any changes to code include changes to documentation.

References

Fixes #1373

Progress on: #

zacck-zz commented 6 years ago

Some clauses defined seem difficult or impossible to reach so their tests were omitted

{:error, :validation_error_on_syncing_installation, Changeset.t()}
{:error, :validation_error_on_marking_installation_processed, Changeset.t()}
{:error, :unexpected_transaction_outcome, any}
{:error, :validation_error_on_deleting_removed_repos, {list, list}}
- {:error, :validation_error_on_syncing_existing_repos, {list, list}} However, if these clauses can be caused by some updates upstream we should cover them with tests

zacck-zz commented 6 years ago

🤘

begedin commented 6 years ago

Merged 👍

Thank you for your time and work @zacck