Fix for Issue #836, bulk import apps

Author: kalvarez2

ui/src/app/apps/apps-bulk-import/properties/apps-bulk-import-properties.component.spec.ts

@@ -122,4 +122,16 @@ describe('AppsBulkImportPropertiesComponent', () => {
     }, 1000);
   });
 
+  it('should split the properties by new lines', () => {
+    [
+      {properties: '', force: true, expectedProp: ['']},
+      {properties: 'a=a', force: false, expectedProp: ['a=a']},
+      {properties: 'bar', force: false, expectedProp: ['bar']},
+      {properties: 'foo=bar=bar', force: false, expectedProp: ['foo=bar=bar']},
+      {properties: 'foo=bar\nbar', force: false, expectedProp: ['foo=bar', 'bar']}
+    ].forEach(r => {
+      const propsRequest =  component.prepareBulkImportRequest(r.properties, r.properties);
+      expect(propsRequest.properties).toEqual(r.expectedProp);
+    } );
+  });
 });

ui/src/app/apps/apps-bulk-import/properties/apps-bulk-import-properties.component.ts

@@ -80,17 +80,23 @@ export class AppsBulkImportPropertiesComponent implements OnDestroy {
     } catch (e) {
     }
   }
-
+  prepareBulkImportRequest(force, importProps:string){
+    return {
+      force: force,
+      properties: importProps.split('\n'),
+      uri: ''
+    };
+  }
   /**
    * Bulk Import Apps.
    * Submit the parameters
    */
   submit() {
-    const busy = this.appsService.bulkImportApps({
-      force: this.form.get('force').value,
-      properties: this.form.get('properties').value.toString().split('/n'),
-      uri: ''
-    }).pipe(takeUntil(this.ngUnsubscribe$))
+    const reqImportBulkApps = this.prepareBulkImportRequest(
+      this.form.get('force').value,
+      this.form.get('properties').value.toString());
+    const busy = this.appsService.bulkImportApps(reqImportBulkApps)
+      .pipe(takeUntil(this.ngUnsubscribe$))
       .subscribe(data => {
         this.notificationService.success('Apps Imported.');
         this.router.navigate(['apps']);