Merge pull request #29 from UCSD-E4E/25-as-a-user-i-would-like-e4edm-…

…push-to-include-the-dataset-directory-so-that-i-do-not-lose-the-dataset-context

25 as a user i would like e4edm push to include the dataset directory so that i do not lose the dataset context

e4e_data_management/core.py

@@ -296,7 +296,9 @@ def push(self, path: Path) -> None:
         self.active_dataset.validate()
 
         # Duplicate to destination
-        self.duplicate([path])
+        destination = path.joinpath(self.active_dataset.name)
+        destination.mkdir(parents=True, exist_ok=False)
+        self.duplicate([destination])
 
     def zip(self, output_path: Path) -> None:
         """This will zip the active and completed dataset to the specified path

tests/test_push.py

@@ -28,6 +28,8 @@ def test_push(single_mission_data: Tuple[Tuple[Mock, DataManager, Path], Tuple[P
         push_path = Path(push_dir)
         app.push(push_path)
 
+        assert push_path.joinpath(app.active_dataset.name).is_dir()
+
 @pytest.mark.parametrize('readme_name', [
     'readme.md',
     'readme.MD',