@Override
// @Secured TODO: we need to handle this property, since getTestByRelation will not work as metric
// is associated with more tests
public void removeMetric(Metric metric, Test test) throws ServiceException {
Metric freshMetric = metricDAO.get(metric.getId());
Test freshTest = testDAO.get(test.getId());
// List<Test> newTests = freshMetric.getTests().stream().filter(o ->
// !o.equals(freshTest)).collect(Collectors.toList());
// freshMetric.setTests(newTests);
// List<Metric> newMetrics = freshTest.getMetrics().stream().filter(o ->
// !o.equals(freshMetric)).collect(Collectors.toList());
// freshTest.setMetrics(newMetrics);
freshMetric.getTests().remove(freshTest);
freshTest.getMetrics().remove(freshMetric);
metricDAO.update(freshMetric);
testDAO.update(freshTest);
if (freshMetric.getTests() == null || freshMetric.getTests().isEmpty()) {
metricDAO.remove(freshMetric);
}
}
