Skip to content

Feat/enable CI for local main branch#140

Merged
pieterlukasse merged 3 commits into
2.15.0-DEVfrom
feat/enable_ci_for_local_main_branch
Jun 12, 2024
Merged

Feat/enable CI for local main branch#140
pieterlukasse merged 3 commits into
2.15.0-DEVfrom
feat/enable_ci_for_local_main_branch

Conversation

@pieterlukasse

Copy link
Copy Markdown

Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/VADC-1102

Improvements

  • unit tests are reinstated in our local fork, following acceptance of the same upstream (see OHDSI@7898f7e)

@pieterlukasse pieterlukasse force-pushed the feat/enable_ci_for_local_main_branch branch from 75a31b2 to b923793 Compare June 11, 2024 19:50
@pieterlukasse pieterlukasse force-pushed the feat/enable_ci_for_local_main_branch branch from b923793 to f01e0f0 Compare June 11, 2024 19:56
@pieterlukasse

pieterlukasse commented Jun 11, 2024

Copy link
Copy Markdown
Author

hm...seems most tests are now actually failing with

 java.lang.BootstrapMethodError: java.lang.NoSuchMethodError: org.apache.commons.io.IOUtils.byteArray()[B
Caused by: java.lang.NoSuchMethodError: org.apache.commons.io.IOUtils.byteArray()[B

likely because of some dependency update, maybe here 06067d5

@m0nhawk

m0nhawk commented Jun 11, 2024

Copy link
Copy Markdown

I think it's because of this:

WebAPI/pom.xml

Lines 832 to 834 in 06067d5

<groupId>commons-io</groupId>
<artifactId>commons-io</artifactId>
<version>2.7</version>

And this was introduced in 2.9.0: https://javadoc.io/doc/commons-io/commons-io/2.9.0/org/apache/commons/io/IOUtils.html#byteArray--

@pieterlukasse

Copy link
Copy Markdown
Author

@m0nhawk nice catch, although this line has been there for the last 3 years.... so I still think it is a side effect of another update, since the tests pass on upstream master.
Anyway, I can try to update to 2.9.0 and see what happens...

@pieterlukasse pieterlukasse force-pushed the feat/enable_ci_for_local_main_branch branch from b5261b3 to c8c7e83 Compare June 12, 2024 09:59
@pieterlukasse

pieterlukasse commented Jun 12, 2024

Copy link
Copy Markdown
Author

done...now I get another error:

java.lang.NoSuchMethodError: org.apache.commons.io.IOUtils.skip(Ljava/io/InputStream;JLjava/util/function/Supplier;)J

looks like we need 2.14.0

@pieterlukasse pieterlukasse force-pushed the feat/enable_ci_for_local_main_branch branch from 99962b3 to a43248a Compare June 12, 2024 17:20

@m0nhawk m0nhawk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

@pieterlukasse pieterlukasse merged commit fbebf9a into 2.15.0-DEV Jun 12, 2024
@pieterlukasse pieterlukasse deleted the feat/enable_ci_for_local_main_branch branch June 12, 2024 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants