Commit 709c25a4 authored by Jai Bhagat's avatar Jai Bhagat
Browse files

refact: preserve promise.then behavior for acceptance tests

parent 9d28d9eb
Branches unavailable
No related merge requests found
Showing with 12 additions and 12 deletions
+12 -12
......@@ -36,18 +36,18 @@ export default class AllocationRoute extends Route.extend(WithWatchers) {
}
}
async model() {
try {
// Preload the job for the allocation since it's required for the breadcrumb trail
const allocation = await super.model(...arguments);
const jobId = allocation?.belongsTo('job').id();
const getJob = this.store.findRecord('job', jobId);
const getNamespaces = this.store.findAll('namespace');
await Promise.all([getJob, getNamespaces]);
return allocation;
} catch {
notifyError(this);
}
model() {
// Preload the job for the allocation since it's required for the breadcrumb trail
return super
.model(...arguments)
.then((allocation) => {
const jobId = allocation.belongsTo('job').id();
return this.store
.findRecord('job', jobId)
.then(() => this.store.findAll('namespace')) // namespaces belong to a job and are an asynchronous relationship so we can peak them later on
.then(() => allocation);
})
.catch(notifyError(this));
}
@watchRecord('allocation') watch;
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment