826261 - Robocop: Add test for Share Link option from the Firefox Mobile Menu

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Adrian Tamas (:AdrianT)]

Attachment: Share Link test

This Robocop test covers the share link feature. It tests that all available share options are displayed in the share options list.

Moztrap test case: https://moztrap.mozilla.org/manage/case/295/

Comment 1

[Geoff Brown [:gbrown]]

Comment on attachment 697430 [details] [diff] [review]
Share Link test

Review of attachment 697430 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/base/tests/testShareLink.java.in
@@ +96,5 @@
> +    }
> +
> +    private boolean optionDisplayed(String shareOption, ArrayList<String> displayedOptions) {
> +        for (String displayedOption: displayedOptions) {
> +            if (shareOption == displayedOption) {

Use .equals in preference to == for string comparison.

@@ +115,5 @@
> +                       return true;
> +               }
> +               return false;
> +            }
> +        }, WAIT_FOR_TEST_TIMEOUT);

Use BaseTest.MAX_WAIT_MS - https://hg.mozilla.org/mozilla-central/file/6955309291ee/mobile/android/base/tests/BaseTest.java.in#l35 - unless you really need a different timeout value.

Comment 2

[Adrian Tamas (:AdrianT)]

Attachment: Share Link test v1.1

Made the changes requested by Geoff

Comment 3

[Joel Maher ( :jmaher ) (UTC -0800)]

Comment on attachment 697438 [details] [diff] [review]
Share Link test v1.1

Review of attachment 697438 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for the patch.

please upload a new patch with the nit's addressed and I can land.  This passes consistently while running locally!

::: mobile/android/base/tests/testShareLink.java.in
@@ +51,5 @@
> +        }
> +        return shareOptions;
> +    }
> +
> +    public void verifyOptions(ArrayList<String> shareOptions){

nit: space between ){

@@ +53,5 @@
> +    }
> +
> +    public void verifyOptions(ArrayList<String> shareOptions){
> +        Device device = new Device();
> +        if (device.version == "2.x") {

how about "device.version.equals("2.x")"

@@ +58,5 @@
> +            ArrayList<String> displayedOptions = new ArrayList();
> +            ListView shareMenu = getDisplayedShareList();
> +
> +            /* Will have to go in the ListView, get each child, for the child separate the icon and the label
> +            and from the label get the label text in a String Array */

nit: align the two lines of text: 
/* Will ..
   and ... */

@@ +59,5 @@
> +            ListView shareMenu = getDisplayedShareList();
> +
> +            /* Will have to go in the ListView, get each child, for the child separate the icon and the label
> +            and from the label get the label text in a String Array */
> +            for (int i = 0; i < shareMenu.getAdapter().getCount();i++){

nit: space between ){

@@ +82,5 @@
> +            }
> +
> +            for (String option:shareOptions) {
> +                 // Verify if the option is present in the list of displayed share options
> +                 mAsserter.ok(optionDisplayed(option,displayedOptions), "Share option found", option);

nit: space between (option, displayedOptions)

@@ +118,5 @@
> +        }, MAX_WAIT_MS);
> +        mAsserter.ok(success,"Got the displayed share options?", "Got the share options list");
> +        return list;
> +    }
> +    

nit: extra whitespace.

Comment 4

[Adrian Tamas (:AdrianT)]

Attachment: Share Link test v1.2

Cleaned up the patch as requested

Comment 5

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/5a20d8fd94cd