Added tests for ModelDiscoverer
Change-Id: I8cfd022677b341b28c4765c1ec4e0a4e69b3679a
diff --git a/src/app/datasources/helpers/model-discoverer.service.ts b/src/app/datasources/helpers/model-discoverer.service.ts
index ba9cb13..4306825 100644
--- a/src/app/datasources/helpers/model-discoverer.service.ts
+++ b/src/app/datasources/helpers/model-discoverer.service.ts
@@ -106,7 +106,7 @@
return this.$q.resolve('true');
})
.catch(err => {
- this.$log.error(`[XosModelDiscovererService] Model ${model.name} NOT stored`);
+ this.$log.error(`[XosModelDiscovererService] Model ${model.name} NOT stored`, err);
return this.$q.resolve('false');
});
pArray.push(p);
@@ -116,11 +116,12 @@
// the Model Loader promise won't ever be reject, in case it will be resolve with value false,
// that's because we want to wait anyway for all the models to be loaded
if (res.indexOf('false') > -1) {
- d.resolve(false);
+ return d.resolve(false);
}
d.resolve(true);
})
- .catch(() => {
+ .catch((e) => {
+ this.$log.error(`[XosModelDiscovererService]`, e);
d.resolve(false);
})
.finally(() => {