Skip to content

chore: fix dev e2e tests for policy violation#589

Merged
milan-cb merged 1 commit intomainfrom
milan/CDPSDK-2166
Feb 6, 2026
Merged

chore: fix dev e2e tests for policy violation#589
milan-cb merged 1 commit intomainfrom
milan/CDPSDK-2166

Conversation

@milan-cb
Copy link
Contributor

@milan-cb milan-cb commented Feb 6, 2026

Description

This PR edits the EVM address for specific policy tests to ensure the correct error is thrown. Without this change, there is a different error for a sanctioned address being thrown which conflicts with the error matching schema, causing E2E tests to fail.

Tests

Checklist

A couple of things to include in your PR for completeness:

  • Updated the typescript README if relevant
  • Updated the python README if relevant
  • Added a changelog entry
  • Added e2e tests if introducing new functionality

@linear
Copy link

linear bot commented Feb 6, 2026

@cb-heimdall
Copy link

cb-heimdall commented Feb 6, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 2/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@milan-cb milan-cb requested a review from avidreder February 6, 2026 23:54
Copy link
Contributor

@avidreder avidreder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@milan-cb milan-cb merged commit 39dd852 into main Feb 6, 2026
39 checks passed
@milan-cb milan-cb deleted the milan/CDPSDK-2166 branch February 6, 2026 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants