1290557 - Add basic notification tests

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: Testing, defect, P5)

Description

[Jonathan Almeida [:jonalmeida]]

While looking at bug 1279301, I noticed we don't have any specific tests that can be run to ensure basic functionality of notifications still worked correctly.

Comment 1

[Jonathan Almeida [:jonalmeida]]

Sebastian, do you have any ideas of simple tests that can be written?

Adding the 'good next bug' tag, but feel free to change this if you don't think this should be changed to something else.

Comment 2

[Sebastian Kaspari (:sebastian; :pocmo)]

Also see bug 1287683 comment 2 for details: UI tests can't access notifications and we do not use the UIAutomator framework (which could access notifications) anywhere yet.

However as we do not intend to test that the Android system works (=> actually clicking notifications) we might be able to mock the parts we can't access: If we want to test what happens if we click a notification/action: Can we just send the intent ourselves during test? If we just want to test that a notification is shown: Can we mock NotificationManager in our code and verify that it gets called with the right parameters?

Comment 3

[Sebastian Kaspari (:sebastian; :pocmo)]

For setting 'good next bug' we should have a rough idea about what to do, so that we can mentor it. This doesn't seem to be a good candidate (yet).
As soon as we have concrete ideas about how a test could look like, we can file those as separate bugs and make them mentor bugs.

Comment 4

[Tomislav Jurin]

I would like to take this on if it is still open.

Comment 5

[Geoff Brown [:gbrown]]

Mass closing Firefox for Android :: Testing bugs with no progress in 2017.

If this bug is important to you, please re-open.

Comment 6

[Geoff Brown [:gbrown]]

(In reply to Tomislav Jurin from comment #4)
> I would like to take this on if it is still open.

Sorry we missed this! If still interested, please needinfo the mentor, or needinfo me.