fix: gix::open::Permissions now can be compared, and add is_isolated() method.

Author: Byron

gix/src/open/mod.rs

@@ -3,7 +3,7 @@ use std::path::PathBuf;
 use crate::{bstr::BString, config};
 
 /// Permissions associated with various resources of a git repository
-#[derive(Debug, Clone)]
+#[derive(Copy, Clone, Ord, PartialOrd, PartialEq, Eq, Debug, Hash)]
 pub struct Permissions {
     /// Control which environment variables may be accessed.
     pub env: permissions::Environment,

gix/src/open/permissions.rs

@@ -112,7 +112,7 @@ impl Default for Attributes {
 }
 
 /// Permissions related to the usage of environment variables
-#[derive(Debug, Clone, Copy)]
+#[derive(Copy, Clone, Ord, PartialOrd, PartialEq, Eq, Debug, Hash)]
 pub struct Environment {
     /// Control whether resources pointed to by `XDG_CONFIG_HOME` can be used when looking up common configuration values.
     ///
@@ -170,6 +170,7 @@ impl Environment {
     }
 }
 
+/// Lifecycle
 impl Permissions {
     /// Secure permissions are similar to `all()`
     pub fn secure() -> Self {
@@ -200,6 +201,14 @@ impl Permissions {
     }
 }
 
+/// Access
+impl Permissions {
+    /// Return true if these permissions only allow repository-local configuration and deny environment access.
+    pub fn is_isolated(&self) -> bool {
+        *self == Self::isolated()
+    }
+}
+
 impl gix_sec::trust::DefaultForLevel for Permissions {
     fn default_for_level(level: Trust) -> Self {
         match level {

gix/tests/gix/repository/open.rs

@@ -4,6 +4,12 @@ use gix::bstr::BString;
 
 use crate::util::named_subrepo_opts;
 
+#[test]
+fn open_permissions_is_isolated() {
+    assert!(gix::open::Permissions::isolated().is_isolated());
+    assert!(!gix::open::Permissions::all().is_isolated());
+}
+
 #[test]
 fn on_root_with_decomposed_unicode() -> crate::Result {
     let tmp = gix_testtools::tempfile::TempDir::new()?;