-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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.
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
[PDI-17274] during metadata injection of constant values allocate spa… #6315
Conversation
Build Completed❌ This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues. Build Commandsmvn -B -f 'pom.xml' -pl 'engine,plugins/meta-inject' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install Cleanup Commandsmvn -B -f 'pom.xml' -pl 'engine,plugins/meta-inject' -P '!assemblies' -amd build-helper:remove-project-artifact Changed files engine/src/main/java/org/pentaho/di/core/injection/bean/BeanInjectionInfo.java
engine/src/main/java/org/pentaho/di/core/injection/bean/BeanInjector.java
engine/src/main/java/org/pentaho/di/core/injection/bean/BeanLevelInfo.java
engine/src/test/java/org/pentaho/di/core/injection/MetaBeanLevel1.java
engine/src/test/java/org/pentaho/di/core/injection/MetaBeanLevel2.java
engine/src/test/java/org/pentaho/di/core/injection/bean/BeanInjectorTest.java
plugins/meta-inject/src/main/java/org/pentaho/di/trans/steps/metainject/MetaInject.java Newly Broken Tests:org.pentaho.di.core.lifecycle.KettleLifecycleSupportTest.testOnEnvironmentInit:🔴 org.mockito.exceptions.misusing.UnfinishedStubbingException:
Unfinished stubbing detected here:
-> at org.pentaho.di.core.injection.bean.BeanInjectorTest.getCollectionSize_BeanLevelInfo_Exception(BeanInjectorTest.java:350)
E.g. thenReturn() may be missing.
Examples of correct stubbing:
when(mock.isOk()).thenReturn(true);
when(mock.isOk()).thenThrow(exception);
doThrow(exception).when(mock).someVoidMethod();
Hints:
1. missing thenReturn()
2. you are trying to stub a final method, Unit Test CoverageThese statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
|
14839f9
to
e35bfae
Compare
Build Completed❌ This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues. Build Commandsmvn -B -f 'pom.xml' -pl 'engine,plugins/meta-inject' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install Cleanup Commandsmvn -B -f 'pom.xml' -pl 'engine,plugins/meta-inject' -P '!assemblies' -amd build-helper:remove-project-artifact Changed files engine/src/main/java/org/pentaho/di/core/injection/bean/BeanInjectionInfo.java
engine/src/main/java/org/pentaho/di/core/injection/bean/BeanInjector.java
engine/src/main/java/org/pentaho/di/core/injection/bean/BeanLevelInfo.java
engine/src/test/java/org/pentaho/di/core/injection/MetaBeanLevel1.java
engine/src/test/java/org/pentaho/di/core/injection/MetaBeanLevel2.java
engine/src/test/java/org/pentaho/di/core/injection/bean/BeanInjectorTest.java
plugins/meta-inject/src/main/java/org/pentaho/di/trans/steps/metainject/MetaInject.java Newly Broken Tests:org.pentaho.di.core.lifecycle.KettleLifecycleSupportTest.testOnEnvironmentInit:🔴 org.mockito.exceptions.misusing.UnfinishedStubbingException:
Unfinished stubbing detected here:
-> at org.pentaho.di.core.injection.bean.BeanInjectorTest.getCollectionSize_BeanLevelInfo_Exception(BeanInjectorTest.java:373)
E.g. thenReturn() may be missing.
Examples of correct stubbing:
when(mock.isOk()).thenReturn(true);
when(mock.isOk()).thenThrow(exception);
doThrow(exception).when(mock).someVoidMethod();
Hints:
1. missing thenReturn()
2. you are trying to stub a final method, Unit Test CoverageThese statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
|
e35bfae
to
7e7f27a
Compare
Build Completed❌ This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues. Build Commandsmvn -B -f 'pom.xml' -pl 'engine,plugins/meta-inject' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install Cleanup Commandsmvn -B -f 'pom.xml' -pl 'engine,plugins/meta-inject' -P '!assemblies' -amd build-helper:remove-project-artifact Changed files engine/src/main/java/org/pentaho/di/core/injection/bean/BeanInjectionInfo.java
engine/src/main/java/org/pentaho/di/core/injection/bean/BeanInjector.java
engine/src/main/java/org/pentaho/di/core/injection/bean/BeanLevelInfo.java
engine/src/test/java/org/pentaho/di/core/injection/MetaBeanLevel1.java
engine/src/test/java/org/pentaho/di/core/injection/MetaBeanLevel2.java
engine/src/test/java/org/pentaho/di/core/injection/bean/BeanInjectorTest.java
plugins/meta-inject/src/main/java/org/pentaho/di/trans/steps/metainject/MetaInject.java Newly Broken Tests:org.pentaho.di.core.lifecycle.KettleLifecycleSupportTest.testOnEnvironmentInit:🔴 org.mockito.exceptions.misusing.UnfinishedStubbingException:
Unfinished stubbing detected here:
-> at org.pentaho.di.core.injection.bean.BeanInjectorTest.getCollectionSize_BeanLevelInfo_Exception(BeanInjectorTest.java:373)
E.g. thenReturn() may be missing.
Examples of correct stubbing:
when(mock.isOk()).thenReturn(true);
when(mock.isOk()).thenThrow(exception);
doThrow(exception).when(mock).someVoidMethod();
Hints:
1. missing thenReturn()
2. you are trying to stub a final method, Unit Test CoverageThese statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
|
…ce for collections - update BeanInjector.setProperty(...) to include allocateCollectionField(...)
7e7f27a
to
113e409
Compare
Build Completed🔥 This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues. Build Commandsmvn -B -f 'pom.xml' -pl 'engine,plugins/meta-inject' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install Cleanup Commandsmvn -B -f 'pom.xml' -pl 'engine,plugins/meta-inject' -P '!assemblies' -amd build-helper:remove-project-artifact Changed files engine/src/main/java/org/pentaho/di/core/injection/bean/BeanInjectionInfo.java
engine/src/main/java/org/pentaho/di/core/injection/bean/BeanInjector.java
engine/src/main/java/org/pentaho/di/core/injection/bean/BeanLevelInfo.java
engine/src/test/java/org/pentaho/di/core/injection/MetaBeanLevel1.java
engine/src/test/java/org/pentaho/di/core/injection/MetaBeanLevel2.java
engine/src/test/java/org/pentaho/di/core/injection/bean/BeanInjectorTest.java
plugins/meta-inject/src/main/java/org/pentaho/di/trans/steps/metainject/MetaInject.java Unit Test CoverageThese statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
|
@pentaho/r2d2 please review |
…ce for collections (pentaho#6315) - update BeanInjector.setProperty(...) to include allocateCollectionField(...)
…ce for collections (pentaho#6315) - update BeanInjector.setProperty(...) to include allocateCollectionField(...)
…ce for collections (pentaho#6315) - update BeanInjector.setProperty(...) to include allocateCollectionField(...)
…ce for collections