EPMRPP-111704 || add method to unlock dashboard filters (#1190)

Author: grabsefx

src/main/java/com/epam/ta/reportportal/dao/DashboardRepository.java

@@ -85,4 +85,29 @@ OR id IN (SELECT widget_id FROM widget_ids)
       """, nativeQuery = true)
   void toggleDashboardLock(@Param("dashboardId") Long dashboardId, @Param("isLocked") boolean isLocked);
 
+  /**
+   * Unlocks all dashboard filters that are related to the specified dashboard
+   * and not related to any other locked dashboard.
+   *
+   * @param dashboardId id of the dashboard to unlock filters for
+   */
+  @Modifying
+  @Query(value = """
+      UPDATE owned_entity SET locked = false
+      WHERE id IN (
+        SELECT DISTINCT wf.filter_id
+        FROM widget_filter wf
+        JOIN dashboard_widget dw ON wf.widget_id = dw.widget_id
+        WHERE dw.dashboard_id = :dashboardId
+      )
+      AND id NOT IN (
+        SELECT DISTINCT wf.filter_id
+        FROM widget_filter wf
+        JOIN dashboard_widget dw ON wf.widget_id = dw.widget_id
+        JOIN owned_entity oe ON dw.dashboard_id = oe.id
+        WHERE oe.locked = true AND oe.id != :dashboardId
+      );
+      """, nativeQuery = true)
+  void unlockDashboardFilters(@Param("dashboardId") Long dashboardId);
+
 }

src/test/java/com/epam/ta/reportportal/dao/DashboardRepositoryTest.java

@@ -27,6 +27,7 @@
 import com.epam.ta.reportportal.commons.querygen.Filter;
 import com.epam.ta.reportportal.commons.querygen.FilterCondition;
 import com.epam.ta.reportportal.entity.dashboard.Dashboard;
+import jakarta.persistence.EntityManager;
 import java.util.List;
 import java.util.Optional;
 import org.junit.jupiter.api.Test;
@@ -44,32 +45,38 @@
 class DashboardRepositoryTest extends BaseTest {
 
   @Autowired
-  private DashboardRepository repository;
+  private DashboardRepository dashboardRepository;
+
+  @Autowired
+  private UserFilterRepository filterRepository;
+
+  @Autowired
+  private EntityManager entityManager;
 
   @Test
   public void shouldFindByIdAndProjectIdWhenExists() {
-    Optional<Dashboard> dashboard = repository.findByIdAndProjectId(13L, 1L);
+    Optional<Dashboard> dashboard = dashboardRepository.findByIdAndProjectId(13L, 1L);
 
     assertTrue(dashboard.isPresent());
   }
 
   @Test
   public void shouldNotFindByIdAndProjectIdWhenIdNotExists() {
-    Optional<Dashboard> dashboard = repository.findByIdAndProjectId(55L, 1L);
+    Optional<Dashboard> dashboard = dashboardRepository.findByIdAndProjectId(55L, 1L);
 
     assertFalse(dashboard.isPresent());
   }
 
   @Test
   public void shouldNotFindByIdAndProjectIdWhenProjectIdNotExists() {
-    Optional<Dashboard> dashboard = repository.findByIdAndProjectId(5L, 11L);
+    Optional<Dashboard> dashboard = dashboardRepository.findByIdAndProjectId(5L, 11L);
 
     assertFalse(dashboard.isPresent());
   }
 
   @Test
   public void shouldNotFindByIdAndProjectIdWhenIdAndProjectIdNotExist() {
-    Optional<Dashboard> dashboard = repository.findByIdAndProjectId(55L, 11L);
+    Optional<Dashboard> dashboard = dashboardRepository.findByIdAndProjectId(55L, 11L);
 
     assertFalse(dashboard.isPresent());
   }
@@ -78,7 +85,7 @@ public void shouldNotFindByIdAndProjectIdWhenIdAndProjectIdNotExist() {
   void findAllByProjectId() {
     final long superadminProjectId = 1L;
 
-    final List<Dashboard> dashboards = repository.findAllByProjectId(superadminProjectId);
+    final List<Dashboard> dashboards = dashboardRepository.findAllByProjectId(superadminProjectId);
 
     assertNotNull(dashboards, "Dashboards should not be null");
     assertEquals(4, dashboards.size(), "Unexpected dashboards size");
@@ -88,7 +95,7 @@ void findAllByProjectId() {
 
   @Test
   void shouldFindBySpecifiedNameAndProjectId() {
-    assertTrue(repository.existsByNameAndProjectId("test admin dashboard", 1L));
+    assertTrue(dashboardRepository.existsByNameAndProjectId("test admin dashboard", 1L));
   }
 
   private Filter buildDefaultFilter() {
@@ -103,21 +110,39 @@ private Filter buildDefaultFilter() {
   void shouldFindByFilterAndSortByLocked() {
     Filter filter = buildDefaultFilter();
     Pageable pageable = PageRequest.of(1, 50, Sort.by("locked"));
-    Page<Dashboard> page = repository.findByFilter(filter, pageable);
+    Page<Dashboard> page = dashboardRepository.findByFilter(filter, pageable);
     assertEquals(1, page.getTotalElements());
   }
 
   @Test
   void shouldFindByFilter() {
     Filter filter = buildDefaultFilter();
-    List<Dashboard> byFilter = repository.findByFilter(filter);
+    List<Dashboard> byFilter = dashboardRepository.findByFilter(filter);
     assertEquals(1, byFilter.size());
   }
 
   @Test
   void toggleDashboardLock() {
-    repository.toggleDashboardLock(13L, true);
-    Dashboard dashboard = repository.findById(13L).get();
+    dashboardRepository.toggleDashboardLock(13L, true);
+    Dashboard dashboard = dashboardRepository.findById(13L).get();
     assertTrue(dashboard.getLocked());
   }
+
+  @Test
+  void unlockDashboardFilters() {
+    dashboardRepository.toggleDashboardLock(13L, true);
+    dashboardRepository.toggleDashboardLock(18L, true);
+    entityManager.flush();
+    entityManager.clear();
+
+    assertTrue(filterRepository.findById(2L).get().getLocked());
+    assertTrue(filterRepository.findById(3L).get().getLocked());
+
+    dashboardRepository.unlockDashboardFilters(13L);
+    entityManager.clear();
+
+    assertFalse(filterRepository.findById(2L).get().getLocked());
+    assertTrue(filterRepository.findById(3L).get().getLocked());
+  }
+
 }