TOOLS-474: --ignoreBlanks is invalid with JSON input type

Former-commit-id: ade9b6fcc37b5566c86232d3622fd3ea395fbf7c

2 files changed, 10 insertions, 0 deletions

diff --git a/mongoimport/mongoimport.go b/mongoimport/mongoimport.go
index 1442a981960..c580245b6e6 100644
--- a/mongoimport/mongoimport.go
+++ b/mongoimport/mongoimport.go
@@ -141,6 +141,9 @@ func (mongoImport *MongoImport) ValidateSettings(args []string) error {
 		if mongoImport.InputOptions.FieldFile != nil {
 			return fmt.Errorf("can not use --fieldFile when input type is JSON")
 		}
+		if mongoImport.IngestOptions.IgnoreBlanks {
+			return fmt.Errorf("can not use --ignoreBlanks when input type is JSON")
+		}
 	}
 
 	if mongoImport.IngestOptions.UpsertFields != "" {
diff --git a/mongoimport/mongoimport_test.go b/mongoimport/mongoimport_test.go
index 6eb52c10e48..64a21b00cf4 100644
--- a/mongoimport/mongoimport_test.go
+++ b/mongoimport/mongoimport_test.go
@@ -157,6 +157,13 @@ func TestMongoImportValidateSettings(t *testing.T) {
 			So(mongoImport.ValidateSettings([]string{}), ShouldNotBeNil)
 		})
 
+		Convey("an error should be thrown if --ignoreBlanks is used with JSON input", func() {
+			mongoImport, err := NewMongoImport()
+			So(err, ShouldBeNil)
+			mongoImport.IngestOptions.IgnoreBlanks = true
+			So(mongoImport.ValidateSettings([]string{}), ShouldNotBeNil)
+		})
+
 		Convey("no error should be thrown if --headerline is not supplied "+
 			"but --fieldFile is supplied", func() {
 			mongoImport, err := NewMongoImport()