Skip to content

Commit

Permalink
refactor(struct): Remove redundant type conversions (#3345)
Browse files Browse the repository at this point in the history
  • Loading branch information
hopehook committed Oct 16, 2022
1 parent 45c758e commit 33ab0fc
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 11 deletions.
8 changes: 2 additions & 6 deletions binding/binding_test.go
Expand Up @@ -1107,9 +1107,7 @@ func testFormBindingForType(t *testing.T, method, path, badPath, body, badBody s
assert.Equal(t,
struct {
Idx int "form:\"idx\""
}(struct {
Idx int "form:\"idx\""
}{Idx: 123}),
}{Idx: 123},
obj.StructFoo)
case "StructPointer":
obj := FooStructForStructPointerType{}
Expand All @@ -1118,9 +1116,7 @@ func testFormBindingForType(t *testing.T, method, path, badPath, body, badBody s
assert.Equal(t,
struct {
Name string "form:\"name\""
}(struct {
Name string "form:\"name\""
}{Name: "thinkerou"}),
}{Name: "thinkerou"},
*obj.StructPointerFoo)
case "Map":
obj := FooStructForMapType{}
Expand Down
8 changes: 4 additions & 4 deletions binding/form_mapping_test.go
Expand Up @@ -114,7 +114,7 @@ func TestMappingPrivateField(t *testing.T) {
}
err := mappingByPtr(&s, formSource{"field": {"6"}}, "form")
assert.NoError(t, err)
assert.Equal(t, int(0), s.f)
assert.Equal(t, 0, s.f)
}

func TestMappingUnknownFieldType(t *testing.T) {
Expand All @@ -133,7 +133,7 @@ func TestMappingURI(t *testing.T) {
}
err := mapURI(&s, map[string][]string{"field": {"6"}})
assert.NoError(t, err)
assert.Equal(t, int(6), s.F)
assert.Equal(t, 6, s.F)
}

func TestMappingForm(t *testing.T) {
Expand All @@ -142,7 +142,7 @@ func TestMappingForm(t *testing.T) {
}
err := mapForm(&s, map[string][]string{"field": {"6"}})
assert.NoError(t, err)
assert.Equal(t, int(6), s.F)
assert.Equal(t, 6, s.F)
}

func TestMapFormWithTag(t *testing.T) {
Expand All @@ -151,7 +151,7 @@ func TestMapFormWithTag(t *testing.T) {
}
err := MapFormWithTag(&s, map[string][]string{"field": {"6"}}, "externalTag")
assert.NoError(t, err)
assert.Equal(t, int(6), s.F)
assert.Equal(t, 6, s.F)
}

func TestMappingTime(t *testing.T) {
Expand Down
2 changes: 1 addition & 1 deletion render/render_test.go
Expand Up @@ -173,7 +173,7 @@ func TestRenderAsciiJSON(t *testing.T) {
assert.Equal(t, "application/json", w1.Header().Get("Content-Type"))

w2 := httptest.NewRecorder()
data2 := float64(3.1415926)
data2 := 3.1415926

err = (AsciiJSON{data2}).Render(w2)
assert.NoError(t, err)
Expand Down

0 comments on commit 33ab0fc

Please sign in to comment.