Possibility to filter notification target(s) by user(s)

[Xplouder]

Let me know if the tests are mandatory or just optional.

[coveralls]

Coverage decreased (-0.3%) to 99.581% when pulling 0a49b71 on Xplouder:feature/filter_by_users into 89d609e on gomoob:master.

[bgaillard]

Hi @Xplouder and thanks for this new feature.

Let me know if the tests are mandatory or just optional.

Yes, since version 1.8.0 code coverage is 100%, to respect our quality standards we wish to keep 100% code coverage now.

So could you complete the unit tests associated to the Notification class to increase the code coverage ?

Thanks

[coveralls]

Coverage increased (+0.0006%) to 99.832% when pulling c96b3ef on Xplouder:feature/filter_by_users into 89d609e on gomoob:master.

[coveralls]

Coverage increased (+0.0006%) to 99.832% when pulling c96b3ef on Xplouder:feature/filter_by_users into 89d609e on gomoob:master.

[coveralls]

Coverage increased (+0.0006%) to 99.832% when pulling c96b3ef on Xplouder:feature/filter_by_users into 89d609e on gomoob:master.

[Xplouder]

@bgaillard Any ETA to get this merged?