Add regression tests for Remote::list() with an empty list

Test currently fails in debug builds with unsafe preconditions of
`slice::from_raw_parts()` being violated, in release builds it currently passes
but relies on undefined behavior.

Author: DanielEScherzer

src/remote.rs

@@ -921,6 +921,20 @@ mod tests {
         drop(origin.clone());
     }
 
+    #[test]
+    fn empty_remote_list() {
+        // Regression tests for issue #1217
+        let td = TempDir::new().unwrap();
+        let repo = Repository::init(td.path()).unwrap();
+
+        let remote_dir = TempDir::new().unwrap();
+        let _remote_repo = Repository::init_bare(remote_dir.path()).unwrap();
+        let remote_url = format!("file://{}", remote_dir.path().display());
+        let mut remote = repo.remote("origin", &remote_url).unwrap();
+        remote.connect(Direction::Fetch).unwrap();
+        assert_eq!(0, remote.list().unwrap().len());
+    }
+
     #[test]
     fn is_valid_name() {
         assert!(Remote::is_valid_name("foobar"));