Trim pipeline upload url before sending to backend (#2630)

frontend/src/components/UploadPipelineDialog.test.tsx

@@ -81,6 +81,28 @@ describe('UploadPipelineDialog', () => {
     expect(spy).toHaveBeenLastCalledWith(true, 'test name', null, '', ImportMethod.LOCAL, '');
   });
 
+  it('calls close callback with trimmed file url and pipeline name when confirmed', () => {
+    const spy = jest.fn();
+    tree = shallow(<UploadPipelineDialog open={false} onClose={spy} />);
+    // Click 'Import by URL'
+    tree.find('#uploadFromUrlBtn').simulate('change');
+    (tree.instance() as UploadPipelineDialog).handleChange('fileUrl')({
+      target: { value: '\n https://www.google.com/test-file.txt ' },
+    });
+    (tree.instance() as UploadPipelineDialog).handleChange('uploadPipelineName')({
+      target: { value: 'test name' },
+    });
+    tree.find('#confirmUploadBtn').simulate('click');
+    expect(spy).toHaveBeenLastCalledWith(
+      true,
+      'test name',
+      null,
+      'https://www.google.com/test-file.txt',
+      ImportMethod.URL,
+      '',
+    );
+  });
+
   it('trims file extension for pipeline name suggestion', () => {
     tree = shallow(<UploadPipelineDialog open={false} onClose={jest.fn()} />);
     const file = { name: 'test_upload_file.tar.gz' };

frontend/src/components/UploadPipelineDialog.tsx

@@ -257,7 +257,7 @@ class UploadPipelineDialog extends React.Component<
         confirmed,
         this.state.uploadPipelineName,
         this.state.file,
-        this.state.fileUrl,
+        this.state.fileUrl.trim(),
         this.state.importMethod,
         this.state.uploadPipelineDescription,
       );