Bug 1341514 - Using the valid update url provider in test. r=gcp

MozReview-Commit-ID: 50NcRd2e9hB
This commit is contained in:
Thomas Nguyen
2017-03-02 00:07:53 +08:00
parent 25058b30c8
commit c68b0c13e5
2 changed files with 5 additions and 3 deletions

View File

@@ -26,4 +26,3 @@ tags = trackingprotection
[test_donottrack.html] [test_donottrack.html]
[test_classifier_changetablepref.html] [test_classifier_changetablepref.html]
[test_reporturl.html] [test_reporturl.html]
skip-if = true # Bug 1341514

View File

@@ -94,6 +94,7 @@ function testOnWindow(aTestData, aCallback, aTestCreater) {
is(button, undefined, "Report button should not be showed"); is(button, undefined, "Report button should not be showed");
win.close(); win.close();
resolve(); resolve();
return;
} }
button.click(); button.click();
@@ -158,11 +159,13 @@ var testDatas = [
expectedReportUri: "http://itisaphishingsite.org/phishing.html" expectedReportUri: "http://itisaphishingsite.org/phishing.html"
}, },
// Non-google provider, no report button is showed // Non-google provider, no report button is showed.
// Test provider needs a valid update URL (mozilla for example) otherwise
// the updates inserting the test data will fail.
{ topUrl: "http://fakeitisaphishingsite.org/phishing.html", { topUrl: "http://fakeitisaphishingsite.org/phishing.html",
testUrl: "fakeitisaphishingsite.org/phishing.html", testUrl: "fakeitisaphishingsite.org/phishing.html",
list: "fake-phish-simple", list: "fake-phish-simple",
provider: "fake", provider: "mozilla",
blockCreater : createBlockedPage blockCreater : createBlockedPage
}, },