Remove unused vis code, don't freeze types

Author: rmosolgo

lib/graphql/schema/ractor_shareable.rb

@@ -38,9 +38,6 @@ def freeze_schema
           @frozen_directives = Ractor.make_shareable(directives)
 
           Ractor.make_shareable(visibility)
-          visibility.types.each do |_name, type_defn|
-            type_defn.freeze
-          end
           Ractor.make_shareable(introspection_system)
           extend(FrozenMethods)
 

lib/graphql/schema/visibility.rb

@@ -32,7 +32,6 @@ def initialize(schema, dynamic:, preload:, profiles:, migration_errors:)
           Visibility::Profile
         end
         @preload = preload
-        @preloaded = false
         @profiles = profiles
         @cached_profiles = {}
         @dynamic = dynamic
@@ -81,10 +80,6 @@ def preload?
       end
 
       def preload
-        if @preloaded
-          raise "Invariant: can't preload twice"
-        end
-        @preloaded = true
         # Traverse the schema now (and in the *_configured hooks below)
         # To make sure things are loaded during boot
         @preloaded_types = Set.new
@@ -215,7 +210,6 @@ def ensure_all_loaded(types_to_visit)
       end
 
       def load_all(types: nil)
-        return if frozen?
         if @visit.nil?
           # Set up the visit system
           @interface_type_memberships = Hash.new { |h, interface_type| h[interface_type] = [] }.compare_by_identity

spec/graphql/schema/ractor_shareable_spec.rb

@@ -69,7 +69,7 @@ class Query < GraphQL::Schema::Object
         query(q)
       end
 
-      assert_equal "Query", new_schema.execute("{ __typename }")["data"]["__typename"]
+      assert_equal "Query", new_schema.execute("{ __typename @include(if: true) }")["data"]["__typename"]
     end
   end
 end