remove NPE preventing check that should be not needed (#7386)

Author: MBoegers

rewrite-maven/src/main/java/org/openrewrite/maven/ChangeDependencyGroupIdAndArtifactId.java

@@ -201,10 +201,6 @@ private Set<String> getSafeVersionPlaceholdersToChange(String groupId, String ar
                 MavenResolutionResult result = getResolutionResult();
                 ResolvedPom resolvedPom = result.getPom();
                 Pom requestedPom = resolvedPom.getRequested();
-                // Pom fields default to emptyList() via @Builder.Default, but deserialization can leave them null
-                if (requestedPom.getDependencies() == null) {
-                    return Collections.emptySet();
-                }
                 Set<String> relevantProperties = requestedPom.getDependencies().stream()
                         .filter(d -> isProperty(d.getVersion()) &&
                                 matchesGlob(resolvedPom.getValue(d.getGroupId()), groupId) &&
@@ -429,13 +425,7 @@ private boolean checkIfNewDependencyPresent(@Nullable String groupId, @Nullable
 
             private boolean isDependencyManaged(Scope scope, String groupId, String artifactId) {
                 MavenResolutionResult result = getResolutionResult();
-
-                List<ResolvedManagedDependency> managedDependencies = result.getPom().getDependencyManagement();
-                if (managedDependencies == null) {
-                    return false;
-                }
-
-                for (ResolvedManagedDependency managedDependency : managedDependencies) {
+                for (ResolvedManagedDependency managedDependency : result.getPom().getDependencyManagement()) {
                     if (groupId.equals(managedDependency.getGroupId()) && artifactId.equals(managedDependency.getArtifactId())) {
                         return scope.isInClasspathOf(managedDependency.getScope());
                     }
@@ -447,12 +437,7 @@ private boolean canAffectManagedDependency(MavenResolutionResult result, Scope s
                 // We're only going to be able to effect managed dependencies that are either direct or are brought in as direct via a local parent
                 // `ChangeManagedDependencyGroupIdAndArtifactId` cannot manipulate BOM imported managed dependencies nor direct dependencies from remote parents
                 Pom requestedPom = result.getPom().getRequested();
-
-                List<ManagedDependency> managedDependencies = requestedPom.getDependencyManagement();
-                if (managedDependencies == null) {
-                    return false;
-                }
-                for (ManagedDependency requestedManagedDependency : managedDependencies) {
+                for (ManagedDependency requestedManagedDependency : requestedPom.getDependencyManagement()) {
                     if (matchesGlob(requestedManagedDependency.getGroupId(), groupId) && matchesGlob(requestedManagedDependency.getArtifactId(), artifactId)) {
                         if (requestedManagedDependency instanceof ManagedDependency.Defined) {
                             return scope.isInClasspathOf(Scope.fromName(((ManagedDependency.Defined) requestedManagedDependency).getScope()));
@@ -469,10 +454,6 @@ private Set<String> getSafeVersionPlaceholdersToChange(String groupId, String ar
                 MavenResolutionResult result = getResolutionResult();
                 ResolvedPom resolvedPom = result.getPom();
                 Pom requestedPom = resolvedPom.getRequested();
-                // Pom fields default to emptyList() via @Builder.Default, but deserialization can leave them null
-                if (requestedPom.getDependencies() == null) {
-                    return Collections.emptySet();
-                }
                 Set<String> relevantProperties = requestedPom.getDependencies().stream()
                         .filter(d -> isProperty(d.getVersion()) &&
                                 matchesGlob(resolvedPom.getValue(d.getGroupId()), groupId) &&