You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Postgres integration test suite 033 matches expected and actual tracking events. However, the experimental parser fires its tracking event non-deterministically. Commits 7a32129, and c43c5fa were supposed to fix this, yet circleci still sees inconsistent failures.
This set of tests may not actually be all that useful as is and we may want to rework how we test events going forward anyway.
The text was updated successfully, but these errors were encountered:
We can definitely do that! But, that will only address tests that directly test tracking events. There are still intermittent failures in other tests (example). This is because some tests bootstrap a (sometimes incomplete) project config and this doesn't always initialize the global current user. Since there are tracking events being sent non-deterministically when parsing model files, some tests are hitting this code path that expect an initialized global user! I tried to address this in #3604
Thanks @kwigley! Just unblocking ourselves here for now will be super helpful and then we can dive deep on what is actually going on with these tests and make them more stable.
Postgres integration test suite 033 matches expected and actual tracking events. However, the experimental parser fires its tracking event non-deterministically. Commits 7a32129, and c43c5fa were supposed to fix this, yet circleci still sees inconsistent failures.
This set of tests may not actually be all that useful as is and we may want to rework how we test events going forward anyway.
The text was updated successfully, but these errors were encountered: