Skip to content

Bump up chrome driver #13925

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 21, 2025
Merged

Bump up chrome driver #13925

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6578c83
Bump up chrome driver
MarcialRosales May 21, 2025
44dd282
Eliminate flake around listing live amqp connections
MarcialRosales May 21, 2025
a028db8
Briefly explain how to start a second rabbitmq server
MarcialRosales May 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions selenium/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -168,6 +168,12 @@ suites/authnz-mgt/oauth-with-uaa.sh test happy-login.js
been implemented yet.


If your test requires two RabbitMQ servers, typically required when testing WSR or shovels or federation,
you can run the second server, a.k.a. `downstream`, as follows:
```
suites/<yoursuite>.sh start-other-rabbitmq
```

## Test case configuration

RabbitMQ and other components such as UAA, or Keycloak, require configuration files which varies
Expand Down
2 changes: 1 addition & 1 deletion selenium/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
"author": "",
"license": "ISC",
"dependencies": {
"chromedriver": "^134.0",
"chromedriver": "^135.0",
"ejs": "^3.1.8",
"express": "^4.18.2",
"geckodriver": "^3.0.2",
Expand Down
5 changes: 3 additions & 2 deletions selenium/test/connections/amqp10/sessions-for-monitoring-user.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ const { By, Key, until, Builder } = require('selenium-webdriver')
require('chromedriver')
const assert = require('assert')
const { open: openAmqp, once: onceAmqp, on: onAmqp, close: closeAmqp } = require('../../amqp')
const { buildDriver, goToHome, captureScreensFor, teardown, delay } = require('../../utils')
const { buildDriver, goToHome, captureScreensFor, teardown, delay, doWhile } = require('../../utils')

const LoginPage = require('../../pageobjects/LoginPage')
const OverviewPage = require('../../pageobjects/OverviewPage')
Expand Down Expand Up @@ -98,7 +98,8 @@ describe('Given an amqp10 connection opened, listed and clicked on it', function
assert.equal(2, receivedAmqpMessageCount)

await delay(5*1000) // wait until page refreshes
let sessions = await connectionPage.getSessions()
let sessions = await doWhile(function() { return connectionPage.getSessions() },
function(obj) { return obj != undefined })
let incomingLink = connectionPage.getIncomingLinkInfo(sessions.incoming_links, 0)
assert.equal(2, incomingLink.deliveryCount)

Expand Down
1 change: 0 additions & 1 deletion selenium/test/pageobjects/ConnectionPage.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ const { By, Key, until, Builder } = require('selenium-webdriver')
const BasePage = require('./BasePage')


const OVERVIEW_SECTION = By.css('div#main div.section#connection-overview-section')
const SESSIONS_SECTION = By.css('div#main div.section#connection-sessions-section')
const SESSIONS_TABLE = By.css('div.section#connection-sessions-section table.list#sessions')
const INCOMING_LINKS_TABLE = By.css('div.section#connection-sessions-section table.list#incoming-links')
Expand Down
Loading