github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-01-18 21:02:48 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

DEV: Fix settledState debug message (#24544)

Browse Source 
A properly formatted string instead of circular-JSON error
This commit is contained in:
Jarek Radosz 2023-11-24 12:18:28 +01:00 committed by GitHub
parent 2befff5101
commit 1f2a25331b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
app/assets/javascripts/discourse/tests/setup-tests.js
View File

@ -419,10 +419,19 @@ function patchFailedAssertion() {
QUnit.assert.pushResult = function (resultInfo) {
if (!resultInfo.result && !isSettled()) {
const settledState = getSettledState();
let stateString = Object.entries(settledState)
.filter(([, value]) => value === true)
.map(([key]) => key)
.join(", ");
if (settledState.pendingRequestCount > 0) {
stateString += `, pending requests: ${settledState.pendingRequestCount}`;
}
// eslint-disable-next-line no-console
console.warn(
"ℹ️ Hint: when the assertion failed, the Ember runloop was not in a settled state. Maybe you missed an `await` further up the test? Or maybe you need to manually add `await settled()` before your assertion?",
JSON.stringify(getSettledState())
`ℹ️ Hint: when the assertion failed, the Ember runloop was not in a settled state. Maybe you missed an \`await\` further up the test? Or maybe you need to manually add \`await settled()\` before your assertion? (${stateString})`
);
}