when used as a library, return a Promise

Author: addam

bin/obj23dtiles.js

@@ -257,4 +257,8 @@ var options = {
     outputDirectory : outputDirectory
 };
 
-obj23dtiles(objPath, outputPath, options);
+console.time('Total');
+obj23dtiles(objPath, outputPath, options)
+    .then(() => {
+        console.timeEnd('Total');
+    });

lib/obj23dtiles.js

@@ -11,7 +11,6 @@ module.exports = obj23dtiles;
 obj23dtiles.combine = combine;
 
 function obj23dtiles(objPath, outputPath, options) {
-    console.time('Total');
 
     if(typeof options.tilesetOptions === 'string') {
         options.tilesetOptions = fsExtra.readJsonSync(options.tilesetOptions);
@@ -49,9 +48,6 @@ function obj23dtiles(objPath, outputPath, options) {
                     tasks.push(fsExtra.writeJson(tilesetPath, tileset, {spaces: 2}));
                     return Promise.all(tasks);
                 })
-                .then(function() {
-                    console.timeEnd('Total');
-                })
                 .catch(function(error) {
                     console.log(error.message || error);
                     process.exit(1);
@@ -64,7 +60,7 @@ function obj23dtiles(objPath, outputPath, options) {
                 process.exit(1);
             }
 
-            obj2Tileset(objPath, outputPath, options)
+            return obj2Tileset(objPath, outputPath, options)
                 .then(function(result) {
                     var i3dm = result.i3dm;
                     var batchTableJson = result.batchTableJson;
@@ -84,9 +80,6 @@ function obj23dtiles(objPath, outputPath, options) {
                     tasks.push(fsExtra.writeJson(tilesetPath, tileset, {spaces: 2}));
                     return Promise.all(tasks);
                 })
-                .then(function() {
-                    console.timeEnd('Total');
-                })
                 .catch(function(error) {
                     console.log(error.message || error);
                     process.exit(1);
@@ -96,7 +89,7 @@ function obj23dtiles(objPath, outputPath, options) {
     else if (options && options.b3dm) {
         options.binary = true;
         options.batchId = true;
-        obj2B3dm(objPath, options)
+        return obj2B3dm(objPath, options)
             .then(function(result){
                 var b3dm = result.b3dm;
                 var batchTableJson = result.batchTableJson;
@@ -109,9 +102,6 @@ function obj23dtiles(objPath, outputPath, options) {
                 fsExtra.ensureDirSync(path.dirname(outputPath));
                 return fsExtra.outputFile(outputPath, b3dm);
             })
-            .then(function() {
-                console.timeEnd('Total');
-            })
             .catch(function(error) {
                 console.log(error.message || error);
                 process.exit(1);
@@ -137,9 +127,6 @@ function obj23dtiles(objPath, outputPath, options) {
                 fsExtra.ensureDirSync(path.dirname(outputPath));
                 return fsExtra.outputFile(outputPath, i3dm);
             })
-            .then(function() {
-                console.timeEnd('Total');
-            })
             .catch(function(error) {
                 console.log(error.message || error);
                 process.exit(1);
@@ -158,9 +145,6 @@ function obj23dtiles(objPath, outputPath, options) {
                 };
                 return fsExtra.outputJson(outputPath, gltf, jsonOptions);
             })
-            .then(function() {
-                console.timeEnd('Total');
-            })
             .catch(function(error) {
                 console.log(error.message || error);
                 process.exit(1);