Adjust fixtures to cover the fix that we are doing

filiphr · Jul 6, 2024 · 08d4e8e · 08d4e8e
1 parent 8bfeed0
commit 08d4e8e
Show file tree

Hide file tree

Showing 8 changed files with 16 additions and 16 deletions.
diff --git a/...org/mapstruct/ap/test/bugs/_3601/StringMultiConditionListPropertyConditionMapperImpl.java b/...org/mapstruct/ap/test/bugs/_3601/StringMultiConditionListPropertyConditionMapperImpl.java
@@ -29,8 +29,8 @@ public Target map(Source source, List<String> sourceIds) {
  target.currentId = source.getUuid();
  }
  }
- if ( isNotEmpty( sourceIds ) ) {
-  List<String> list = sourceIds;
+ List<String> list = sourceIds;
+ if ( list != null ) {
  target.targetIds = new ArrayList<String>( list );
  }
 

diff --git a/...struct/ap/test/bugs/_3601/StringMultiConditionListSourceParameterConditionMapperImpl.java b/...struct/ap/test/bugs/_3601/StringMultiConditionListSourceParameterConditionMapperImpl.java
@@ -29,8 +29,8 @@ public Target map(Source source, List<String> sourceIds) {
  target.currentId = source.getUuid();
  }
  }
- List<String> list = sourceIds;
- if ( list != null ) {
+ if ( isNotEmpty( sourceIds ) ) {
+  List<String> list = sourceIds;
  target.targetIds = new ArrayList<String>( list );
  }
 

diff --git a/.../mapstruct/ap/test/bugs/_3601/StringPropertyConditionListPropertyConditionMapperImpl.java b/.../mapstruct/ap/test/bugs/_3601/StringPropertyConditionListPropertyConditionMapperImpl.java
@@ -29,8 +29,8 @@ public Target map(Source source, List<String> sourceIds) {
  target.currentId = source.getUuid();
  }
  }
- if ( isNotEmpty( sourceIds ) ) {
-  List<String> list = sourceIds;
+ List<String> list = sourceIds;
+ if ( list != null ) {
  target.targetIds = new ArrayList<String>( list );
  }
 

diff --git a/...uct/ap/test/bugs/_3601/StringPropertyConditionListSourceParameterConditionMapperImpl.java b/...uct/ap/test/bugs/_3601/StringPropertyConditionListSourceParameterConditionMapperImpl.java
@@ -29,8 +29,8 @@ public Target map(Source source, List<String> sourceIds) {
  target.currentId = source.getUuid();
  }
  }
- List<String> list = sourceIds;
- if ( list != null ) {
+ if ( isNotEmpty( sourceIds ) ) {
+  List<String> list = sourceIds;
  target.targetIds = new ArrayList<String>( list );
  }
 

diff --git a/...uct/ap/test/bugs/_3601/StringSourceParameterConditionListPropertyConditionMapperImpl.java b/...uct/ap/test/bugs/_3601/StringSourceParameterConditionListPropertyConditionMapperImpl.java
@@ -29,8 +29,8 @@ public Target map(Source source, List<String> sourceIds) {
  target.currentId = source.getUuid();
  }
  }
- if ( isNotEmpty( sourceIds ) ) {
-  List<String> list = sourceIds;
+ List<String> list = sourceIds;
+ if ( list != null ) {
  target.targetIds = new ArrayList<String>( list );
  }
 

diff --git a/...test/bugs/_3601/StringSourceParameterConditionListSourceParameterConditionMapperImpl.java b/...test/bugs/_3601/StringSourceParameterConditionListSourceParameterConditionMapperImpl.java
@@ -27,8 +27,8 @@ public Target map(Source source, List<String> sourceIds) {
  if ( source != null ) {
  target.currentId = source.getUuid();
  }
- List<String> list = sourceIds;
- if ( list != null ) {
+ if ( isNotEmpty( sourceIds ) ) {
+  List<String> list = sourceIds;
  target.targetIds = new ArrayList<String>( list );
  }
 

diff --git a/...fixtures/org/mapstruct/ap/test/bugs/_3601/single/OnlyListPropertyConditionMapperImpl.java b/...fixtures/org/mapstruct/ap/test/bugs/_3601/single/OnlyListPropertyConditionMapperImpl.java
@@ -31,8 +31,8 @@ public Target map(Source source, List<String> sourceIds) {
  target.currentId = source.getUuid();
  }
  }
- if ( isNotEmpty( sourceIds ) ) {
-  List<String> list = sourceIds;
+ List<String> list = sourceIds;
+ if ( list != null ) {
  target.targetIds = new ArrayList<String>( list );
  }
 

diff --git a/...s/org/mapstruct/ap/test/bugs/_3601/single/OnlyListSourceParameterConditionMapperImpl.java b/...s/org/mapstruct/ap/test/bugs/_3601/single/OnlyListSourceParameterConditionMapperImpl.java
@@ -29,8 +29,8 @@ public Target map(Source source, List<String> sourceIds) {
  if ( source != null ) {
  target.currentId = source.getUuid();
  }
- List<String> list = sourceIds;
- if ( list != null ) {
+ if ( isNotEmpty( sourceIds ) ) {
+  List<String> list = sourceIds;
  target.targetIds = new ArrayList<String>( list );
  }