Skip to content
This repository has been archived by the owner on Oct 16, 2023. It is now read-only.

Fix Selenium UI tests to work with new jersey #1008

Closed
GCHQDeveloper314 opened this issue Jul 13, 2022 · 1 comment · Fixed by #1009
Closed

Fix Selenium UI tests to work with new jersey #1008

GCHQDeveloper314 opened this issue Jul 13, 2022 · 1 comment · Fixed by #1009
Assignees
@GCHQDeveloper314
Labels
dependencies enhancement
Milestone
v2.0.0-alpha-0.3

Comments

@GCHQDeveloper314
Copy link
Member

In Gaffer 2.0.0-alpha-3, jersey was updated to 2.36 (Gaffer #2689). This involved some dependency changes. These have caused the Selenium UI tests to fail with:

[ERROR] uk.gov.gchq.gaffer.ui.QueryBuilderST  Time elapsed: 3.563 s  <<< ERROR!
java.lang.IllegalStateException: InjectionManagerFactory not found.
	at org.glassfish.jersey.internal.inject.Injections.lambda$lookupInjectionManagerFactory$0(Injections.java:74)
	at java.base/java.util.Optional.orElseThrow(Optional.java:408)
	at org.glassfish.jersey.internal.inject.Injections.lookupInjectionManagerFactory(Injections.java:74)
	at org.glassfish.jersey.internal.inject.Injections.createInjectionManager(Injections.java:44)
	at org.glassfish.jersey.client.ClientConfig$State.initRuntime(ClientConfig.java:413)
	at org.glassfish.jersey.internal.util.collection.Values$LazyValueImpl.get(Values.java:317)
	at org.glassfish.jersey.client.ClientConfig.getRuntime(ClientConfig.java:820)
	at org.glassfish.jersey.client.ClientRequest.getClientRuntime(ClientRequest.java:176)
	at org.glassfish.jersey.client.ClientRequest.getInjectionManager(ClientRequest.java:567)
	at org.glassfish.jersey.client.JerseyWebTarget.onBuilder(JerseyWebTarget.java:371)
	at org.glassfish.jersey.client.JerseyWebTarget.request(JerseyWebTarget.java:199)
	at org.glassfish.jersey.client.JerseyWebTarget.request(JerseyWebTarget.java:38)
	at uk.gov.gchq.gaffer.proxystore.ProxyStore.createRequest(ProxyStore.java:272)
	at uk.gov.gchq.gaffer.proxystore.ProxyStore.doGet(ProxyStore.java:232)

Making similar jersey dependency changes to Gaffer should get the tests working again.
@GCHQDeveloper314 GCHQDeveloper314 added this to the v2.0.0-alpha-0.3 milestone Jul 13, 2022
@GCHQDeveloper314 GCHQDeveloper314 self-assigned this Jul 13, 2022
@github-actions github-actions bot mentioned this issue Jul 13, 2022
Merged
@GCHQDeveloper314 GCHQDeveloper314 linked a pull request Jul 13, 2022 that will close this issue
Gh-1008: Fix Selenium UI tests to work with new jersey #1009
Merged
t92549 pushed a commit that referenced this issue Jul 13, 2022
@GCHQDeveloper314
Gh-1008: Fix Selenium UI tests to work with new jersey (#1009)
f1d59f3 
* Allow use of newer Guava
By requiring the version be at least 25

* Support newer jersey
Gaffer now uses 2.36 which requires an extra dependency
@t92549
Copy link
Contributor

t92549 commented Jul 13, 2022 •
edited
Loading

Closed by #1009

@t92549 t92549 closed this as completed Jul 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@GCHQDeveloper314 GCHQDeveloper314

Labels
dependencies enhancement
Projects
None yet
Milestone
v2.0.0-alpha-0.3
Development

Successfully merging a pull request may close this issue.

Gh-1008: Fix Selenium UI tests to work with new jersey gchq/gaffer-tools
2 participants
@t92549 @GCHQDeveloper314