Skip to content

Fix data_category for sessions envelope items#3473

Merged
sl0thentr0py merged 1 commit into
masterfrom
neel/fix-sessions-data-category
Aug 29, 2024
Merged

Fix data_category for sessions envelope items#3473
sl0thentr0py merged 1 commit into
masterfrom
neel/fix-sessions-data-category

Conversation

@sl0thentr0py

@sl0thentr0py sl0thentr0py commented Aug 29, 2024

Copy link
Copy Markdown
Member

noticed this while testing potel

@sl0thentr0py
sl0thentr0py merged commit 1541240 into master Aug 29, 2024
@sl0thentr0py
sl0thentr0py deleted the neel/fix-sessions-data-category branch August 29, 2024 13:06
@codecov

codecov Bot commented Aug 29, 2024

Copy link
Copy Markdown

❌ 4 Tests Failed:

Tests completed Failed Passed Skipped
11522 4 11518 1635
View the top 3 failed tests by shortest run time 
tests.integrations.aws_lambda.test_aws test_init_error[python3.10]
Stack Traces | 10.6s run time 
.../integrations/aws_lambda/test_aws.py:334: in test_init_error
    (event,) = envelope_items
E   ValueError: not enough values to unpack (expected 1, got 0)

tests.integrations.aws_lambda.test_aws test_init_error[python3.11]
Stack Traces | 10.7s run time 
.../integrations/aws_lambda/test_aws.py:334: in test_init_error
    (event,) = envelope_items
E   ValueError: not enough values to unpack (expected 1, got 0)

tests.integrations.aws_lambda.test_aws test_init_error[python3.9]
Stack Traces | 10.8s run time 
.../integrations/aws_lambda/test_aws.py:334: in test_init_error
    (event,) = envelope_items
E   ValueError: not enough values to unpack (expected 1, got 0)

To view individual test run time comparison to the main branch, go to the Test Analytics Dashboard

arjennienhuis pushed a commit to arjennienhuis/sentry-python that referenced this pull request Sep 30, 2024
@sl0thentr0py @arjenzorgdoc
Fix data_category for sessions envelope items (getsentry#3473)
aff8872
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sentrivana sentrivana sentrivana approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sl0thentr0py @sentrivana