Skip to content

Commit 8084637

Browse files
authored
Merge pull request #136 from RandomAPI/bugfix/rug-134
Email addresses are lowercase
2 parents 5e7d0d5 + fe2c650 commit 8084637

2 files changed

Lines changed: 11 additions & 1 deletion

File tree

api/1.3/api.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ class Generator {
170170
timezone
171171
});
172172

173-
this.include('email', tr.transliterate(`${name[0]}.${name[1]}`).replace(/ /g, '') + '@example.com');
173+
this.include('email', tr.transliterate(`${name[0]}.${name[1]}`).replace(/ /g, '').toLowerCase() + '@example.com');
174174

175175
let salt = random(2, 8);
176176
let password = this.password === undefined ? randomItem(this.datasets.common.passwords) : this.genPassword();

spec/api/modern/1.3.js

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,16 @@ module.exports = (server, version=defVersion) => {
1616
return user.email.indexOf(' ') === -1;
1717
})).to.equal(true);
1818
});
19+
20+
// Email addresses should be lowercase
21+
it(`should be lowercase`, async () => {
22+
const res = await request(server).get(`/api/${version}?results=5000&inc=email`);
23+
const result = JSON.parse(res.text);
24+
25+
expect(result.results.every(user => {
26+
return user.email.toLowerCase() === user.email;
27+
})).to.equal(true);
28+
});
1929
});
2030

2131
// 1.3 should support all previous version features as well

0 commit comments

Comments
 (0)