Skip to content

Commit 5eede97

Browse files
committed
fix: improve async tests stability and performance
1 parent 69dab48 commit 5eede97

File tree

91 files changed

+198
-173
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

91 files changed

+198
-173
lines changed
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
import { wait } from "../../utils/resolve";
22

3-
export const steps = [{}, wait(1)];
3+
export const steps = [{}, wait];

packages/runtime-tags/src/__tests__/fixtures/async-deep-recursive/__snapshots__/dom.expected/tags/recurse.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ const $await_content = /* @__PURE__ */_._await_content("#text/0", `<!>${$templat
1414
const $try_content__await_promise = /* @__PURE__ */_._await_promise("#text/0");
1515
const $try_content__setup = $scope => {
1616
$await_content($scope);
17-
$try_content__await_promise($scope, resolveAfter(0, 1));
17+
$try_content__await_promise($scope, resolveAfter(0));
1818
};
1919
const $if_content__input_level = /* @__PURE__ */_._if_closure("#text/0", 0, $scope => _._attr($scope["#div/0"], "data-level", $scope._.input_level));
2020
const $if_content__try = /* @__PURE__ */_._try("#text/1", "<!><!><!>", /* over(1), replace, over(2) */"b%c", $try_content__setup);

packages/runtime-tags/src/__tests__/fixtures/async-deep-recursive/__snapshots__/html.expected/tags/recurse.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ const $content = input => {
1111
_._try($scope1_id, "#text/1", _._content_resume("__tests__/tags/recurse.marko_2_content", () => {
1212
const $scope2_id = _._scope_id();
1313
const $scope2_reason = _._scope_reason();
14-
_._await($scope2_id, "#text/0", resolveAfter(0, 1), () => {
14+
_._await($scope2_id, "#text/0", resolveAfter(0), () => {
1515
const $scope3_id = _._scope_id();
1616
_._serialize_if($scope0_reason, /* input.level */0) && _._script($scope3_id, "__tests__/tags/recurse.marko_3_input_level");
1717
const $childScope = _._peek_scope_id();
@@ -23,12 +23,12 @@ const $content = input => {
2323
_: _._scope_with_id($scope2_id),
2424
"#childScope/0": _._serialize_if($scope0_reason, /* input.level */0) && _._existing_scope($childScope),
2525
"ClosureSignalIndex:input_level": _._serialize_if($scope0_reason, /* input.level */0) && 0
26-
}, "__tests__/tags/recurse.marko", "6:7");
26+
}, "__tests__/tags/recurse.marko", "7:7");
2727
_._resume_branch($scope3_id);
2828
}, _._serialize_guard($scope0_reason, /* input.level */0));
2929
_._serialize_if($scope0_reason, /* input.level */0) && _._scope($scope2_id, {
3030
_: _._scope_with_id($scope1_id)
31-
}, "__tests__/tags/recurse.marko", "4:5");
31+
}, "__tests__/tags/recurse.marko", "5:5");
3232
}, $scope1_id), {
3333
placeholder: _.attrTag({
3434
content: _._content_resume("__tests__/tags/recurse.marko_4_content", () => {
@@ -40,7 +40,7 @@ const $content = input => {
4040
_._html(`</div>${_._el_resume($scope1_id, "#div/0", _._serialize_guard($scope0_reason, /* input.level */0))}`);
4141
_._serialize_if($scope0_reason, /* input.level */0) && _._scope($scope1_id, {
4242
_: _._scope_with_id($scope0_id)
43-
}, "__tests__/tags/recurse.marko", "2:1");
43+
}, "__tests__/tags/recurse.marko", "3:1");
4444
return 0;
4545
}
4646
}, $scope0_id, "#text/0", _._serialize_guard($scope0_reason, /* input.level */0), _._serialize_guard($scope0_reason, /* input.level */0), _._serialize_guard($scope0_reason, /* input.level */0), 0, 1);
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
import { resolveAfter } from "../../../utils/resolve";
2+
23
if=input.level
34
div data-level=input.level
45
try
56
@placeholder -- LOADING...
6-
await=resolveAfter(0, 1)
7+
await=resolveAfter(0)
78
recurse level=input.level - 1
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
import { flush, wait } from "../../utils/resolve";
22

3-
export const steps = [{}, flush, flush, flush, flush, wait(5)];
3+
export const steps = [{}, flush, flush, flush, flush, wait];

packages/runtime-tags/src/__tests__/fixtures/async-multi-resolve-in-order-and-update/__snapshots__/dom.expected/template.hydrate.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
// size: 705 (min) 224 (brotli)
2-
const multiply = (multiplier, n) => resolveAfter(multiplier * n, n),
1+
// size: 703 (min) 222 (brotli)
2+
const multiply = (multiplier, n) => resolveAfter(multiplier * n),
33
$await_content5__result = _._const(2, ($scope) =>
44
_._text($scope.a, $scope.c),
55
),

packages/runtime-tags/src/__tests__/fixtures/async-multi-resolve-in-order-and-update/__snapshots__/dom.expected/template.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
export const $template = "<button>increment</button><p>1 * <!> = <!></p><p>2 * <!> = <!></p><p>3 * <!> = <!></p><p>4 * <!> = <!></p><p>5 * <!> = <!></p>";
22
export const $walks = /* get, over(1), next(1), over(1), replace, over(2), replace, out(1), next(1), over(1), replace, over(2), replace, out(1), next(1), over(1), replace, over(2), replace, out(1), next(1), over(1), replace, over(2), replace, out(1), next(1), over(1), replace, over(2), replace, out(1) */" bDb%c%lDb%c%lDb%c%lDb%c%lDb%c%l";
33
import { resolveAfter } from "../../utils/resolve";
4-
const multiply = (multiplier, n) => resolveAfter(multiplier * n, n);
4+
const multiply = (multiplier, n) => resolveAfter(multiplier * n);
55
import * as _ from "@marko/runtime-tags/debug/dom";
66
const $await_content5__result = /* @__PURE__ */_._const("result", $scope => _._text($scope["#text/0"], $scope.result));
77
const $await_content5__$params = /* @__PURE__ */_._const("$params6", $scope => $await_content5__result($scope, $scope.$params6[0]));

packages/runtime-tags/src/__tests__/fixtures/async-multi-resolve-in-order-and-update/__snapshots__/html.expected/template.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { resolveAfter } from "../../utils/resolve";
2-
const multiply = (multiplier, n) => resolveAfter(multiplier * n, n);
2+
const multiply = (multiplier, n) => resolveAfter(multiplier * n);
33
import * as _ from "@marko/runtime-tags/debug/html";
44
export default _._template("__tests__/template.marko", input => {
55
const $scope0_id = _._scope_id();

packages/runtime-tags/src/__tests__/fixtures/async-multi-resolve-in-order-and-update/template.marko

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { resolveAfter } from "../../utils/resolve";
2-
static const multiply = (multiplier, n) => resolveAfter(multiplier * n, n);
2+
static const multiply = (multiplier, n) => resolveAfter(multiplier * n);
33

44
<let/n=2>
55
<button onClick() { n++ }>increment</button>

packages/runtime-tags/src/__tests__/fixtures/async-multi-resolve-in-order-and-update/test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { flush, wait } from "../../utils/resolve";
22

3-
export const steps = [{}, flush, wait(3), click, wait(4)];
3+
export const steps = [{}, flush, wait, click, wait];
44

55
export const skip_equivalent = true; // in-order streaming
66

0 commit comments

Comments
 (0)