Skip to content

Conversation

sentrivana
Copy link
Contributor

@sentrivana sentrivana commented Sep 2, 2025

  • move the langchain test suites to be governed by toxgen
    • this indirectly results in removing the -latest tests in the AI group 🎉
      • -latest tests predate toxgen and langchain was the last non-toxgen test suite (the rest was just skipped) -- all AI tests are now pinned
  • updated the naming scheme to use dashes instead of underscores for variants so that it's clearer if something is part of the name of the integration or if it denotes a variant
    • for instance, openai-base means this is the base variant of the openai test suite, but openai_agents means this is the openai_agents test suite (no variant)

I'm explicitly ignoring the two alpha versions of 1.0 since adapting the integration to work with those is out of scope: dedicated issue

Part of #4506

Copy link

codecov bot commented Sep 2, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
23940 1 23939 2291
View the top 1 failed test(s) by shortest run time
tests.integrations.wsgi.test_wsgi::test_auto_session_tracking_with_aggregates
Stack Traces | 0.095s run time
.../integrations/wsgi/test_wsgi.py:430: in test_auto_session_tracking_with_aggregates
    assert session_aggregates[0]["crashed"] == 1
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
E   KeyError: 'crashed'

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@sentrivana sentrivana marked this pull request as ready for review September 2, 2025 09:25
@sentrivana sentrivana requested a review from a team as a code owner September 2, 2025 09:25
cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@sentrivana sentrivana marked this pull request as draft September 2, 2025 09:42
cursor[bot]

This comment was marked as outdated.

@sentrivana sentrivana marked this pull request as ready for review September 2, 2025 11:13
@sentrivana sentrivana merged commit 65755f9 into master Sep 2, 2025
139 of 141 checks passed
@sentrivana sentrivana deleted the ivana/toxgen/move-langchain branch September 2, 2025 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants