★ wanayoo — archive 1999 https://github.com/nodejs/node/commit/8849d8073aNouvelle recherche | Portail wanayoo
Skip to content
Permalink
Browse files

test: cover path.basename when path and ext are the same

In path.basename was a case when the path
and the extension is the same and this wasn't
covered with tests.

I covered this case both in Windows and Unix environments.

PR-URL: #24570
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
  • Loading branch information...
moczolaszlo authored and rvagg committed Nov 22, 2018
1 parent 12d7107 commit 8849d8073a26cf7fa769b6d51dfdd683b3ec2358
Showing with 2 additions and 0 deletions.
  1. +2 −0 test/parallel/test-path-basename.js
@@ -29,6 +29,7 @@ assert.strictEqual(path.basename('/aaa/'), 'aaa');
assert.strictEqual(path.basename('/aaa/b'), 'b');
assert.strictEqual(path.basename('/a/b'), 'b');
assert.strictEqual(path.basename('//a'), 'a');
assert.strictEqual(path.basename('a', 'a'), '');

// On Windows a backslash acts as a path separator.
assert.strictEqual(path.win32.basename('\\dir\\basename.ext'), 'basename.ext');
@@ -53,6 +54,7 @@ assert.strictEqual(path.win32.basename('C:basename.ext\\'), 'basename.ext');
assert.strictEqual(path.win32.basename('C:basename.ext\\\\'), 'basename.ext');
assert.strictEqual(path.win32.basename('C:foo'), 'foo');
assert.strictEqual(path.win32.basename('file:stream'), 'file:stream');
assert.strictEqual(path.win32.basename('a', 'a'), '');

// On unix a backslash is just treated as any other character.
assert.strictEqual(path.posix.basename('\\dir\\basename.ext'),

0 comments on commit 8849d80

Please sign in to comment.
You can’t perform that action at this time.