Skip to content

test: add recordings for new import tests#157

Open
v-alexmoraru wants to merge 8 commits intomicrosoft:mainfrom
v-alexmoraru:dev/v-alexmoraru/record-new-tests-import
Open

test: add recordings for new import tests#157
v-alexmoraru wants to merge 8 commits intomicrosoft:mainfrom
v-alexmoraru:dev/v-alexmoraru/record-new-tests-import

Conversation

@v-alexmoraru
Copy link
Member

No description provided.

@v-alexmoraru v-alexmoraru marked this pull request as ready for review February 5, 2026 10:01
@v-alexmoraru v-alexmoraru requested a review from a team as a code owner February 5, 2026 10:01

def test_import_create_new_reflex_item_success(
# region Parametrized Tests
@import_update_existing_item_success_params
Copy link
Contributor

Choose a reason for hiding this comment

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

If this replaces all specific test per item, for example this replace test_import_update_existing_notebook_item_success, then we don't need the recording test_import_update_existing_notebook_item_success.yaml, right?
(I think the new recording is test_import_update_existing_item_success[Notebook].yaml)

Same for other items.

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.

3 participants

Comments