Test : Check Snackbar is displayed when the deck is created

[neeldoshii]

Purpose / Description

This test checks whether the toast is shown to user after the deck has been created.

Fix

Anki-Android/AnkiDroid/src/main/java/com/ichi2/anki/dialogs/CreateDeckDialog.kt

Line 50 in 1c5a6b3

@NeedsTest("Ensure a toast is shown on a successful action")

Note :
I am currently new towards writing test feel free to pin out or recommend the mistakes or advise while testing.

[david-allison]

We prefer Robolectric (/test/) over /androidTest/ unless there's a good reason

(don't worry about the GSoC deadline regarding testing, we can still talk about it afterwards)

[neeldoshii]

Few Doubts: @david-allison

1. @NeedsTest("Ensure a toast is shown on a successful action")
   I am not sure is this right or wrong. From what i observed creating a deck or renaming a deck we are showing snackbar not toast. I did wrote the unit test (robolectric) for checking whether it showed toast or not but it was giving me null everytime so i later got to know that we are calling snackbar on successful creation. Just like ShadowToast do we have ShadowSnackbar which does the work?

For now I have just fixed my old instrument test.

[david-allison]

If you annotate (git blame), you'll see the @NeedsTest was added in 2022 and you'll see the full context of why we wanted a test

• feat: improve deck creation/rename UX #12139

---

Then look at the history of the class to understand how things have changed over time

See from the git history for the file: be85b46 (ShowSnackbarInsteadOfToast)

---

The intention of the @NeedsTest is that a user is provided feedback when an action is made.

So test for snackbars, not toasts, and as above, use /test/ rather than /androidTest

[david-allison]

as below

[neeldoshii]

Hi Maintainers, I am unsure of how can I check if the Snackbar is displayed or not in Roboelectric? There used to be shadowsnackbar but its now deprecated.

Can you share me what method do we have to test snackbar or any resource link?

[david-allison]

This should be in /test/, not in /androidTest

Closing this as stale to reduce our number of pending PRs. I was going to fix this up, but it needs a rewrite.

Feel free to recreate when this is done