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

Add CDI integration test #317

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add CDI integration test #317

wants to merge 5 commits into from

Conversation

ajm01
Copy link
Member

@ajm01 ajm01 commented May 7, 2019

Introduce a new IT test suite for the cdi booster - and also covers jsonP as well?

This is an open liberty guide specific to cdi and jsonp and is EE based (no microprofile)

@ajm01 ajm01 requested review from scottkurz and awisniew90 May 7, 2019 16:02
@ajm01 ajm01 force-pushed the cdiIT branch 3 times, most recently from 08bdefc to 61b9eb8 Compare May 7, 2019 21:11
@scottkurz scottkurz changed the title Cdi it Add CDI / JSON-P integration tests May 7, 2019
@scottkurz scottkurz changed the base branch from master to jvm-prop May 7, 2019 21:48
@scottkurz scottkurz changed the base branch from jvm-prop to master May 7, 2019 21:48
@ajm01 ajm01 force-pushed the cdiIT branch 2 times, most recently from a4b3d99 to 8306ca1 Compare May 8, 2019 17:22
@scottkurz scottkurz changed the title Add CDI / JSON-P integration tests Add integration test May 10, 2019
@scottkurz scottkurz changed the title Add integration test Add CDI integration test May 10, 2019
<artifactId>cdi</artifactId>
<version>0.2-SNAPSHOT</version>
</dependency>
<dependency>
<groupId>io.openliberty.boosters</groupId>
<groupId>boost.boosters</groupId>
<artifactId>jsonp</artifactId>
<version>0.2-SNAPSHOT</version>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'jsonp' dependency isn't really doing anything here, at least not in Liberty (probably not in TomEE either). Maybe slightly confusing to leave it in not a huge deal though.

We should start looking at using 'jsonp' together with a JSON provider dependency to use a non-default provider, as a totally separate issue though.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants