Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added failing test: "Fetching a belongsTo that is set to null removes the record from a relationship - sync - and do it twice" #9326

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

eodb
Copy link

@eodb eodb commented Apr 5, 2024

This is a PR to facilitate the fixing of #8990.
An extra test "Fetching a belongsTo that is set to null removes the record from a relationship - sync - and do it twice", has been added.

Description

It is actually the same test as the existing one that precedes it: "Fetching a belongsTo that is set to null removes the record from a relationship - sync" but only run twice immediately after each other.
If the user.accounts are not accessed in between (see the assert.strictEqual - put in comment), the test fails, otherwise the test succeeds.

…ord from a relationship - sync - and do it twice
@runspired runspired self-assigned this Apr 19, 2024
@runspired runspired added 🎯 canary PR is targeting canary (default) 🏷️ test This PR primarily adds tests for a feature 🏷️ bug This PR primarily fixes a reported issue labels Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯 canary PR is targeting canary (default) 🏷️ bug This PR primarily fixes a reported issue 🏷️ test This PR primarily adds tests for a feature
Projects
Status: stalled
Development

Successfully merging this pull request may close these issues.

2 participants