Merge pull request #30 from haines/frozen-string-literals

Frozen string literals

Author: Nathaniel Woodthorpe

graphql-remote_loader.gemspec

@@ -1,4 +1,6 @@
 # coding: utf-8
+# frozen_string_literal: true
+
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'graphql/remote_loader/version'

lib/graphql/remote_loader.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "graphql"
 require "graphql/batch"
 

lib/graphql/remote_loader/loader.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "prime"
 require "json"
 require_relative "query_merger"
@@ -19,9 +21,7 @@ def self.load(query, context: {}, variables: {})
 
         store_context(context)
 
-        interpolate_variables!(query, variables)
-
-        self.for.load([query, prime, @context])
+        self.for.load([interpolate_variables(query, variables), prime, @context])
       end
 
       # Loads the value, then if the query was successful, fulfills promise with
@@ -97,6 +97,10 @@ def perform(queries_and_primes)
       # E.g.
       #   interpolate_variables("foo(bar: $my_var)", { my_var: "buzz" })
       #   => "foo(bar: \"buzz\")"
+      def self.interpolate_variables(query, variables = {})
+        query.dup.tap { |mutable_query| interpolate_variables!(mutable_query, variables) }
+      end
+
       def self.interpolate_variables!(query, variables = {})
         variables.each do |variable, value|
           case value

lib/graphql/remote_loader/query_merger.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module GraphQL
   module RemoteLoader
     # Given a list of queries and their prime UIDs, generate the merged and labeled

lib/graphql/remote_loader/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module GraphQL
   module RemoteLoader
     VERSION = "1.0.5"

spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bundler/setup'
 Bundler.setup
 

spec/unit/lib/graphql/remote_loader/loader_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 class TestLoader < GraphQL::RemoteLoader::Loader

spec/unit/lib/graphql/remote_loader/query_merger_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe GraphQL::RemoteLoader::QueryMerger do