Skip to content

Commit ed468fc

Browse files
Merge pull request #471 from emberjs/ember-fetch-tests
Add test to prove ember-fetch@8 works with the current test-waiter system/integration
2 parents 248b0f6 + 2e96f89 commit ed468fc

File tree

5 files changed

+22
-43
lines changed

5 files changed

+22
-43
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
{
2+
"greeting": "hello there!"
3+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
import { module, test } from 'qunit';
2+
import { setupTest } from 'ember-qunit';
3+
import { settled } from '@ember/test-helpers';
4+
// this is bad, because fetch is actually ember-fetch, and we didn't install "fetch"
5+
import fetch from 'fetch';
6+
7+
module('ember-fetch@8', function (hooks) {
8+
setupTest(hooks);
9+
10+
test('it waits for the fetch to be done', async function (assert) {
11+
const promise = fetch('test-file.json');
12+
13+
assert.strictEqual(promise._state, undefined);
14+
assert.strictEqual(promise._result, undefined);
15+
await settled();
16+
assert.strictEqual(promise._state, 1);
17+
assert.strictEqual(promise._result?.statusText, 'OK');
18+
});
19+
});

test-apps/ember-fetch-v8/tests/helpers/index.ts

-43
This file was deleted.

test-apps/ember-fetch-v8/tests/integration/.gitkeep

Whitespace-only changes.

test-apps/ember-fetch-v8/tests/unit/.gitkeep

Whitespace-only changes.

0 commit comments

Comments
 (0)