Don't append a slash to file paths

ansible · Mar 25, 2021 · 1da8956 · 1da8956
1 parent 9b702e4
commit 1da8956
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/awx/ui_next/src/screens/Inventory/shared/InventorySourceSubForms/SCMSubForm.jsx b/awx/ui_next/src/screens/Inventory/shared/InventorySourceSubForms/SCMSubForm.jsx
@@ -122,9 +122,6 @@ const SCMSubForm = ({ autoPopulateProject, i18n }) => {
           onSelect={(event, value) => {
             setIsOpen(false);
             value = value.trim();
-            if (!value.endsWith('/')) {
-              value += '/';
-            }
             sourcePathHelpers.setValue(value);
           }}
           aria-label={i18n._(t`Select source path`)}

diff --git a/awx/ui_next/src/screens/Inventory/shared/InventorySourceSubForms/SCMSubForm.test.jsx b/awx/ui_next/src/screens/Inventory/shared/InventorySourceSubForms/SCMSubForm.test.jsx
@@ -98,7 +98,7 @@ describe('<SCMSubForm />', () => {
     });
     wrapper.update();
     expect(wrapper.find('Select#source_path').prop('selections')).toEqual(
-      'bar/'
+      'bar'
     );
 
     await act(async () => {
@@ -138,7 +138,7 @@ describe('<SCMSubForm />', () => {
       customWrapper.find('Select').invoke('onSelect')({}, 'newPath');
     });
     customWrapper.update();
-    expect(customWrapper.find('Select').prop('selections')).toBe('newPath/');
+    expect(customWrapper.find('Select').prop('selections')).toBe('newPath');
   });
   test('Update on project update should be disabled', async () => {
     const customInitialValues = {