Skip to content

Conversation

@mwooten
Copy link
Contributor

@mwooten mwooten commented Apr 4, 2018

Hi Carolyn,

Can you review/approve the changes to two files I had to change in weblogic-deploy-tooling, because I need to use code from weblogic-deploy-tooling in the new weblogic-deploy-testing project I'm working on?

The details are in the comments of Issue# 41 and Issue # 42

@mwooten mwooten requested a review from CarolynRountree April 4, 2018 19:45
WLSDPLY-20020=Filter ID {0} is invalid
WLSDPLY-20021=Filter path {0} does not exist
WLSDPLY-20022=Error loading filter path {0}

Copy link
Member

Choose a reason for hiding this comment

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

Why are integration test messages in WDT? Shouldn't these be in the integration test project?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It looks like the stuff we found with ExceptionHelper.java (and exception_helper.py) using a static resource bundle name, is the reason I was getting the MissingResourceException. I will find a way to not use the get_message() function in either of those, in the weblogic-deploy-testing project.

The changes made to wlsdeploy_rb.properties can be reverted, but I still need the change to FileUtils.java to be reviewed and merged.

@ddsharpe ddsharpe merged commit 235fe74 into master Apr 10, 2018
@ddsharpe ddsharpe deleted the Issue#-41,42-Additions-to-facilitate-the-weblogic-deploy-testing-project branch April 10, 2018 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants