Skip to content

Revert "fix amqplib upstream tests failing due to missing tag for release"#7174

Merged
BridgeAR merged 2 commits intomasterfrom
revert-2098-fix-amqplib-upstream-test-error
Jan 5, 2026
Merged

Revert "fix amqplib upstream tests failing due to missing tag for release"#7174
BridgeAR merged 2 commits intomasterfrom
revert-2098-fix-amqplib-upstream-test-error

Conversation

@BridgeAR
Copy link
Copy Markdown
Member

@BridgeAR BridgeAR commented Jan 5, 2026

Reverts #2098

I just noticed this and this should be resolved by now.

@BridgeAR BridgeAR requested review from a team as code owners January 5, 2026 16:56
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 5, 2026

Overall package size

Self size: 4.39 MB
Deduped: 5.21 MB
No deduping: 5.21 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.0 | 68.46 kB | 797.03 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.49%. Comparing base (5c1e8c5) to head (4c8d1de).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7174   +/-   ##
=======================================
  Coverage   84.49%   84.49%           
=======================================
  Files         524      524           
  Lines       22463    22463           
=======================================
  Hits        18981    18981           
  Misses       3482     3482           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@datadog-official
Copy link
Copy Markdown

datadog-official Bot commented Jan 5, 2026

⚠️ Tests

Fix all issues with Cursor

⚠️ Warnings

❄️ 2 New flaky tests detected

cypress@latest commonJS test management attempt to fix can attempt to fix and mark last attempt as failed if every attempt fails from integration-tests/cypress/cypress.spec.js (Datadog) (Fix with Cursor)
Cannot read properties of undefined (reading 'content')
tests.test_resource_renaming.Test_Resource_Renaming_Stats_Aggregation_Keys.test_stats_aggregation_with_method_and_endpoint[express5] from system_tests_suite (Datadog) (Fix with Cursor)
AssertionError: Expected 5 hits for GET /resource_renaming/api/users/{param:int}, got 4
assert 4 == 5

self = <tests.test_resource_renaming.Test_Resource_Renaming_Stats_Aggregation_Keys object at 0x7fd554cfaf60>

    @bug(library="python", reason="APMSP-2359")  # trace exporter uses a wrong fieldname
    def test_stats_aggregation_with_method_and_endpoint(self):
        """Test that stats are aggregated by method and endpoint"""
        stats_points = []
        for stats_data in interfaces.library.get_data("/v0.6/stats"):
...

ℹ️ Info

🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 4c8d1de | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@pr-commenter
Copy link
Copy Markdown

pr-commenter Bot commented Jan 5, 2026

Benchmarks

Benchmark execution time: 2026-01-05 19:29:37

Comparing candidate commit 4c8d1de in PR branch revert-2098-fix-amqplib-upstream-test-error with baseline commit 5c1e8c5 in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 288 metrics, 32 unstable metrics.

@BridgeAR BridgeAR merged commit c715142 into master Jan 5, 2026
1067 of 1070 checks passed
@BridgeAR BridgeAR deleted the revert-2098-fix-amqplib-upstream-test-error branch January 5, 2026 21:27
@dd-octo-sts dd-octo-sts Bot mentioned this pull request Jan 12, 2026
dd-octo-sts Bot pushed a commit that referenced this pull request Jan 12, 2026
nina9753 pushed a commit that referenced this pull request Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants