Can't push apps with app manifests if routes were shared

Move app and route space match to common method

Author: nookala

app/actions/manifest_route_update.rb

@@ -92,10 +92,8 @@ def find_or_create_valid_route(app, manifest_route, user_audit_info)
               domain: existing_domain,
               manifest_triggered: true
             )
-          elsif route.space.guid != app.space_guid
-            # check if route is shared with space
-            spaces = route.shared_spaces
-            raise InvalidRoute.new('Routes cannot be mapped to destinations in different spaces') if spaces.blank? || spaces.none? { |space| space.values[:id] == app.space.id }
+          elsif route.app_spaces_no_match?(app)
+            raise InvalidRoute.new('Routes cannot be mapped to destinations in different spaces')
           elsif manifest_route[:options] && route[:options] != manifest_route[:options]
             # remove nil values from options
             manifest_route[:options] = manifest_route[:options].compact

app/controllers/v3/routes_controller.rb

@@ -362,7 +362,7 @@ def validate_app_guids!(apps_hash, desired_app_guids)
   end
 
   def validate_app_spaces!(apps_hash, route)
-    return unless apps_hash.values.any? { |app| app.space != route.space && route.shared_spaces.exclude?(app.space) }
+    return unless apps_hash.values.any? { |app| route.app_spaces_no_match?(app) }
 
     unprocessable!("Routes destinations must be in either the route's space or the route's shared spaces")
   end

app/models/runtime/route.rb

@@ -194,6 +194,10 @@ def self.user_visibility_filter(user)
       }
     end
 
+    def app_spaces_no_match?(app)
+      app.space != space && shared_spaces.exclude?(app.space) 
+    end
+
     delegate :in_suspended_org?, to: :space
 
     def tcp?

spec/unit/models/runtime/route_spec.rb

@@ -1570,5 +1570,35 @@ def assert_invalid_path(path)
         end
       end
     end
+
+    describe 'app spaces and route shared spaces' do
+      let!(:domain) { SharedDomain.make }
+
+      context 'when app and route space  not shared' do
+        let!(:app) { AppModel.make }
+        let!(:route) { Route.make(host: 'potato', domain: domain, path: '/some-path') }
+
+        it 'no space match and not shared and returns true' do
+          expect(route.app_spaces_no_match?(app)).to be(true)
+        end
+
+        it 'match space and returns false' do
+          route.space = app.space
+          expect(route.app_spaces_no_match?(app)).to be(false)
+        end
+      end
+
+      context 'when app and route space shared' do
+        let!(:app) { AppModel.make }
+        let!(:route_share) { RouteShare.new }
+        let(:user_audit_info) { instance_double(UserAuditInfo).as_null_object }
+        let!(:route) { Route.make(host: 'potato', domain: domain, path: '/some-path') }
+        let!(:shared_route) { route_share.create(route, [app.space], user_audit_info) }
+
+        it 'shared space match and returns false' do
+          expect(route.app_spaces_no_match?(app)).to be(false)
+        end
+      end
+    end
   end
 end