Skip to content

Commit

Permalink
Refactor tests related to Person Gender and created tests for remaini…
Browse files Browse the repository at this point in the history
…ng one

Signed-off-by: Jonathan Schweder <jonathanschweder@gmail.com>
  • Loading branch information
jaswdr committed Oct 10, 2020
1 parent 6bf2a47 commit 46994ea
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 5 deletions.
4 changes: 1 addition & 3 deletions person.go
Original file line number Diff line number Diff line change
Expand Up @@ -100,8 +100,6 @@ var (
"Zboncak", "Zemlak", "Ziemann", "Zieme", "Zulauf"}

suffix = []string{"Jr.", "Sr.", "I", "II", "III", "IV", "V", "MD", "DDS", "PhD", "DVM"}

gender = []string{"Male", "Female"}
)

// Person is a faker struct for Person
Expand Down Expand Up @@ -217,7 +215,7 @@ func (p Person) NameFemale() string {

// Gender returns a fake Gender for Person
func (p Person) Gender() string {
return p.Faker.RandomStringElement(gender)
return p.Faker.RandomStringElement([]string{p.GenderMale(), p.GenderFemale()})
}

// NameAndGender returns a fake NameAndGender for Person
Expand Down
12 changes: 10 additions & 2 deletions person_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -53,15 +53,13 @@ func TestLastName(t *testing.T) {
func TestName(t *testing.T) {
p := New().Person()
name := p.Name()
gender := p.Gender()
Expect(t, true, len(name) > 0)
Expect(t, false, strings.Contains(name, "{{titleMale}}"))
Expect(t, false, strings.Contains(name, "{{firstNameMale}}"))
Expect(t, false, strings.Contains(name, "{{titleFemale}}"))
Expect(t, false, strings.Contains(name, "{{firstNameFemale}}"))
Expect(t, false, strings.Contains(name, "{{lastName}}"))
Expect(t, false, strings.Contains(name, "{{suffix}}"))
Expect(t, true, gender == "Male" || gender == "Female")
}

func TestGender(t *testing.T) {
Expand All @@ -70,6 +68,16 @@ func TestGender(t *testing.T) {
Expect(t, true, gender == "Male" || gender == "Female")
}

func TestGenderMale(t *testing.T) {
p := New().Person()
Expect(t, "Male", p.GenderMale())
}

func TestGenderFemale(t *testing.T) {
p := New().Person()
Expect(t, "Female", p.GenderFemale())
}

func TestNameAndGender(t *testing.T) {
p := New().Person()
name, gender := p.NameAndGender()
Expand Down

0 comments on commit 46994ea

Please sign in to comment.