Test different sets of PIs per project. Closes #21.

R/project_contacts.R

@@ -81,9 +81,10 @@ project_contacts <- function(extract, type = c("receiver", "tag")) {
       ),
       by = "detectedby"
       ]
-      pis <- merge(pis[, -c("PI", "POC", "PI_emails", "POC_emails")],
+      pis <- merge(
+        pis[, -c("PI", "POC", "PI_emails", "POC_emails")],
         pi_key,
-        on = "detectedby"
+        by = "detectedby"
       )
       pis <- unique(pis, by = "detectedby")
       # maybe need to also merge their geom?

tests/testthat/test-project_contacts.R

@@ -107,3 +107,36 @@ test_that("right things are returned for receivers", {
     all(grepl("@|", pi_table$POC_emails))
   )
 })
+
+
+
+
+test_that("Multiple sets of tag PIs per project are summarized", {
+  matched_multiple_pis <- matched
+  matched_multiple_pis[matched_multiple_pis$detectedby == 'HFX',][1, 'contact_pi'] <-
+    c("Matthew Apostle (matt@bible), Mary Mother (mary@bible)")
+
+
+  pi_table <- project_contacts(matched_multiple_pis, "tag")
+
+  expect_false(
+    any(
+      duplicated(pi_table$project_name)
+    )
+  )
+
+  expect_match(
+    pi_table[pi_table$project_name == "HFX",]$PI,
+    "Matthew Apostle, Mary Mother, Dave Hebert, Fred Whoriskey"
+  )
+
+  expect_match(
+    pi_table[pi_table$project_name == "HFX",]$PI_emails,
+    "matt@bible,mary@bible,david.hebert@dfo-mpo.gc.ca,fwhoriskey@dal.ca"
+  )
+
+})
+
+test_that("Multiple sets of receiver PIs per project are summarized", {
+
+})