ADM-898[backend][frontend]: feat: block' in board mapping is mutually exclusive with 'flag as block'

backend/src/main/java/heartbeat/service/board/jira/JiraService.java

@@ -844,7 +844,8 @@ private CycleTimeInfoDTO getCycleTime(CardHistoryResponseDTO cardHistoryResponse
 				keyFlagged);
 		List<CycleTimeInfo> cycleTimeInfos = boardUtil.getCycleTimeInfos(statusChangedArray, realDoneStatus,
 				treatFlagCardAsBlock);
-		List<CycleTimeInfo> originCycleTimeInfos = boardUtil.getOriginCycleTimeInfos(statusChangedArray);
+		List<CycleTimeInfo> originCycleTimeInfos = boardUtil.getOriginCycleTimeInfos(statusChangedArray,
+				treatFlagCardAsBlock);
 
 		return CycleTimeInfoDTO.builder()
 			.cycleTimeInfos(cycleTimeInfos)

backend/src/main/java/heartbeat/util/BoardUtil.java

@@ -20,10 +20,12 @@ public class BoardUtil {
 
 	private final WorkDay workDay;
 
-	public List<CycleTimeInfo> getOriginCycleTimeInfos(List<StatusChangedItem> statusChangedArray) {
+	public List<CycleTimeInfo> getOriginCycleTimeInfos(List<StatusChangedItem> statusChangedArray,
+			Boolean treatFlagCardAsBlock) {
 		List<StatusTimeStamp> flagTimeStamp = getFlagTimeStamps(statusChangedArray);
 		List<StatusTimeStamp> columnTimeStamp = getColumnTimeStamps(statusChangedArray);
-		List<CycleTimeInfo> originCycleTimeInfos = calculateOriginCycleTime(flagTimeStamp, columnTimeStamp);
+		List<CycleTimeInfo> originCycleTimeInfos = calculateOriginCycleTime(flagTimeStamp, columnTimeStamp,
+				treatFlagCardAsBlock);
 		return getCollectRemovedDuplicates(originCycleTimeInfos);
 	}
 
@@ -50,7 +52,7 @@ public List<CycleTimeInfo> getCycleTimeInfos(List<StatusChangedItem> statusChang
 	}
 
 	private List<CycleTimeInfo> calculateOriginCycleTime(List<StatusTimeStamp> flagTimeStamp,
-			List<StatusTimeStamp> columnTimeStamp) {
+			List<StatusTimeStamp> columnTimeStamp, Boolean treatFlagCardAsBlock) {
 		List<CycleTimeInfo> originCycleTimeInfos = new ArrayList<>();
 
 		for (StatusTimeStamp columnTimeStampItem : columnTimeStamp) {
@@ -62,9 +64,11 @@ private List<CycleTimeInfo> calculateOriginCycleTime(List<StatusTimeStamp> flagT
 				.build());
 		}
 
-		double totalFlagTimeInDays = calculateTotalFlagCycleTime(flagTimeStamp);
-		originCycleTimeInfos
-			.add(CycleTimeInfo.builder().day(totalFlagTimeInDays).column(CardStepsEnum.FLAG.getValue()).build());
+		if (Boolean.TRUE.equals(treatFlagCardAsBlock)) {
+			double totalFlagTimeInDays = calculateTotalFlagCycleTime(flagTimeStamp);
+			originCycleTimeInfos
+				.add(CycleTimeInfo.builder().day(totalFlagTimeInDays).column(CardStepsEnum.FLAG.getValue()).build());
+		}
 
 		return originCycleTimeInfos;
 	}
@@ -101,10 +105,8 @@ private List<CycleTimeInfo> calculateCycleTime(List<String> realDoneStatus, List
 		}
 		if (!isBlockColumnExisted(columnTimeStamp) && totalFlagTimeInDays > 0) {
 			double blockDays = totalFlagTimeInDays - totalFlagAndRealDoneOverlapTime;
-			cycleTimeInfos.add(CycleTimeInfo.builder()
-				.day(blockDays)
-				.column(CardStepsEnum.BLOCK.getValue().toUpperCase())
-				.build());
+			cycleTimeInfos.add(
+					CycleTimeInfo.builder().day(blockDays).column(CardStepsEnum.FLAG.getValue().toUpperCase()).build());
 		}
 
 		return cycleTimeInfos;

backend/src/test/java/heartbeat/service/jira/JiraServiceTest.java

@@ -1166,7 +1166,7 @@ public void shouldProcessCustomFieldsForCardsWhenCallGetStoryPointsAndCycleTime(
 		when(jiraFeignClient.getJiraCardHistoryByCount(any(), any(), anyInt(), anyInt(), any()))
 			.thenReturn(CARD_HISTORY_MULTI_RESPONSE_BUILDER().build());
 		when(boardUtil.getCycleTimeInfos(any(), any(), any())).thenReturn(CYCLE_TIME_INFO_LIST());
-		when(boardUtil.getOriginCycleTimeInfos(any())).thenReturn(CYCLE_TIME_INFO_LIST());
+		when(boardUtil.getOriginCycleTimeInfos(any(), any())).thenReturn(CYCLE_TIME_INFO_LIST());
 
 		CardCollection doneCards = jiraService.getStoryPointsAndCycleTimeAndReworkInfoForDoneCards(
 				storyPointsAndCycleTimeRequest, jiraBoardSetting.getBoardColumns(), List.of("Zhang San"), "");

backend/src/test/java/heartbeat/util/BoardUtilTest.java

@@ -81,16 +81,29 @@ void calculateCycleTimeWhenTreatFlagCardAsBlockIsFalse() {
 	}
 
 	@Test
-	void calculateOriginCycleTimeOfColumn() {
+	void shouldCalculateOriginCycleTimeGivenTreatFlagCardAsBlockIsTrue() {
 		List<StatusChangedItem> statusChangedItems = StatusChangedItemsListAndCycleTimeInfosListFixture
 			.STATUS_CHANGED_ITEMS_LIST_OF_ORIGIN();
 		List<CycleTimeInfo> statusChangedItemsExpect = StatusChangedItemsListAndCycleTimeInfosListFixture
 			.CYCLE_TIME_INFOS_LIST_OF_ORIGIN();
 
 		when(workDay.calculateWorkDaysBy24Hours(anyLong(), anyLong()))
 			.thenReturn(StatusChangedItemsListAndCycleTimeInfosListFixture.EXPECT_DAYS);
-		List<CycleTimeInfo> result = boardUtil.getOriginCycleTimeInfos(statusChangedItems);
+		List<CycleTimeInfo> result = boardUtil.getOriginCycleTimeInfos(statusChangedItems, Boolean.TRUE);
 		Assertions.assertEquals(statusChangedItemsExpect, result);
 	}
 
+	@Test
+	void shouldCalculateOriginCycleTimeGivenTreatFlagCardAsBlockIsFalse() {
+		List<StatusChangedItem> statusChangedItems = StatusChangedItemsListAndCycleTimeInfosListFixture
+			.STATUS_CHANGED_ITEMS_LIST_OF_ORIGIN();
+		List<CycleTimeInfo> statusChangedItemsWithoutFlagExpect = StatusChangedItemsListAndCycleTimeInfosListFixture
+			.CYCLE_TIME_INFOS_LIST_OF_ORIGIN_WITHOUT_FLAG();
+
+		when(workDay.calculateWorkDaysBy24Hours(anyLong(), anyLong()))
+			.thenReturn(StatusChangedItemsListAndCycleTimeInfosListFixture.EXPECT_DAYS);
+		List<CycleTimeInfo> result = boardUtil.getOriginCycleTimeInfos(statusChangedItems, Boolean.FALSE);
+		Assertions.assertEquals(statusChangedItemsWithoutFlagExpect, result);
+	}
+
 }

backend/src/test/java/heartbeat/util/StatusChangedItemsListAndCycleTimeInfosListFixture.java

@@ -18,8 +18,8 @@ public static List<StatusChangedItem> STATUS_CHANGED_ITEMS_LIST_OF_REAL_DONE_COL
 
 	public static List<CycleTimeInfo> CYCLE_TIME_INFOS_LIST_OF_REAL_DONE_COLUMN() {
 		return List.of(CycleTimeInfo.builder().column("DONE").day(EXPECT_DAYS).build(),
-				CycleTimeInfo.builder().column("BLOCK").day(0.0).build(),
-				CycleTimeInfo.builder().column("IN PROGRESS").day(EXPECT_DAYS).build());
+				CycleTimeInfo.builder().column("IN PROGRESS").day(EXPECT_DAYS).build(),
+				CycleTimeInfo.builder().column("FLAG").day(0.0).build());
 	}
 
 	public static List<StatusChangedItem> STATUS_CHANGED_ITEMS_LIST_OF_BLOCK_COLUMN() {
@@ -79,4 +79,10 @@ public static List<CycleTimeInfo> CYCLE_TIME_INFOS_LIST_OF_ORIGIN() {
 				CycleTimeInfo.builder().column("FLAG").day(EXPECT_DAYS).build());
 	}
 
+	public static List<CycleTimeInfo> CYCLE_TIME_INFOS_LIST_OF_ORIGIN_WITHOUT_FLAG() {
+		return List.of(CycleTimeInfo.builder().column("DONE").day(EXPECT_DAYS).build(),
+				CycleTimeInfo.builder().column("BLOCK").day(EXPECT_DAYS).build(),
+				CycleTimeInfo.builder().column("IN PROGRESS").day(EXPECT_DAYS).build());
+	}
+
 }

frontend/__tests__/containers/MetricsStep/CycleTime.test.tsx

@@ -6,7 +6,7 @@ import {
   updateTreatFlagCardAsBlock,
 } from '@src/context/Metrics/metricsSlice';
 import { BOARD_MAPPING, ERROR_MESSAGE_TIME_DURATION, LIST_OPEN, NO_RESULT_DASH } from '../../fixtures';
-import { CYCLE_TIME_SETTINGS_TYPES, METRICS_CONSTANTS } from '@src/constants/resources';
+import { CYCLE_TIME_SETTINGS_TYPES, MESSAGE, METRICS_CONSTANTS } from '@src/constants/resources';
 import { act, render, screen, waitFor, within } from '@testing-library/react';
 import { CycleTime } from '@src/containers/MetricsStep/CycleTime';
 import { setupStore } from '../../utils/setupStoreUtil';
@@ -406,4 +406,38 @@ describe('CycleTime', () => {
       expect(mockedUseAppDispatch).not.toHaveBeenCalledWith(saveDoneColumn([]));
     });
   });
+
+  it('should show warning message when both mapping block column and add flag as block Given mapping relationship by status', () => {
+    (selectMetricsContent as jest.Mock).mockReturnValue({
+      cycleTimeSettingsType: CYCLE_TIME_SETTINGS_TYPES.BY_STATUS,
+      cycleTimeSettings: [
+        ...cycleTimeSettings,
+        {
+          column: 'Blocked',
+          status: 'BLOCKED',
+          value: 'Block',
+        },
+      ],
+    });
+    setup();
+
+    expect(screen.getByText(MESSAGE.FLAG_CARD_DROPPED_WARNING)).toBeVisible();
+  });
+
+  it('should show warning message when both mapping block column and add flag as block Given mapping relationship by column', () => {
+    (selectMetricsContent as jest.Mock).mockReturnValue({
+      cycleTimeSettingsType: CYCLE_TIME_SETTINGS_TYPES.BY_COLUMN,
+      cycleTimeSettings: [
+        ...cycleTimeSettings,
+        {
+          column: 'Blocked',
+          status: 'BLOCKED',
+          value: 'Block',
+        },
+      ],
+    });
+    setup();
+
+    expect(screen.getByText(MESSAGE.FLAG_CARD_DROPPED_WARNING)).toBeVisible();
+  });
 });

frontend/__tests__/containers/MetricsStep/MetricsStep.test.tsx

@@ -217,7 +217,7 @@ describe('MetricsStep', () => {
 
       await userEvent.click(doneSelectTrigger as HTMLInputElement);
 
-      const noneOption = within(screen.getByRole('presentation')).getByText('----');
+      const noneOption = within(screen.getAllByRole('presentation')[1]).getByText('----');
       await userEvent.click(noneOption);
 
       expect(realDoneSettingSection).toHaveTextContent(SELECT_CONSIDER_AS_DONE_MESSAGE);

frontend/__tests__/context/metricsSlice.test.ts

@@ -61,6 +61,7 @@ const initState = {
   classification: [],
   treatFlagCardAsBlock: true,
   assigneeFilter: ASSIGNEE_FILTER_TYPES.LAST_ASSIGNEE,
+  displayFlagCardDropWarning: false,
   importedData: {
     importedCrews: [],
     importedAssigneeFilter: ASSIGNEE_FILTER_TYPES.LAST_ASSIGNEE,

frontend/__tests__/fixtures.ts

@@ -467,7 +467,6 @@ export const MOCK_REPORT_RESPONSE: ReportResponseDTO = {
     fromAnalysis: null,
     fromInDev: null,
     fromBlock: 111,
-    fromFlag: null,
     fromReview: 111,
     fromWaitingForTesting: 111,
     fromTesting: null,

frontend/__tests__/hooks/reportMapper/report.test.tsx

@@ -171,7 +171,7 @@ export const EXPECTED_REPORT_VALUES = {
       ],
     },
     {
-      id: 5,
+      id: 4,
       name: (
         <React.Fragment>
           From <BoldText>review</BoldText> to<BoldText> in dev</BoldText>
@@ -185,7 +185,7 @@ export const EXPECTED_REPORT_VALUES = {
       ],
     },
     {
-      id: 6,
+      id: 5,
       name: (
         <React.Fragment>
           From <BoldText>waiting for testing</BoldText> to<BoldText> in dev</BoldText>
@@ -199,7 +199,7 @@ export const EXPECTED_REPORT_VALUES = {
       ],
     },
     {
-      id: 8,
+      id: 7,
       name: (
         <React.Fragment>
           From <BoldText>done</BoldText> to<BoldText> in dev</BoldText>
@@ -213,7 +213,7 @@ export const EXPECTED_REPORT_VALUES = {
       ],
     },
     {
-      id: 9,
+      id: 8,
       name: <React.Fragment>Total rework cards</React.Fragment>,
       valueList: [
         {
@@ -223,7 +223,7 @@ export const EXPECTED_REPORT_VALUES = {
       ],
     },
     {
-      id: 10,
+      id: 9,
       name: <React.Fragment>Rework cards ratio</React.Fragment>,
       valueList: [
         {