Skip to content

Commit 5e7d0d5

Browse files
authored
Merge pull request #135 from RandomAPI/bugfix/rug-133
Remove spaces from email address #133
2 parents 33d943d + 4e2a347 commit 5e7d0d5

2 files changed

Lines changed: 14 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]) + '.' + tr.transliterate(name[1]).replace(/ /g, '') + '@example.com');
173+
this.include('email', tr.transliterate(`${name[0]}.${name[1]}`).replace(/ /g, '') + '@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: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,19 @@ const defVersion = "1.3";
55

66
module.exports = (server, version=defVersion) => {
77

8+
describe(`Email address formatting`, () => {
9+
10+
// No spaces in email address
11+
it(`should not include spaces in email address`, async () => {
12+
const res = await request(server).get(`/api/${version}?results=5000&inc=email`);
13+
const result = JSON.parse(res.text);
14+
15+
expect(result.results.every(user => {
16+
return user.email.indexOf(' ') === -1;
17+
})).to.equal(true);
18+
});
19+
});
20+
821
// 1.3 should support all previous version features as well
922
describe(`Checking support of previous version features`, () => {
1023
describe('1.2', require('./1.2').bind(this, server, version));

0 commit comments

Comments
 (0)