aboutsummaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorTom Collier <[email protected]>2015-07-30 09:42:27 -0700
committerTom Collier <[email protected]>2015-07-30 09:43:26 -0700
commitcab6575c7378cc8f1e6426acc47fac09b415e21b (patch)
tree3c46467e5bba038ee2b4215bf30230c693edbbee /test
parentac4ad6dec9ef34fa3d76d4bc49b95a13ccbc5555 (diff)
downloadfaker-cab6575c7378cc8f1e6426acc47fac09b415e21b.tar.xz
faker-cab6575c7378cc8f1e6426acc47fac09b415e21b.zip
Only append suffix to streetName if suffix is present
When streetSuffix returns an empty string, do not append a space to the end of the name part of streetName. With the prior behavior, occasionally, this streetName would return something like "Ashley " (notice the trailing space), which is undesirable in many circumstances.
Diffstat (limited to 'test')
-rw-r--r--test/address.unit.js13
1 files changed, 12 insertions, 1 deletions
diff --git a/test/address.unit.js b/test/address.unit.js
index 1ea76452..b5df1d24 100644
--- a/test/address.unit.js
+++ b/test/address.unit.js
@@ -73,7 +73,6 @@ describe("address.js", function () {
});
afterEach(function () {
- faker.random.number.restore();
faker.name.firstName.restore();
faker.name.lastName.restore();
faker.address.streetSuffix.restore();
@@ -87,6 +86,8 @@ describe("address.js", function () {
assert.ok(!faker.name.firstName.called);
assert.ok(faker.name.lastName.calledOnce);
assert.ok(faker.address.streetSuffix.calledOnce);
+
+ faker.random.number.restore();
});
it("occasionally returns first name + suffix", function () {
@@ -98,6 +99,16 @@ describe("address.js", function () {
assert.ok(faker.name.firstName.calledOnce);
assert.ok(!faker.name.lastName.called);
assert.ok(faker.address.streetSuffix.calledOnce);
+
+ faker.random.number.restore();
+ });
+
+ it("trims trailing whitespace from the name", function() {
+ faker.address.streetSuffix.restore();
+
+ sinon.stub(faker.address, 'streetSuffix').returns("")
+ var street_name = faker.address.streetName();
+ assert.ok(!street_name.match(/ $/));
});
});