Fix for -i command bug

mbloch · Jul 27, 2023 · a83f8ce · a83f8ce
1 parent 1e9232d
commit a83f8ce
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/cli/mapshaper-run-command.mjs b/src/cli/mapshaper-run-command.mjs
@@ -91,7 +91,7 @@ import '../commands/mapshaper-subdivide';
 function commandAcceptsMultipleTargetDatasets(name) {
   return name == 'rotate' || name == 'info' || name == 'proj' ||
     name == 'drop' || name == 'target' || name == 'if' || name == 'elif' ||
-    name == 'else' || name == 'endif' || name == 'run';
+    name == 'else' || name == 'endif' || name == 'run' || name == 'i';
 }
 
 function commandAcceptsEmptyTarget(name) {

diff --git a/test/commands-test.mjs b/test/commands-test.mjs
@@ -43,6 +43,15 @@ describe('mapshaper-run-commands.js', function () {
       })
     });
 
+  it('Fix: -i command throws error when there are multiple target datasets',async function() {
+    var a = 'foo\na';
+    var b = 'bar\nb';
+    var c = 'baz\nc';
+    var cmd = '-i a.csv -i b.csv -target * -i c.csv -o';
+    var out = await api.applyCommands(cmd, {'a.csv': a, 'b.csv': b, 'c.csv': c});
+    assert.equal(out['c.csv'], c);
+  })
+
   describe('Issue #264 applyCommands()', function() {
     it ('should throw error if input is a file path, not file content', function(done) {
       mapshaper.applyCommands('-i input.shp -o out.json', {'input.shp': 'test/data/two_states.shp'}, function(