Skip to content

Commit

Permalink
Fix Ruby test skip declaration (#3904)
Browse files Browse the repository at this point in the history
  • Loading branch information
marcotc authored Feb 3, 2025
1 parent 239c3eb commit 2377809
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/run-end-to-end.yml
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ jobs:
if: always() && steps.build.outcome == 'success' && contains(inputs.scenarios, '"REMOTE_CONFIG_MOCKED_BACKEND_ASM_DD_NOCACHE"')
run: ./run.sh REMOTE_CONFIG_MOCKED_BACKEND_ASM_DD_NOCACHE
- name: Run AGENT_NOT_SUPPORTING_SPAN_EVENTS scenario
if: always() && steps.build.outcome == 'success' && contains(inputs.scenarios, 'AGENT_NOT_SUPPORTING_SPAN_EVENTS') && (inputs.library != 'ruby' || matrix.weblog == 'rack')
if: always() && steps.build.outcome == 'success' && contains(inputs.scenarios, 'AGENT_NOT_SUPPORTING_SPAN_EVENTS')
run: ./run.sh AGENT_NOT_SUPPORTING_SPAN_EVENTS
- name: Run APPSEC_MISSING_RULES scenario
# C++ 1.2.0 freeze when the rules file is missing
Expand Down
7 changes: 7 additions & 0 deletions manifests/ruby.yml
Original file line number Diff line number Diff line change
Expand Up @@ -554,6 +554,13 @@ tests/:
Test_Meta: v1.7.0
Test_MetaDatadogTags: v1.9.0
Test_MetricsStandardTags: v1.7.0
test_span_events.py:
Test_SpanEvents_WithAgentSupport:
"*": irrelevant
rack: v2.10.0
Test_SpanEvents_WithoutAgentSupport:
"*": irrelevant
rack: v2.10.0
test_standard_tags.py:
Test_StandardTagsClientIp: v1.10.1
Test_StandardTagsMethod: v1.8.0
Expand Down
1 change: 0 additions & 1 deletion tests/test_span_events.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ class Test_SpanEvents_WithAgentSupport:
def setup_v04_v07_default_format(self):
self.r = weblog.get("/add_event")

@missing_feature(context.library in ["ruby"], reason="Native serialization not implemented")
def test_v04_v07_default_format(self):
"""For traces that default to the v0.4 or v0.7 format, send events as a top-level `span_events` field"""
interfaces.library.assert_trace_exists(self.r)
Expand Down

0 comments on commit 2377809

Please sign in to comment.