Skip to main content

nap_core/
repository.rs

1//! Universe repository — filesystem layout and manifest CRUD.
2//!
3//! A NAP repository represents a single fictional universe.
4//! Repository structure:
5//!
6//! ```text
7//! starwars/               ← universe root (Git repo)
8//! ├── .nap/               ← NAP metadata
9//! │   └── config.yaml     ← repository config
10//! ├── universe.yaml       ← world manifest (root-level)
11//! ├── characters/
12//! │   ├── lukeskywalker.yaml
13//! │   └── darthvader.yaml
14//! ├── locations/
15//! │   └── tatooine.yaml
16//! ├── scenes/
17//! │   └── cantina-scene.yaml
18//! └── props/
19//! ```
20
21use std::path::{Path, PathBuf};
22
23use tracing::{debug, info};
24
25use crate::commit::{Change, Commit};
26use crate::error::NapError;
27use crate::manifest::Manifest;
28use crate::types::EntityType;
29use crate::uri::NapUri;
30use crate::vcs::VcsBackend;
31
32/// NAP metadata directory name.
33const NAP_DIR: &str = ".nap";
34
35/// A NAP universe repository.
36pub struct Repository {
37    /// Filesystem path to the repository root.
38    pub root: PathBuf,
39    /// The universe name (derived from directory name).
40    pub universe: String,
41    /// The VCS backend (Git, Fossil, etc.).
42    vcs: Box<dyn VcsBackend>,
43}
44
45impl Repository {
46    /// Open an existing NAP repository at the given path.
47    pub fn open(path: &Path, vcs: Box<dyn VcsBackend>) -> Result<Self, NapError> {
48        let nap_dir = path.join(NAP_DIR);
49        if !nap_dir.exists() {
50            return Err(NapError::RepositoryNotFound(path.display().to_string()));
51        }
52        let universe = path
53            .file_name()
54            .and_then(|n| n.to_str())
55            .unwrap_or("unknown")
56            .to_string();
57
58        debug!(
59            path = %path.display(),
60            universe = %universe,
61            "opened NAP repository"
62        );
63
64        Ok(Self {
65            root: path.to_path_buf(),
66            universe,
67            vcs,
68        })
69    }
70
71    /// Initialize a new NAP repository.
72    pub fn init(path: &Path, universe: &str, vcs: Box<dyn VcsBackend>) -> Result<Self, NapError> {
73        let repo_root = path.join(universe);
74        if repo_root.join(NAP_DIR).exists() {
75            return Err(NapError::RepositoryAlreadyExists(
76                repo_root.display().to_string(),
77            ));
78        }
79
80        info!(
81            path = %repo_root.display(),
82            universe = %universe,
83            "initializing NAP repository"
84        );
85
86        // Create directory structure
87        std::fs::create_dir_all(&repo_root)?;
88        std::fs::create_dir_all(repo_root.join(NAP_DIR))?;
89
90        // Create entity type subdirectories
91        for entity_type in EntityType::subdirectory_types() {
92            std::fs::create_dir_all(repo_root.join(entity_type.directory_name()))?;
93        }
94
95        // Create .nap/config.yaml
96        let config = format!(
97            "# NAP Repository Configuration\nuniverse: {universe}\nprotocol_version: \"0.1.0\"\n"
98        );
99        std::fs::write(repo_root.join(NAP_DIR).join("config.yaml"), config)?;
100
101        // Create universe.yaml (world manifest)
102        let world_manifest = Manifest::new(
103            universe,
104            EntityType::World,
105            universe,
106            &format!("{universe} Universe"),
107        );
108        world_manifest.to_file(&repo_root.join("universe.yaml"))?;
109
110        // Initialize VCS
111        vcs.init(&repo_root)?;
112
113        // Initial commit
114        vcs.commit(
115            &repo_root,
116            &format!("Initialize {universe} universe"),
117            "nap-init",
118        )?;
119
120        info!(
121            path = %repo_root.display(),
122            universe = %universe,
123            "NAP repository initialized successfully"
124        );
125
126        Ok(Self {
127            root: repo_root,
128            universe: universe.to_string(),
129            vcs,
130        })
131    }
132
133    /// Get the full filesystem path to an entity's manifest file.
134    pub fn manifest_path(&self, entity_type: EntityType, entity_id: &str) -> PathBuf {
135        let uri = NapUri::new(&self.universe, entity_type, entity_id);
136        self.root.join(uri.manifest_path())
137    }
138
139    /// Read a manifest from the repository.
140    pub fn read_manifest(
141        &self,
142        entity_type: EntityType,
143        entity_id: &str,
144    ) -> Result<Manifest, NapError> {
145        let path = self.manifest_path(entity_type, entity_id);
146        debug!(
147            path = %path.display(),
148            entity_type = %entity_type,
149            entity_id = %entity_id,
150            "reading manifest"
151        );
152        Manifest::from_file(&path)
153    }
154
155    /// Read a manifest at a specific VCS ref (commit, branch, tag).
156    pub fn read_manifest_at_ref(
157        &self,
158        entity_type: EntityType,
159        entity_id: &str,
160        reference: &str,
161    ) -> Result<Manifest, NapError> {
162        let uri = NapUri::new(&self.universe, entity_type, entity_id);
163        let file_path = uri.manifest_path();
164
165        debug!(
166            file_path = %file_path,
167            reference = %reference,
168            "reading manifest at ref"
169        );
170
171        let content = self
172            .vcs
173            .read_file_at_ref(&self.root, &file_path, Some(reference))?;
174        Manifest::from_yaml(&content)
175    }
176
177    /// Write a manifest to the repository (does NOT commit).
178    pub fn write_manifest(&self, manifest: &Manifest) -> Result<PathBuf, NapError> {
179        let uri: NapUri = manifest.id.parse()?;
180        let path = self.root.join(uri.manifest_path());
181
182        debug!(
183            path = %path.display(),
184            manifest_id = %manifest.id,
185            "writing manifest"
186        );
187
188        manifest.to_file(&path)?;
189        Ok(path)
190    }
191
192    /// Create a new entity manifest and commit it.
193    pub fn create_entity(
194        &self,
195        entity_type: EntityType,
196        entity_id: &str,
197        name: &str,
198        author: &str,
199    ) -> Result<(Manifest, String), NapError> {
200        let mut manifest = Manifest::new(&self.universe, entity_type, entity_id, name);
201
202        // Validate against schema before writing
203        crate::schema::validate_manifest(&manifest)
204            .map_err(|errors| NapError::ManifestValidationError(errors.join("; ")))?;
205
206        // Write the manifest
207        self.write_manifest(&manifest)?;
208
209        // Commit via VCS
210        let commit_message = format!("Create {entity_type} '{name}' ({entity_id})");
211        let commit_hash = self.vcs.commit(&self.root, &commit_message, author)?;
212
213        // Update manifest with head pointer
214        manifest.head = Some(commit_hash.clone());
215        manifest.bump_version();
216        self.write_manifest(&manifest)?;
217
218        info!(
219            manifest_id = %manifest.id,
220            commit_hash = %commit_hash,
221            "created entity"
222        );
223
224        Ok((manifest, commit_hash))
225    }
226
227    /// Update an existing manifest and commit the changes.
228    pub fn commit_manifest(
229        &self,
230        manifest: &mut Manifest,
231        message: &str,
232        author: &str,
233        changes: Vec<Change>,
234    ) -> Result<Commit, NapError> {
235        // Validate against schema before writing
236        crate::schema::validate_manifest(manifest)
237            .map_err(|errors| NapError::ManifestValidationError(errors.join("; ")))?;
238
239        // Bump version
240        manifest.bump_version();
241
242        // Write updated manifest (without new head — we don't know it yet)
243        self.write_manifest(manifest)?;
244
245        // Compute manifest hash
246        let manifest_hash = manifest.content_hash()?.as_str().to_string();
247
248        // VCS commit — produces the new HEAD hash
249        let vcs_hash = self.vcs.commit(&self.root, message, author)?;
250
251        // Create NAP commit object with the now-known VCS hash
252        let nap_commit = Commit::new(
253            manifest.head.clone(),
254            author,
255            message,
256            &manifest_hash,
257            changes,
258        );
259
260        // Update head pointer and write again (leaves working tree dirty,
261        // same pattern as create_entity)
262        manifest.head = Some(vcs_hash.clone());
263        self.write_manifest(manifest)?;
264
265        debug!(
266            manifest_id = %manifest.id,
267            version = manifest.version,
268            nap_commit_id = %nap_commit.id,
269            vcs_hash = %vcs_hash,
270            "manifest committed"
271        );
272
273        Ok(nap_commit)
274    }
275
276    /// Get the commit history for a specific entity.
277    pub fn history(
278        &self,
279        entity_type: EntityType,
280        entity_id: &str,
281        limit: usize,
282    ) -> Result<Vec<crate::vcs::CommitInfo>, NapError> {
283        let uri = NapUri::new(&self.universe, entity_type, entity_id);
284        let file_path = uri.manifest_path();
285        self.vcs.log(&self.root, Some(&file_path), limit)
286    }
287
288    /// List all entity IDs of a given type in the repository.
289    pub fn list_entities(&self, entity_type: EntityType) -> Result<Vec<String>, NapError> {
290        let dir = self.root.join(entity_type.directory_name());
291        if !dir.exists() {
292            return Ok(vec![]);
293        }
294
295        let mut entities = Vec::new();
296        for entry in std::fs::read_dir(&dir)? {
297            let entry = entry?;
298            let path = entry.path();
299            if path.extension().and_then(|e| e.to_str()) == Some("yaml")
300                && let Some(stem) = path.file_stem().and_then(|s| s.to_str())
301            {
302                entities.push(stem.to_string());
303            }
304        }
305        entities.sort();
306        Ok(entities)
307    }
308
309    /// Delete an entity manifest and commit the deletion.
310    pub fn delete_entity(
311        &self,
312        entity_type: EntityType,
313        entity_id: &str,
314        author: &str,
315    ) -> Result<String, NapError> {
316        let path = self.manifest_path(entity_type, entity_id);
317        if !path.exists() {
318            return Err(NapError::ManifestNotFound(path.display().to_string()));
319        }
320
321        std::fs::remove_file(&path)?;
322
323        let message = format!("Delete {entity_type} '{entity_id}'");
324        let hash = self.vcs.commit(&self.root, &message, author)?;
325        info!(entity_type = %entity_type, entity_id = %entity_id, "deleted entity");
326        Ok(hash)
327    }
328
329    /// Create a branch in the underlying VCS.
330    pub fn create_branch(&self, name: &str) -> Result<(), NapError> {
331        self.vcs.create_branch(&self.root, name)
332    }
333
334    /// Switch to a branch.
335    pub fn switch_branch(&self, name: &str) -> Result<(), NapError> {
336        self.vcs.switch_branch(&self.root, name)
337    }
338
339    /// Create a tag.
340    pub fn create_tag(&self, name: &str) -> Result<(), NapError> {
341        self.vcs.create_tag(&self.root, name)
342    }
343
344    /// List branches.
345    pub fn list_branches(&self) -> Result<Vec<String>, NapError> {
346        self.vcs.list_branches(&self.root)
347    }
348
349    /// List tags.
350    pub fn list_tags(&self) -> Result<Vec<String>, NapError> {
351        self.vcs.list_tags(&self.root)
352    }
353
354    /// Revert a commit by creating a new VCS commit that undoes the specified one.
355    ///
356    /// The revert is a universe-level operation (not entity-scoped) — a single
357    /// Git commit can touch multiple files across multiple entity types.  After
358    /// reverting, working-tree files are restored to their pre-commit content
359    /// and a new revert commit is created in VCS history.
360    pub fn revert_commit(&self, commit_hash: &str, author: &str) -> Result<String, NapError> {
361        let new_hash = self.vcs.revert(&self.root, commit_hash)?;
362
363        // Re-read all entity manifests and update their `head` pointer
364        // so manifests are consistent with the new VCS state.
365        for et in EntityType::subdirectory_types() {
366            if let Ok(ids) = self.list_entities(*et) {
367                for id in &ids {
368                    if let Ok(mut manifest) = self.read_manifest(*et, id) {
369                        manifest.head = Some(new_hash.clone());
370                        self.write_manifest(&manifest).ok();
371                    }
372                }
373            }
374        }
375
376        info!(
377            commit = %commit_hash,
378            revert = %new_hash,
379            author = %author,
380            "commit reverted"
381        );
382
383        Ok(new_hash)
384    }
385
386    /// Get current HEAD hash.
387    pub fn head_hash(&self) -> Result<String, NapError> {
388        self.vcs.head_hash(&self.root)
389    }
390
391    // ── Remote operations ─────────────────────────────────────────
392
393    /// Add a remote to the repository.
394    pub fn add_remote(&self, name: &str, url: &str) -> Result<(), NapError> {
395        self.vcs.add_remote(&self.root, name, url)
396    }
397
398    /// Remove a remote from the repository.
399    pub fn remove_remote(&self, name: &str) -> Result<(), NapError> {
400        self.vcs.remove_remote(&self.root, name)
401    }
402
403    /// List remotes as `(name, url)` pairs.
404    pub fn list_remotes(&self) -> Result<Vec<(String, String)>, NapError> {
405        self.vcs.list_remotes(&self.root)
406    }
407
408    /// Push the current branch to a remote.
409    pub fn push(&self, remote: Option<&str>, branch: Option<&str>) -> Result<(), NapError> {
410        self.vcs.push(&self.root, remote, branch)
411    }
412
413    /// Pull the current branch from a remote.
414    pub fn pull(&self, remote: Option<&str>, branch: Option<&str>) -> Result<(), NapError> {
415        self.vcs.pull(&self.root, remote, branch)
416    }
417
418    /// Access the VCS backend (for the resolver to read files at specific refs).
419    pub fn vcs(&self) -> &dyn VcsBackend {
420        self.vcs.as_ref()
421    }
422}
423
424// ── In-memory mock VcsBackend for testing ──────────────────────────────
425
426#[cfg(test)]
427mod mock_backend {
428    use std::collections::HashMap;
429    use std::path::Path;
430    use std::sync::Mutex;
431    use std::sync::atomic::{AtomicU64, Ordering};
432
433    use crate::error::NapError;
434    use crate::vcs::{CommitInfo, VcsBackend};
435
436    /// A VcsBackend that simulates VCS operations in memory without shelling
437    /// out.  Used by the Repository unit tests so they don't require a real
438    /// `lore` binary.
439    pub(crate) struct MockBackend {
440        /// Incrementing counter for commit hashes.
441        counter: AtomicU64,
442        /// Stored commit metadata keyed by "hash".
443        commits: Mutex<Vec<CommitInfo>>,
444        /// Remote tracking.
445        remotes: Mutex<HashMap<String, String>>,
446        /// Branches.
447        branches: Mutex<Vec<String>>,
448        /// Current branch.
449        current_branch: Mutex<String>,
450        /// Tags.
451        tags: Mutex<Vec<String>>,
452    }
453
454    impl MockBackend {
455        pub fn new() -> Self {
456            Self {
457                counter: AtomicU64::new(1),
458                commits: Mutex::new(Vec::new()),
459                remotes: Mutex::new(HashMap::new()),
460                branches: Mutex::new(vec!["main".to_string()]),
461                current_branch: Mutex::new("main".to_string()),
462                tags: Mutex::new(Vec::new()),
463            }
464        }
465    }
466
467    impl VcsBackend for MockBackend {
468        fn init(&self, _path: &Path) -> Result<(), NapError> {
469            Ok(())
470        }
471
472        fn commit(&self, _path: &Path, message: &str, author: &str) -> Result<String, NapError> {
473            let n = self.counter.fetch_add(1, Ordering::SeqCst);
474            // Use a valid 40-char hex hash so manifest schema validation passes.
475            let hash = format!("{:040x}", n);
476            let mut commits = self.commits.lock().unwrap();
477            let parent = commits.last().map(|c| c.id.clone());
478            let info = CommitInfo {
479                id: hash.clone(),
480                parent,
481                author: author.to_string(),
482                message: message.to_string(),
483                timestamp: chrono::Utc::now().to_rfc3339(),
484            };
485            commits.push(info);
486            Ok(hash)
487        }
488
489        fn read_file_at_ref(
490            &self,
491            repo_path: &Path,
492            file_path: &str,
493            _reference: Option<&str>,
494        ) -> Result<String, NapError> {
495            let full_path = repo_path.join(file_path);
496            std::fs::read_to_string(&full_path)
497                .map_err(|e| NapError::Other(format!("mock: read failed: {}", e)))
498        }
499
500        fn log(
501            &self,
502            _path: &Path,
503            _file: Option<&str>,
504            _limit: usize,
505        ) -> Result<Vec<CommitInfo>, NapError> {
506            let commits = self.commits.lock().unwrap();
507            Ok(commits.clone())
508        }
509
510        fn create_branch(&self, _path: &Path, name: &str) -> Result<(), NapError> {
511            self.branches.lock().unwrap().push(name.to_string());
512            Ok(())
513        }
514
515        fn switch_branch(&self, _path: &Path, name: &str) -> Result<(), NapError> {
516            *self.current_branch.lock().unwrap() = name.to_string();
517            Ok(())
518        }
519
520        fn create_tag(&self, _path: &Path, name: &str) -> Result<(), NapError> {
521            let mut tags = self.tags.lock().unwrap();
522            if !tags.contains(&name.to_string()) {
523                tags.push(name.to_string());
524            }
525            Ok(())
526        }
527
528        fn current_branch(&self, _path: &Path) -> Result<String, NapError> {
529            Ok(self.current_branch.lock().unwrap().clone())
530        }
531
532        fn head_hash(&self, _path: &Path) -> Result<String, NapError> {
533            let commits = self.commits.lock().unwrap();
534            commits
535                .last()
536                .map(|c| c.id.clone())
537                .ok_or_else(|| NapError::VcsError("no commits".to_string()))
538        }
539
540        fn revert(&self, _repo_path: &Path, commit_hash: &str) -> Result<String, NapError> {
541            let n = self.counter.fetch_add(1, Ordering::SeqCst);
542            let hash = format!("{:040x}", n);
543            let info = CommitInfo {
544                id: hash.clone(),
545                parent: Some(commit_hash.to_string()),
546                author: "revert".to_string(),
547                message: format!("revert {}", commit_hash),
548                timestamp: chrono::Utc::now().to_rfc3339(),
549            };
550            self.commits.lock().unwrap().push(info);
551            Ok(hash)
552        }
553
554        fn list_branches(&self, _path: &Path) -> Result<Vec<String>, NapError> {
555            Ok(self.branches.lock().unwrap().clone())
556        }
557
558        fn list_tags(&self, _path: &Path) -> Result<Vec<String>, NapError> {
559            Ok(self.tags.lock().unwrap().clone())
560        }
561
562        fn add_remote(&self, _path: &Path, name: &str, url: &str) -> Result<(), NapError> {
563            self.remotes
564                .lock()
565                .unwrap()
566                .insert(name.to_string(), url.to_string());
567            Ok(())
568        }
569
570        fn remove_remote(&self, _path: &Path, name: &str) -> Result<(), NapError> {
571            self.remotes.lock().unwrap().remove(name);
572            Ok(())
573        }
574
575        fn list_remotes(&self, _path: &Path) -> Result<Vec<(String, String)>, NapError> {
576            let remotes = self.remotes.lock().unwrap();
577            Ok(remotes
578                .iter()
579                .map(|(k, v)| (k.clone(), v.clone()))
580                .collect())
581        }
582
583        fn push(
584            &self,
585            _path: &Path,
586            _remote: Option<&str>,
587            _branch: Option<&str>,
588        ) -> Result<(), NapError> {
589            Ok(())
590        }
591
592        fn pull(
593            &self,
594            _path: &Path,
595            _remote: Option<&str>,
596            _branch: Option<&str>,
597        ) -> Result<(), NapError> {
598            Ok(())
599        }
600    }
601}
602
603#[cfg(test)]
604mod tests {
605    use super::*;
606    use mock_backend::MockBackend;
607    use tempfile::TempDir;
608
609    fn make_repo(tmp: &TempDir) -> Repository {
610        Repository::init(tmp.path(), "testverse", Box::new(MockBackend::new())).unwrap()
611    }
612
613    #[test]
614    fn test_init_creates_structure() {
615        let tmp = TempDir::new().unwrap();
616        let repo = make_repo(&tmp);
617
618        assert!(repo.root.join(".nap").exists());
619        assert!(repo.root.join("universe.yaml").exists());
620        assert!(repo.root.join("characters").exists());
621        assert!(repo.root.join("locations").exists());
622        assert!(repo.root.join("scenes").exists());
623        assert!(repo.root.join("props").exists());
624    }
625
626    #[test]
627    fn test_create_and_read_entity() {
628        let tmp = TempDir::new().unwrap();
629        let repo = make_repo(&tmp);
630
631        let (manifest, _hash) = repo
632            .create_entity(EntityType::Character, "hero", "The Hero", "test-author")
633            .unwrap();
634
635        assert_eq!(manifest.name, "The Hero");
636        assert_eq!(manifest.entity_type, EntityType::Character);
637
638        // Read it back
639        let read_back = repo.read_manifest(EntityType::Character, "hero").unwrap();
640        assert_eq!(read_back.name, "The Hero");
641    }
642
643    #[test]
644    fn test_list_entities() {
645        let tmp = TempDir::new().unwrap();
646        let repo = make_repo(&tmp);
647
648        repo.create_entity(EntityType::Character, "alice", "Alice", "author")
649            .unwrap();
650        repo.create_entity(EntityType::Character, "bob", "Bob", "author")
651            .unwrap();
652
653        let chars = repo.list_entities(EntityType::Character).unwrap();
654        assert_eq!(chars, vec!["alice", "bob"]);
655    }
656
657    #[test]
658    fn test_commit_manifest_updates() {
659        let tmp = TempDir::new().unwrap();
660        let repo = make_repo(&tmp);
661
662        let (mut manifest, _) = repo
663            .create_entity(EntityType::Character, "hero", "The Hero", "author")
664            .unwrap();
665
666        // Modify and commit
667        manifest.set_property("species", serde_yaml::Value::String("elf".to_string()));
668        let changes = vec![Change::set("properties.species", None, "elf".to_string())];
669        let commit = repo
670            .commit_manifest(&mut manifest, "set species to elf", "author", changes)
671            .unwrap();
672
673        assert!(!commit.id.is_empty());
674        assert_eq!(commit.message, "set species to elf");
675
676        // Verify version incremented
677        let read_back = repo.read_manifest(EntityType::Character, "hero").unwrap();
678        assert!(read_back.version >= 2);
679    }
680
681    #[test]
682    fn test_history() {
683        let tmp = TempDir::new().unwrap();
684        let repo = make_repo(&tmp);
685
686        let (mut manifest, _) = repo
687            .create_entity(EntityType::Character, "hero", "The Hero", "author")
688            .unwrap();
689
690        manifest.set_property(
691            "name",
692            serde_yaml::Value::String("Updated Hero".to_string()),
693        );
694        repo.commit_manifest(&mut manifest, "update name", "author", vec![])
695            .unwrap();
696
697        let hist = repo.history(EntityType::Character, "hero", 10).unwrap();
698        assert!(hist.len() >= 2);
699    }
700
701    #[test]
702    fn test_revert_commit() {
703        let tmp = TempDir::new().unwrap();
704        let repo = make_repo(&tmp);
705
706        // Create entity and note its name
707        let (mut manifest, _) = repo
708            .create_entity(EntityType::Character, "hero", "The Hero", "author")
709            .unwrap();
710        assert_eq!(manifest.name, "The Hero");
711
712        // Modify and commit
713        manifest.set_property("species", serde_yaml::Value::String("elf".to_string()));
714        let changes = vec![Change::set("properties.species", None, "elf".to_string())];
715        let _commit = repo
716            .commit_manifest(&mut manifest, "set species to elf", "author", changes)
717            .unwrap();
718
719        // Verify the property was set
720        let read_back = repo.read_manifest(EntityType::Character, "hero").unwrap();
721        assert_eq!(
722            read_back.properties.get("species").and_then(|v| v.as_str()),
723            Some("elf")
724        );
725
726        // Get the VCS commit hash from the manifest's head pointer.
727        // After commit_manifest, this is the single VCS commit containing
728        // the property change (the head pointer update is left dirty).
729        let vcs_hash = read_back
730            .head
731            .as_ref()
732            .expect("head should be set after commit");
733
734        // Revert that VCS commit
735        let revert_hash = repo.revert_commit(vcs_hash, "author").unwrap();
736        assert!(!revert_hash.is_empty());
737
738        // Verify the manifest head was updated to the revert commit
739        let after_revert = repo.read_manifest(EntityType::Character, "hero").unwrap();
740        assert_eq!(after_revert.head.as_deref(), Some(revert_hash.as_str()));
741
742        // Verify the revert appears in history
743        let hist = repo.history(EntityType::Character, "hero", 10).unwrap();
744        assert!(hist.iter().any(|c| c.id == revert_hash));
745    }
746
747    #[test]
748    fn test_remote_operations() {
749        let tmp = TempDir::new().unwrap();
750        let repo = make_repo(&tmp);
751
752        repo.add_remote("origin", "git@github.com:user/repo.git")
753            .unwrap();
754        let remotes = repo.list_remotes().unwrap();
755        assert_eq!(remotes.len(), 1);
756        assert_eq!(remotes[0].0, "origin");
757
758        repo.remove_remote("origin").unwrap();
759        let remotes = repo.list_remotes().unwrap();
760        assert!(remotes.is_empty());
761    }
762}