From 9fd6512d2b87eb37547fe3e26b1da0a3c3568031 Mon Sep 17 00:00:00 2001
From: Pietro Albini <pietro@pietroalbini.org>
Date: Sun, 7 Apr 2019 22:23:14 +0200
Subject: add ability to exclude team members from rfcbot

---
 src/schema.rs     |  2 ++
 src/static_api.rs |  1 +
 src/validate.rs   | 29 +++++++++++++++++++++++++++++
 3 files changed, 32 insertions(+)

(limited to 'src')

diff --git a/src/schema.rs b/src/schema.rs
index be6b7a1..c7a1b48 100644
--- a/src/schema.rs
+++ b/src/schema.rs
@@ -229,6 +229,8 @@ pub(crate) struct RfcbotData {
     pub(crate) label: String,
     pub(crate) name: String,
     pub(crate) ping: String,
+    #[serde(default)]
+    pub(crate) exclude_members: Vec<String>,
 }
 
 pub(crate) struct DiscordInvite<'a> {
diff --git a/src/static_api.rs b/src/static_api.rs
index 3348d74..c752bc3 100644
--- a/src/static_api.rs
+++ b/src/static_api.rs
@@ -122,6 +122,7 @@ impl<'a> Generator<'a> {
                     .members(&self.data)?
                     .into_iter()
                     .map(|s| s.to_string())
+                    .filter(|member| !rfcbot.exclude_members.contains(&member))
                     .collect::<Vec<_>>();
                 members.sort();
                 teams.insert(
diff --git a/src/validate.rs b/src/validate.rs
index 21570d2..8c33570 100644
--- a/src/validate.rs
+++ b/src/validate.rs
@@ -19,6 +19,7 @@ static CHECKS: &[fn(&Data, &mut Vec<String>)] = &[
     validate_duplicate_permissions,
     validate_permissions,
     validate_rfcbot_labels,
+    validate_rfcbot_exclude_members,
 ];
 
 pub(crate) fn validate(data: &Data) -> Result<(), Error> {
@@ -298,6 +299,34 @@ fn validate_rfcbot_labels(data: &Data, errors: &mut Vec<String>) {
     });
 }
 
+/// Ensure rfcbot's exclude-members only contains not duplicated team members
+fn validate_rfcbot_exclude_members(data: &Data, errors: &mut Vec<String>) {
+    wrapper(data.teams(), errors, move |team, errors| {
+        if let Some(rfcbot) = team.rfcbot_data() {
+            let mut exclude = HashSet::new();
+            let members = team.members(data)?;
+            wrapper(rfcbot.exclude_members.iter(), errors, move |member, _| {
+                if !exclude.insert(member) {
+                    bail!(
+                        "duplicate member in `{}` rfcbot.exclude-members: {}",
+                        team.name(),
+                        member
+                    );
+                }
+                if !members.contains(member.as_str()) {
+                    bail!(
+                        "person `{}` is not a member of team `{}` (in rfcbot.exclude-members)",
+                        member,
+                        team.name()
+                    );
+                }
+                Ok(())
+            });
+        }
+        Ok(())
+    });
+}
+
 fn wrapper<T, I, F>(iter: I, errors: &mut Vec<String>, mut func: F)
 where
     I: Iterator<Item = T>,
-- 
cgit v1.2.3