Fix default issue state coming from home page (#2268)

erda-project · Oct 14, 2021 · 25b024d · 25b024d
1 parent 6039b57
commit 25b024d
Show file tree

Hide file tree

Showing 2 changed files with 46 additions and 2 deletions.
diff --git a/modules/dop/component-protocol/components/issue-manage/issueFilter/render.go b/modules/dop/component-protocol/components/issue-manage/issueFilter/render.go
@@ -245,13 +245,18 @@ func (f *ComponentFilter) InitDefaultOperation(ctx context.Context, state State)
 		if err := f.flushOptsByFilter(filterID, f.InParams.FrontendUrlQuery); err != nil {
 			return err
 		}
-	} else {
-		f.State.FrontendConditionValues.States = res[f.InParams.FrontendFixedIssueType]
 	}
 
+	f.setDefaultState(res, f.InParams.FrontendFixedIssueType)
 	return nil
 }
 
+func (f *ComponentFilter) setDefaultState(stateMap map[string][]int64, key string) {
+	if f.State.FrontendConditionValues.States == nil {
+		f.State.FrontendConditionValues.States = stateMap[key]
+	}
+}
+
 func (f *ComponentFilter) determineFilterID(filterEntity string) string {
 	for _, bm := range f.Bms {
 		if bm.FilterEntity == filterEntity {

diff --git a/modules/dop/component-protocol/components/issue-manage/issueFilter/render_test.go b/modules/dop/component-protocol/components/issue-manage/issueFilter/render_test.go
@@ -83,3 +83,42 @@ func Test_determineFilterID(t *testing.T) {
 	}
 	assert.Equal(t, "123", f.determineFilterID("eyJzdGF0ZXMiOls2NTddLCJhc3NpZ25lZUlEcyI6WyIyIl19"))
 }
+
+func TestComponentFilter_setDefaultState(t *testing.T) {
+	type args struct {
+		stateMap map[string][]int64
+		key      string
+	}
+	tests := []struct {
+		name string
+		args args
+	}{
+		{
+			name: "test",
+			args: args{
+				stateMap: map[string][]int64{
+					"t": {1},
+				},
+				key: "t",
+			},
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			f := &ComponentFilter{}
+			f.setDefaultState(tt.args.stateMap, tt.args.key)
+			assert.Equal(t, []int64{1}, f.State.FrontendConditionValues.States)
+		})
+		t.Run(tt.name, func(t *testing.T) {
+			f := &ComponentFilter{
+				State: State{
+					FrontendConditionValues: FrontendConditions{
+						States: []int64{2},
+					},
+				},
+			}
+			f.setDefaultState(tt.args.stateMap, tt.args.key)
+			assert.Equal(t, []int64{2}, f.State.FrontendConditionValues.States)
+		})
+	}
+}