From a86ef25afc7febcd647db39fe4538f82d0e7d4ea Mon Sep 17 00:00:00 2001
From: Ariel Valentin <arielvalentin@github.com>
Date: Fri, 22 Dec 2023 12:11:07 -0600
Subject: [PATCH] squash: Add appraisals for C Parser test

---
 instrumentation/graphql/Appraisals                       | 9 +++++----
 .../graphql/tracers/graphql_trace_test.rb                | 4 +++-
 .../graphql/tracers/graphql_tracer_test.rb               | 3 ++-
 instrumentation/graphql/test/test_helper.rb              | 9 +++++++++
 4 files changed, 19 insertions(+), 6 deletions(-)

diff --git a/instrumentation/graphql/Appraisals b/instrumentation/graphql/Appraisals
index 99c05f036..dd1f5ca23 100644
--- a/instrumentation/graphql/Appraisals
+++ b/instrumentation/graphql/Appraisals
@@ -24,10 +24,11 @@ appraise 'graphql-2.1' do
   gem 'graphql', '~> 2.1.8'
 end
 
-appraise 'graphql-2.2.0' do
-  gem 'graphql', '2.2.0'
+appraise 'graphql-c_parser-2.2.x' do
+  gem 'graphql', '~> 2.2.0'
+  gem 'graphql-c_parser', '~> 1.0.7'
 end
 
-appraise 'graphql-2.x' do
-  gem 'graphql', '>= 2.2.1', '< 3.0.0'
+appraise 'graphql-2.2.x' do
+  gem 'graphql', '~> 2.2.1', '< 3.0.0'
 end
diff --git a/instrumentation/graphql/test/instrumentation/graphql/tracers/graphql_trace_test.rb b/instrumentation/graphql/test/instrumentation/graphql/tracers/graphql_trace_test.rb
index 59de00473..4ae7fb15e 100644
--- a/instrumentation/graphql/test/instrumentation/graphql/tracers/graphql_trace_test.rb
+++ b/instrumentation/graphql/test/instrumentation/graphql/tracers/graphql_trace_test.rb
@@ -54,6 +54,8 @@
           'graphql.execute_multiplex'
         ]
 
+        expected_spans.delete('graphql.lex') unless trace_lex_supported?
+
         expected_result = {
           'simpleField' => 'Hello.',
           'resolvedField' => { 'originalValue' => 'testing=1', 'uppercasedValue' => 'TESTING=1' }
@@ -103,7 +105,7 @@
         it 'traces the provided schemas' do
           SomeOtherGraphQLAppSchema.execute('query SimpleQuery{ __typename }')
 
-          _(spans.size).must_equal(8)
+          _(spans.select { |s| s.name.start_with?('graphql.') }).wont_be(:empty?)
         end
 
         it 'does not trace all schemas' do
diff --git a/instrumentation/graphql/test/instrumentation/graphql/tracers/graphql_tracer_test.rb b/instrumentation/graphql/test/instrumentation/graphql/tracers/graphql_tracer_test.rb
index 35e5e4b0d..0813c22c0 100644
--- a/instrumentation/graphql/test/instrumentation/graphql/tracers/graphql_tracer_test.rb
+++ b/instrumentation/graphql/test/instrumentation/graphql/tracers/graphql_tracer_test.rb
@@ -53,6 +53,7 @@
           'graphql.execute_query_lazy',
           'graphql.execute_multiplex'
         ]
+        expected_spans.delete('graphql.lex') unless trace_lex_supported?
 
         expected_result = {
           'simpleField' => 'Hello.',
@@ -102,7 +103,7 @@
 
         it 'traces the provided schemas' do
           SomeOtherGraphQLAppSchema.execute('query SimpleQuery{ __typename }')
-          _(spans.size).must_equal(8)
+          _(spans.select { |s| s.name.start_with?('graphql.') }).wont_be(:empty?)
         end
 
         it 'does not trace all schemas' do
diff --git a/instrumentation/graphql/test/test_helper.rb b/instrumentation/graphql/test/test_helper.rb
index 75f349b28..b25005d16 100644
--- a/instrumentation/graphql/test/test_helper.rb
+++ b/instrumentation/graphql/test/test_helper.rb
@@ -125,3 +125,12 @@ def uses_platform_interfaces?
 def gem_version
   Gem::Version.new(GraphQL::VERSION)
 end
+
+# When tracing, is the parser expected to call `lex` before `parse`
+def trace_lex_supported?
+  return @trace_lex_supported if defined?(@trace_lex_supported)
+
+  # In GraphQL 2.2, the default parser was changed such that `lex` is no longer called
+  @trace_lex_supported = Gem::Requirement.new('< 2.2').satisfied_by?(Gem::Version.new(GraphQL::VERSION)) \
+    || (defined?(GraphQL::CParser) == 'constant')
+end