Skip to content

Commit

Permalink
Ensure extension is available on the path for Firefox tests (#9331)
Browse files Browse the repository at this point in the history
  • Loading branch information
pujagani authored Mar 24, 2021
1 parent 64447d4 commit 777cf25
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
9 changes: 9 additions & 0 deletions java/client/test/org/openqa/selenium/firefox/BUILD.bazel
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
load("@rules_jvm_external//:defs.bzl", "artifact")
load("//common:defs.bzl", "copy_file")
load("//java:defs.bzl", "java_selenium_test_suite", "java_test_suite")

LARGE_TESTS = [
Expand All @@ -20,6 +21,7 @@ java_test_suite(
data = [
"//third_party/firebug:firebug-1.5.0-fx.xpi",
"//third_party/firebug:mooltipass-1.1.87.xpi",
":firebug_xpi"
],
deps = [
"//java/client/src/org/openqa/selenium/firefox",
Expand Down Expand Up @@ -64,3 +66,10 @@ java_selenium_test_suite(
artifact("org.mockito:mockito-core"),
],
)

copy_file(
name = "firebug_xpi",
src = "//third_party/firebug:firebug-1.5.0-fx.xpi",
out = "firebug-1.5.0-fx.xpi",
)

Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@
public class FirefoxProfileTest {
private static final String FIREBUG_PATH = "third_party/firebug/firebug-1.5.0-fx.xpi";
private static final String FIREBUG_RESOURCE_PATH =
"/org/openqa/selenium/firefox/firebug-1.5.0-fx.xpi";
"java/client/test/org/openqa/selenium/firefox/firebug-1.5.0-fx.xpi";
private static final String MOOLTIPASS_PATH = "third_party/firebug/mooltipass-1.1.87.xpi";

private FirefoxProfile profile;
Expand Down Expand Up @@ -189,7 +189,6 @@ public void shouldInstallWebExtensionFromDirectory() throws IOException {
}

@Test
@Ignore("Need to figure out why files are not showing up")
public void shouldInstallExtensionUsingClasspath() {
profile.addExtension(FirefoxProfileTest.class, FIREBUG_RESOURCE_PATH);
File profileDir = profile.layoutOnDisk();
Expand Down

0 comments on commit 777cf25

Please sign in to comment.