Skip to content

Commit

Permalink
feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
snewcomer committed Mar 31, 2022
1 parent 9b48697 commit d7c0e2e
Show file tree
Hide file tree
Showing 5 changed files with 21 additions and 21 deletions.
2 changes: 1 addition & 1 deletion packages/-ember-data/tests/integration/store-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,7 @@ module('integration/store - destroy', function (hooks) {
assert.ok(true, 'we made it to the end');

if (DEPRECATE_RSVP_PROMISE) {
assert.expectDeprecation({ id: 'ember-data:rsvp-promise-hanging', count: 1 });
assert.expectDeprecation({ id: 'ember-data:rsvp-unresolved-async', count: 1 });
}

requestPromise.then(() => {
Expand Down
4 changes: 2 additions & 2 deletions packages/-ember-data/tests/unit/store/async-leak-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ module('unit/store async-waiter and leak detection', function (hooks) {
assert.true(waiter(), 'We return true because the waiter is cleared');

if (DEPRECATE_RSVP_PROMISE) {
assert.expectDeprecation({ id: 'ember-data:rsvp-promise-hanging', count: 1 });
assert.expectDeprecation({ id: 'ember-data:rsvp-unresolved-async', count: 1 });
}
});

Expand Down Expand Up @@ -258,7 +258,7 @@ module('unit/store async-waiter and leak detection', function (hooks) {
assert.true(waiter(), 'We return true because the waiter is cleared');

if (DEPRECATE_RSVP_PROMISE) {
assert.expectDeprecation({ id: 'ember-data:rsvp-promise-hanging', count: 1 });
assert.expectDeprecation({ id: 'ember-data:rsvp-unresolved-async', count: 1 });
}
});

Expand Down
18 changes: 9 additions & 9 deletions packages/store/addon/-private/system/fetch-manager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ interface PendingSaveItem {
resolver: RSVP.Deferred<any>;
snapshot: Snapshot;
identifier: RecordIdentifier;
options: { [k: string]: unknown; [SaveOp]: 'createRecord' | 'saveRecord' | 'updateRecord' };
options: { [k: string]: unknown;[SaveOp]: 'createRecord' | 'saveRecord' | 'updateRecord' };
queryRequest: Request;
}

Expand Down Expand Up @@ -147,12 +147,12 @@ export default class FetchManager {
`A Promise while saving ${modelName} did not resolve by the time your model was destroyed. This will error in a future release.`,
false,
{
id: 'ember-data:rsvp-promise-hanging',
id: 'ember-data:rsvp-unresolved-async',
until: '5.0',
for: '@ember-data/store',
since: {
available: '4.4',
enabled: '4.4',
available: '4.5',
enabled: '4.5',
},
}
);
Expand Down Expand Up @@ -370,8 +370,8 @@ export default class FetchManager {
if (missingSnapshots.length) {
warn(
'Ember Data expected to find records with the following ids in the adapter response but they were missing: [ "' +
missingSnapshots.map((r) => r.id).join('", "') +
'" ]',
missingSnapshots.map((r) => r.id).join('", "') +
'" ]',
false,
{
id: 'ds.store.missing-records-from-adapter',
Expand All @@ -393,9 +393,9 @@ export default class FetchManager {
if (pair) {
pair.resolver.reject(
error ||
new Error(
`Expected: '<${snapshot.modelName}:${snapshot.id}>' to be present in the adapter provided payload, but it was not found.`
)
new Error(
`Expected: '<${snapshot.modelName}:${snapshot.id}>' to be present in the adapter provided payload, but it was not found.`
)
);
}
}
Expand Down
6 changes: 3 additions & 3 deletions packages/store/addon/-private/system/store/common.js
Original file line number Diff line number Diff line change
Expand Up @@ -42,12 +42,12 @@ export function guardDestroyedStore(promise, store, label) {
`A Promise did not resolve by the time the store was destroyed. This will error in a future release.`,
false,
{
id: 'ember-data:rsvp-promise-hanging',
id: 'ember-data:rsvp-unresolved-async',
until: '5.0',
for: '@ember-data/store',
since: {
available: '4.4',
enabled: '4.4',
available: '4.5',
enabled: '4.5',
},
}
);
Expand Down
12 changes: 6 additions & 6 deletions packages/store/addon/-private/system/store/finders.js
Original file line number Diff line number Diff line change
Expand Up @@ -222,12 +222,12 @@ export function _findHasMany(adapter, store, internalModel, link, relationship,
`A Promise for fecthing ${relationship.type} did not resolve by the time your model was destroyed. This will error in a future release.`,
false,
{
id: 'ember-data:rsvp-promise-hanging',
id: 'ember-data:rsvp-unresolved-async',
until: '5.0',
for: '@ember-data/store',
since: {
available: '4.4',
enabled: '4.4',
available: '4.5',
enabled: '4.5',
},
}
);
Expand Down Expand Up @@ -281,12 +281,12 @@ export function _findBelongsTo(adapter, store, internalModel, link, relationship
`A Promise for fetching ${relationship.type} did not resolve by the time your model was destroyed. This will error in a future release.`,
false,
{
id: 'ember-data:rsvp-promise-hanging',
id: 'ember-data:rsvp-unresolved-async',
until: '5.0',
for: '@ember-data/store',
since: {
available: '4.4',
enabled: '4.4',
available: '4.5',
enabled: '4.5',
},
}
);
Expand Down

0 comments on commit d7c0e2e

Please sign in to comment.