Skip to content

Commit

Permalink
Add duplicate JIRA issue events to testing data and adjust tests
Browse files Browse the repository at this point in the history 
As the codeface extraction produces duplicate 'add_link' and 'referenced_by'
events from JIRA repositories, the coronet testing data should reflect that.
Adjust the tests to work with added JIRA issue events.

Signed-off-by: Maximilian Löffler <[email protected]>
  • Loading branch information
@maxloeffler
maxloeffler committed Jan 25, 2024
1 parent 2d4707e commit 7cd8828
Show file tree
Hide file tree
Showing 8 changed files with 202 additions and 119 deletions.
2 changes: 2 additions & 0 deletions tests/codeface-data/results/testing/test_feature/feature/issues-jira.list
View file
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@
"ZEPPELIN-328";"[ZEPPELIN-328] Interpreter page should clarify the % magic syntax for interpreter group.name";"[""issue"", ""bug""]";"closed";"[""fixed""]";"2013-04-21 23:52:09";"2013-05-25 20:02:08";"[""GUI"", ""Interpreters""]";"commented";"Olaf";"[email protected]";"2013-06-01 06:50:26";"open";"[""unresolved""]"
"ZEPPELIN-328";"[ZEPPELIN-328] Interpreter page should clarify the % magic syntax for interpreter group.name";"[""issue"", ""bug""]";"closed";"[""fixed""]";"2013-04-21 23:52:09";"2013-05-25 20:02:08";"[""GUI"", ""Interpreters""]";"resolution_updated";"Björn";"[email protected]";"2013-06-01 06:53:06";"fixed";"""unresolved"""
"ZEPPELIN-328";"[ZEPPELIN-328] Interpreter page should clarify the % magic syntax for interpreter group.name";"[""issue"", ""bug""]";"closed";"[""fixed""]";"2013-04-21 23:52:09";"2013-05-25 20:02:08";"[""GUI"", ""Interpreters""]";"referenced_by";"Thomas";"[email protected]";"2017-05-21 12:00:00";"ZEPPELIN-332";"""issue"""
"ZEPPELIN-328";"[ZEPPELIN-328] Interpreter page should clarify the % magic syntax for interpreter group.name";"[""issue"", ""bug""]";"closed";"[""fixed""]";"2013-04-21 23:52:09";"2013-05-25 20:02:08";"[""GUI"", ""Interpreters""]";"add_link";"Thomas";"[email protected]";"2017-05-21 12:00:00";"ZEPPELIN-332";"""issue"""
"ZEPPELIN-332";"[ZEPPELIN-332] CNFE when running SQL query against Cassandra temp table";"[""issue"", ""bug""]";"open";"[""unresolved""]";"2016-07-12 16:01:30";"";"[""Interpreters""]";"referenced_by";"Thomas";"[email protected]";"2017-05-21 12:00:00";"ZEPPELIN-328";"""issue"""
"ZEPPELIN-332";"[ZEPPELIN-332] CNFE when running SQL query against Cassandra temp table";"[""issue"", ""bug""]";"open";"[""unresolved""]";"2016-07-12 16:01:30";"";"[""Interpreters""]";"add_link";"Thomas";"[email protected]";"2017-05-21 12:00:00";"ZEPPELIN-328";"""issue"""
"ZEPPELIN-332";"[ZEPPELIN-332] CNFE when running SQL query against Cassandra temp table";"[""issue"", ""bug""]";"open";"[""unresolved""]";"2016-07-12 16:01:30";"";"[""Interpreters""]";"created";"Björn";"[email protected]";"2016-07-12 16:01:30";"open";"[""unresolved""]"
"ZEPPELIN-332";"[ZEPPELIN-332] CNFE when running SQL query against Cassandra temp table";"[""issue"", ""bug""]";"open";"[""unresolved""]";"2016-07-12 16:01:30";"";"[""Interpreters""]";"commented";"Björn";"[email protected]";"2016-07-12 16:02:30";"open";"[""unresolved""]"
Expand Down
2 changes: 2 additions & 0 deletions tests/codeface-data/results/testing/test_proximity/proximity/issues-jira.list
View file
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@
"ZEPPELIN-328";"[ZEPPELIN-328] Interpreter page should clarify the % magic syntax for interpreter group.name";"[""issue"", ""bug""]";"closed";"[""fixed""]";"2013-04-21 23:52:09";"2013-05-25 20:02:08";"[""GUI"", ""Interpreters""]";"commented";"Olaf";"[email protected]";"2013-06-01 06:50:26";"open";"[""unresolved""]"
"ZEPPELIN-328";"[ZEPPELIN-328] Interpreter page should clarify the % magic syntax for interpreter group.name";"[""issue"", ""bug""]";"closed";"[""fixed""]";"2013-04-21 23:52:09";"2013-05-25 20:02:08";"[""GUI"", ""Interpreters""]";"resolution_updated";"Björn";"[email protected]";"2013-06-01 06:53:06";"fixed";"""unresolved"""
"ZEPPELIN-328";"[ZEPPELIN-328] Interpreter page should clarify the % magic syntax for interpreter group.name";"[""issue"", ""bug""]";"closed";"[""fixed""]";"2013-04-21 23:52:09";"2013-05-25 20:02:08";"[""GUI"", ""Interpreters""]";"referenced_by";"Thomas";"[email protected]";"2017-05-21 12:00:00";"ZEPPELIN-332";"""issue"""
"ZEPPELIN-328";"[ZEPPELIN-328] Interpreter page should clarify the % magic syntax for interpreter group.name";"[""issue"", ""bug""]";"closed";"[""fixed""]";"2013-04-21 23:52:09";"2013-05-25 20:02:08";"[""GUI"", ""Interpreters""]";"add_link";"Thomas";"[email protected]";"2017-05-21 12:00:00";"ZEPPELIN-332";"""issue"""
"ZEPPELIN-332";"[ZEPPELIN-332] CNFE when running SQL query against Cassandra temp table";"[""issue"", ""bug""]";"open";"[""unresolved""]";"2016-07-12 16:01:30";"";"[""Interpreters""]";"referenced_by";"Thomas";"[email protected]";"2017-05-21 12:00:00";"ZEPPELIN-328";"""issue"""
"ZEPPELIN-332";"[ZEPPELIN-332] CNFE when running SQL query against Cassandra temp table";"[""issue"", ""bug""]";"open";"[""unresolved""]";"2016-07-12 16:01:30";"";"[""Interpreters""]";"add_link";"Thomas";"[email protected]";"2017-05-21 12:00:00";"ZEPPELIN-328";"""issue"""
"ZEPPELIN-332";"[ZEPPELIN-332] CNFE when running SQL query against Cassandra temp table";"[""issue"", ""bug""]";"open";"[""unresolved""]";"2016-07-12 16:01:30";"";"[""Interpreters""]";"created";"Björn";"[email protected]";"2016-07-12 16:01:30";"open";"[""unresolved""]"
"ZEPPELIN-332";"[ZEPPELIN-332] CNFE when running SQL query against Cassandra temp table";"[""issue"", ""bug""]";"open";"[""unresolved""]";"2016-07-12 16:01:30";"";"[""Interpreters""]";"commented";"Björn";"[email protected]";"2016-07-12 16:02:30";"open";"[""unresolved""]"
Expand Down
22 changes: 19 additions & 3 deletions tests/test-networks-artifact.R
View file
Original file line number Diff line number Diff line change
Expand Up @@ -117,18 +117,34 @@ patrick::with_parameters_test_that("Network construction of an issue-based artif
type = TYPE.ARTIFACT)
## 2) edges
edges = data.frame(
from = c("<issue-github-3>", "<issue-github-3>", "<issue-jira-ZEPPELIN-332>"),
to = c("<issue-github-2>", "<issue-github-6>", "<issue-jira-ZEPPELIN-328>"),
from = c("<issue-github-3>", "<issue-github-3>", "<issue-jira-ZEPPELIN-328>"),
to = c("<issue-github-2>", "<issue-github-6>", "<issue-jira-ZEPPELIN-332>"),
date = get.date.from.string(c("2016-08-07 15:30:00", "2016-08-07 15:37:02", "2017-05-21 12:00:00")),
artifact.type = c("IssueEvent", "IssueEvent", "IssueEvent"),
issue.id = c("<issue-github-3>", "<issue-github-3>", "<issue-jira-ZEPPELIN-332>"),
issue.id = c("<issue-github-3>", "<issue-github-3>", "<issue-jira-ZEPPELIN-328>"),
event.name = c("add_link", "add_link", "add_link"),
author.name = c("Thomas", "Karl", "Thomas"),
weight = c(1, 1, 1),
type = TYPE.EDGES.INTRA,
relation = "issue"
)

## 3) when constructing directed networks, we cannot deduplicate jira edges
if (test.directed) {
edges = rbind(edges, data.frame(
from = "<issue-jira-ZEPPELIN-332>",
to = "<issue-jira-ZEPPELIN-328>",
date = get.date.from.string("2017-05-21 12:00:00"),
artifact.type = "IssueEvent",
issue.id = "<issue-jira-ZEPPELIN-332>",
event.name = "add_link",
author.name = "Thomas",
weight = 1,
type = TYPE.EDGES.INTRA,
relation = "issue"
))
}

## configurations
proj.conf = ProjectConf$new(CF.DATA, CF.SELECTION.PROCESS, CASESTUDY, ARTIFACT)
proj.conf$update.value("issues.only.comments", FALSE)
Expand Down
41 changes: 21 additions & 20 deletions tests/test-networks-author.R
View file
Original file line number Diff line number Diff line change
Expand Up @@ -494,17 +494,17 @@ test_that("Network construction of the undirected author-issue network with all
rep("Olaf", 3), # <issue-github-4>
rep("Thomas", 4), rep("Karl", 3), rep("Björn", 7), rep("Olaf", 3), rep("Thomas", 3), rep("Thomas", 7),
rep("Thomas", 3), rep("Björn", 6), rep("Olaf", 2), rep("Olaf", 6), # <issue-github-6>
rep("Thomas", 10), rep("Thomas", 7), rep("Olaf", 11), # <issue-jira-ZEPPELIN-328>
rep("Björn", 6), rep("Thomas", 4), rep("Thomas", 4) # <issue-jira-ZEPPELIN-332>
rep("Thomas", 11), rep("Thomas", 8), rep("Olaf", 11), # <issue-jira-ZEPPELIN-328>
rep("Björn", 6), rep("Thomas", 5), rep("Thomas", 5) # <issue-jira-ZEPPELIN-332>
),
to = c(rep("Olaf", 5), rep("Björn", 4), rep("Björn", 3), # <issue-github-1>
rep("Björn", 4), rep("Björn", 3), rep("Olaf", 3), # <issue-github-2>
rep("Karl", 7), rep("Karl", 7), rep("Olaf", 4), # <issue-github-3>
rep("Björn", 3), # <issue-github-4>
rep("udo", 4), rep("udo", 3), rep("udo", 7), rep("udo", 3), rep("Karl", 3), rep("Björn", 7),
rep("Olaf", 3), rep("Karl", 6), rep("Karl", 2), rep("Björn", 6), # <issue-github-6>
rep("Björn", 10), rep("Olaf", 7), rep("Björn", 11), # <issue-jira-ZEPPELIN-328>
rep("Max", 6), rep("Björn", 4), rep("Max", 4) # <issue-jira-ZEPPELIN-332>
rep("Björn", 11), rep("Olaf", 8), rep("Björn", 11), # <issue-jira-ZEPPELIN-328>
rep("Max", 6), rep("Björn", 5), rep("Max", 5) # <issue-jira-ZEPPELIN-332>
),
date = get.date.from.string(c( "2016-07-12 15:59:25", "2016-07-12 15:59:25", "2016-07-12 15:59:59", # <issue-github-1>
"2016-07-12 16:01:01", "2016-07-14 13:37:00", "2016-07-12 15:59:25",
Expand Down Expand Up @@ -537,25 +537,26 @@ test_that("Network construction of the undirected author-issue network with all
"2016-10-05 15:30:02", "2016-12-07 15:30:02", "2016-12-07 15:30:02",
"2017-05-23 12:32:39", "2017-05-23 12:31:34",
"2013-04-21 23:52:09", "2013-04-21 23:52:09", "2017-05-21 12:00:00", # <issue-jira-ZEPPELIN-328>
"2013-05-05 21:46:30", "2013-05-05 21:49:21", "2013-05-05 21:49:34",
"2013-05-06 01:04:34", "2013-05-25 03:48:41", "2013-05-25 04:08:07",
"2013-06-01 06:53:06", "2013-04-21 23:52:09", "2013-04-21 23:52:09",
"2017-05-21 12:00:00", "2013-05-25 03:25:06", "2013-05-25 06:06:53",
"2013-05-25 06:22:23", "2013-06-01 06:50:26", "2013-05-05 21:46:30",
"2013-05-05 21:49:21", "2013-05-05 21:49:34", "2013-05-06 01:04:34",
"2013-05-25 03:48:41", "2013-05-25 04:08:07", "2013-06-01 06:53:06",
"2017-05-21 12:00:00", "2013-05-05 21:46:30", "2013-05-05 21:49:21",
"2013-05-05 21:49:34", "2013-05-06 01:04:34", "2013-05-25 03:48:41",
"2013-05-25 04:08:07", "2013-06-01 06:53:06", "2013-04-21 23:52:09",
"2013-04-21 23:52:09", "2017-05-21 12:00:00", "2017-05-21 12:00:00",
"2013-05-25 03:25:06", "2013-05-25 06:06:53", "2013-05-25 06:22:23",
"2013-06-01 06:50:26",
"2013-06-01 06:50:26", "2013-05-05 21:46:30", "2013-05-05 21:49:21",
"2013-05-05 21:49:34", "2013-05-06 01:04:34", "2013-05-25 03:48:41",
"2013-05-25 04:08:07", "2013-06-01 06:53:06", "2013-05-25 03:25:06",
"2013-05-25 06:06:53", "2013-05-25 06:22:23", "2013-06-01 06:50:26",
"2016-07-12 16:01:30", "2016-07-12 16:02:30", "2016-07-15 19:55:39", # <issue-jira-ZEPPELIN-332>
"2016-07-15 20:07:47", "2016-07-27 20:12:08", "2016-07-28 06:27:52",
"2016-07-12 16:01:30", "2016-07-12 16:02:30", "2016-07-15 19:55:39",
"2017-05-21 12:00:00", "2016-07-15 20:07:47", "2016-07-27 20:12:08",
"2016-07-28 06:27:52", "2017-05-21 12:00:00"
"2017-05-21 12:00:00", "2017-05-21 12:00:00", "2016-07-15 20:07:47",
"2016-07-27 20:12:08", "2016-07-28 06:27:52", "2017-05-21 12:00:00",
"2017-05-21 12:00:00"
)),
artifact.type = "IssueEvent",
issue.id = c(rep("<issue-github-1>", 12), rep("<issue-github-2>", 10), rep("<issue-github-3>", 18),
rep("<issue-github-4>", 3), rep("<issue-github-6>", 44), rep("<issue-jira-ZEPPELIN-328>", 28),
rep("<issue-jira-ZEPPELIN-332>", 14)),
rep("<issue-github-4>", 3), rep("<issue-github-6>", 44), rep("<issue-jira-ZEPPELIN-328>", 30),
rep("<issue-jira-ZEPPELIN-332>", 16)),
event.name = c("created", "commented", "state_updated", "commented", "state_updated", "created", # <issue-github-1>
"commented", "state_updated", "commented", "commented", "state_updated", "commented",
"created", "commented", "merged", "state_updated", "created", "commented", "referenced_by", # <issue-github-2>
Expand All @@ -570,12 +571,12 @@ test_that("Network construction of the undirected author-issue network with all
"subscribed", "mentioned", "subscribed", "commented", "commented", "add_link", "labeled", "referenced_by",
"mentioned", "subscribed", "mentioned", "subscribed", "commented", "referenced_by", "labeled",
"mentioned", "subscribed", "mentioned", "subscribed", "commented", "labeled",
"created", "commented", "referenced_by", "commented", "commented", "commented", "commented", # <issue-jira-ZEPPELIN-328>
"commented", "commented", "resolution_updated", "created", "commented", "referenced_by", "commented", "commented",
"commented", "commented", "commented", "commented", "commented", "commented", "commented",
"created", "commented", "referenced_by", "add_link", "commented", "commented", "commented", "commented", # <issue-jira-ZEPPELIN-328>
"commented", "commented", "resolution_updated", "created", "commented", "referenced_by", "add_link", "commented",
"commented", "commented", "commented", "commented", "commented", "commented", "commented", "commented",
"commented", "resolution_updated", "commented", "commented", "commented", "commented",
"created", "commented", "commented", "commented", "commented", "commented", "created", # <issue-jira-ZEPPELIN-332>
"commented", "commented", "add_link", "commented", "commented", "commented", "add_link"
"commented", "commented", "referenced_by", "add_link", "commented", "commented", "commented", "referenced_by", "add_link"
),
weight = 1,
type = TYPE.EDGES.INTRA,
Expand Down
Loading

0 comments on commit 7cd8828

Please sign in to comment.