Skip to content

Commit f036d2f

Browse files
dosasdosas
authored
Merge branch 'trunk' into fix/typos
2 parents 426ebcb + c3a19c8 commit f036d2f

11 files changed

Lines changed: 60 additions & 59 deletions

File tree

‎.github/workflows/java.yml‎

Lines changed: 20 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -98,9 +98,12 @@ jobs:
9898
with:
9999
query: attr(tags, lint, tests(//java/...))
100100

101-
chrome_tests:
101+
browser_tests:
102102
needs: small_tests
103103
runs-on: ubuntu-latest
104+
strategy:
105+
matrix:
106+
browser: [ chrome, firefox ]
104107
steps:
105108
- name: Checkout source tree
106109
uses: actions/checkout@v2
@@ -110,42 +113,9 @@ jobs:
110113
path: |
111114
~/.cache/bazel-disk
112115
~/.cache/bazel-repo
113-
key: ${{ runner.os }}-bazel-chrome-tests-${{ hashFiles('**/BUILD.bazel') }}
116+
key: ${{ runner.os }}-bazel-${{ matrix.browser }}-tests-${{ hashFiles('**/BUILD.bazel') }}
114117
restore-keys: |
115-
${{ runner.os }}-bazel-chrome-tests-
116-
${{ runner.os }}-bazel-build-
117-
- name: Setup bazelisk
118-
uses: ./.github/actions/setup-bazelisk
119-
- name: Setup Java
120-
uses: actions/setup-java@v1
121-
with:
122-
java-version: '11'
123-
- name: Setup Chrome and ChromeDriver
124-
uses: ./.github/actions/setup-chrome
125-
- name: Start XVFB
126-
run: Xvfb :99 &
127-
- name: Run browser tests in Chrome
128-
uses: ./.github/actions/bazel-test
129-
with:
130-
query: attr(tags, chrome, tests(//java/...)) except attr(tags, remote, tests(//java/...))
131-
env:
132-
DISPLAY: :99
133-
134-
firefox_tests:
135-
needs: small_tests
136-
runs-on: ubuntu-latest
137-
steps:
138-
- name: Checkout source tree
139-
uses: actions/checkout@v2
140-
- name: Cache Bazel artifacts
141-
uses: actions/cache@v2
142-
with:
143-
path: |
144-
~/.cache/bazel-disk
145-
~/.cache/bazel-repo
146-
key: ${{ runner.os }}-bazel-firefox-tests-${{ hashFiles('**/BUILD.bazel') }}
147-
restore-keys: |
148-
${{ runner.os }}-bazel-firefox-tests-
118+
${{ runner.os }}-bazel-${{ matrix.browser }}-tests-
149119
${{ runner.os }}-bazel-build-
150120
- name: Setup bazelisk
151121
uses: ./.github/actions/setup-bazelisk
@@ -155,20 +125,24 @@ jobs:
155125
java-version: '11'
156126
- name: Setup Firefox and GeckoDriver
157127
uses: ./.github/actions/setup-firefox
128+
- name: Setup Chrome and ChromeDriver
129+
uses: ./.github/actions/setup-chrome
158130
- name: Start XVFB
159131
run: Xvfb :99 &
160-
- name: Run browser tests in Firefox
132+
- name: Run browser tests in ${{ matrix.browser }}
161133
uses: ./.github/actions/bazel-test
162134
with:
163-
query: attr(tags, firefox, tests(//java/...)) except attr(tags, 'rc|remote', tests(//java/...))
135+
query: attr(tags, ${{ matrix.browser }}, tests(//java/...)) except attr(tags, 'lint|rc|remote', tests(//java/...))
164136
env:
165137
DISPLAY: :99
166138

167-
168139
server_tests:
169140
if: ${{ always() }}
170-
needs: [ chrome_tests, firefox_tests ]
141+
needs: [ browser_tests ]
171142
runs-on: ubuntu-latest
143+
strategy:
144+
matrix:
145+
browser: [ chrome, firefox ]
172146
steps:
173147
- name: Checkout source tree
174148
uses: actions/checkout@v2
@@ -178,28 +152,27 @@ jobs:
178152
path: |
179153
~/.cache/bazel-disk
180154
~/.cache/bazel-repo
181-
key: ${{ runner.os }}-bazel-server-tests-${{ hashFiles('**/BUILD.bazel') }}
155+
key: ${{ runner.os }}-bazel-${{ matrix.browser }}-server-tests-${{ hashFiles('**/BUILD.bazel') }}
182156
restore-keys: |
183-
${{ runner.os }}-bazel-server-tests-
184-
${{ runner.os }}-bazel-chrome-tests-
185-
${{ runner.os }}-bazel-firefox-tests-
157+
${{ runner.os }}-bazel-${{ matrix.browser }}-server-tests-
158+
${{ runner.os }}-bazel-${{ matrix.browser }}-tests-
186159
${{ runner.os }}-bazel-build-
187160
- name: Setup bazelisk
188161
uses: ./.github/actions/setup-bazelisk
189162
- name: Setup Java
190163
uses: actions/setup-java@v1
191164
with:
192165
java-version: '11'
193-
- name: Setup Chrome and ChromeDriver
194-
uses: ./.github/actions/setup-chrome
195166
- name: Setup Firefox and GeckoDriver
196167
uses: ./.github/actions/setup-firefox
168+
- name: Setup Chrome and ChromeDriver
169+
uses: ./.github/actions/setup-chrome
197170
- name: Start XVFB
198171
run: Xvfb :99 &
199172
- name: Run server tests
200173
uses: ./.github/actions/bazel-test
201174
with:
202-
query: tests(//java/...) except attr(tags, 'rc|lint|ie|edge|edgehtml|safari', tests(//java/...))
175+
query: attr(tags, ${{ matrix.browser }}, tests(//java/...)) intersect attr(tags, 'remote', tests(//java/...))
203176
env:
204177
DISPLAY: :99
205178

‎Rakefile‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ def release_version
5555
end
5656

5757
def version
58-
"#{release_version}.0"
58+
"#{release_version}.1"
5959
end
6060

6161
# The build system used by webdriver is layered on top of rake, and we call it

‎WORKSPACE‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -123,8 +123,8 @@ selenium_register_dotnet()
123123

124124
http_archive(
125125
name = "build_bazel_rules_nodejs",
126-
sha256 = "f0f76a06fd6c10e8fb9a6cb9389fa7d5816dbecd9b1685063f89fb20dc6822f3",
127-
urls = ["https://github.com/bazelbuild/rules_nodejs/releases/download/4.5.1/rules_nodejs-4.5.1.tar.gz"],
126+
sha256 = "778ed059ad37c3373c22adcb5c200cbbca72b370dd4de93379deb8325c0045cb",
127+
urls = ["https://github.com/bazelbuild/rules_nodejs/releases/download/4.5.0/rules_nodejs-4.5.0.tar.gz"],
128128
)
129129

130130
load("@build_bazel_rules_nodejs//:index.bzl", "node_repositories", "npm_install")

‎java/CHANGELOG‎

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,22 @@
1+
v4.1.1
2+
======
3+
4+
* Supported CDP versions: 85, 94, 95, 96
5+
* Allow custom tracing to integrate with existing OpenTelemetry tracing
6+
* Avoiding NoClassDefFoundError by waiting for completion of already started tasks
7+
* use addon parameter with base64 encoding instead of path parameter
8+
that requires using file detector and upload
9+
* support installing temporary addon in Firefox
10+
* Remove references to opera_blink in favour of opera
11+
* BugFix: Handle null parameters in TracedCommandExecutor
12+
* BugFix: Following redirects by default in HttpClient
13+
* BugFix: Forwarding CDP/BiDi connections to the container in Dynamic Grid
14+
* BugFix: Do not log screenshots payload in debug mode
15+
* BugFix: Mapping Slot fields properly for json payloads
16+
* BugFix: Configuring relay service via CLI
17+
* BugFix: Adding a "bind-host" flag to start Grid properly when the
18+
server cannot bind to the host, fixes this for environments like Docker
19+
120
v4.1.0
221
======
322

@@ -12,7 +31,7 @@ v4.1.0
1231
ConcurrentModificationException while reading exception message by parallel threads
1332
* Add retry http request filter for transient errors.
1433
* Fix Javadoc code block formatting
15-
* [grid] Using non-loopback address when default config is used
34+
* [grid] Using non-loopback ad dress when default config is used
1635
* Fix classloading error in host identifier
1736
* Split OpenTelemetry command in order to facilitate search by tags
1837

‎java/src/org/openqa/selenium/netty/server/NettyServer.java‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,9 @@ public NettyServer start() {
164164
throw new UncheckedIOException(new IOException("Start up interrupted", e));
165165
} catch (Exception e) {
166166
if (e instanceof BindException) {
167-
throw new UncheckedIOException(new IOException(String.format("Port %s already in use", port), e));
167+
String errorMessage = String.format(
168+
"Could not bind to address or port is already in use. Host %s, Port %s", host, port);
169+
throw new UncheckedIOException(new IOException(errorMessage, e));
168170
}
169171
throw e;
170172
}

‎java/test/org/openqa/selenium/firefox/BUILD.bazel‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,6 @@ java_selenium_test_suite(
5757
"//third_party/firebug:firebug-1.5.0-fx.xpi",
5858
"//third_party/firebug:mooltipass-1.1.87.xpi",
5959
],
60-
tags = [
61-
"selenium-remote",
62-
],
6360
deps = [
6461
"//java/src/org/openqa/selenium/firefox",
6562
"//java/src/org/openqa/selenium/remote",

‎java/test/org/openqa/selenium/grid/gridui/BUILD.bazel‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@ java_selenium_test_suite(
88
browsers = [
99
"firefox",
1010
],
11+
tags = [
12+
"selenium-remote",
13+
],
1114
deps = [
1215
"//java/src/org/openqa/selenium:core",
1316
"//java/src/org/openqa/selenium/grid",

‎java/test/org/openqa/selenium/grid/router/BUILD.bazel‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,9 @@ java_selenium_test_suite(
4141
"firefox",
4242
"edge",
4343
],
44+
tags = [
45+
"selenium-remote",
46+
],
4447
javacopts = [
4548
"--release",
4649
"11",

‎java/test/org/openqa/selenium/remote/BUILD.bazel‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
load("@rules_jvm_external//:defs.bzl", "artifact")
22
load("//java:defs.bzl", "java_selenium_test_suite", "java_test_suite")
33

4-
LARGE_TESTS = [
4+
REMOTE_TESTS = [
55
"ParallelSessionsTest.java",
66
"RemoteWebDriverScreenshotTest.java",
77
]
@@ -11,7 +11,7 @@ java_test_suite(
1111
size = "small",
1212
srcs = glob(
1313
["*.java"],
14-
exclude = LARGE_TESTS,
14+
exclude = REMOTE_TESTS,
1515
),
1616
javacopts = [
1717
"--release",
@@ -39,7 +39,10 @@ java_test_suite(
3939
java_selenium_test_suite(
4040
name = "large-tests",
4141
size = "large",
42-
srcs = LARGE_TESTS,
42+
tags = [
43+
"selenium-remote",
44+
],
45+
srcs = REMOTE_TESTS,
4346
deps = [
4447
"//java/src/org/openqa/selenium/remote",
4548
"//java/test/org/openqa/selenium/testing:annotations",

‎java/version.bzl‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
SE_VERSION = "4.1.0"
1+
SE_VERSION = "4.1.1"

0 commit comments

Comments
 (0)