Cleanup case tests.

This commit is contained in:
jdalton
2015-02-01 00:12:27 -08:00
parent fe7f082083
commit 0cd942f152

View File

@@ -1453,7 +1453,7 @@
'FOO BAR', 'fooBar', '--foo-bar', '__foo_bar__' 'FOO BAR', 'fooBar', '--foo-bar', '__foo_bar__'
]; ];
var expected = (function() { var converted = (function() {
switch (caseName) { switch (caseName) {
case 'camel': return 'fooBar'; case 'camel': return 'fooBar';
case 'kebab': return 'foo-bar'; case 'kebab': return 'foo-bar';
@@ -1464,11 +1464,8 @@
test('`_.' + methodName + '` should convert `string` to ' + caseName + ' case', 1, function() { test('`_.' + methodName + '` should convert `string` to ' + caseName + ' case', 1, function() {
var actual = _.map(strings, function(string) { var actual = _.map(strings, function(string) {
if (caseName === 'start' && string === 'FOO BAR') { var expected = (caseName === 'start' && string === 'FOO BAR') ? string : converted;
return func(string) === 'FOO BAR'; return func(string) === expected;
} else {
return func(string) === expected;
}
}); });
deepEqual(actual, _.map(strings, _.constant(true))); deepEqual(actual, _.map(strings, _.constant(true)));
@@ -1476,11 +1473,8 @@
test('`_.' + methodName + '` should handle double-converting strings', 1, function() { test('`_.' + methodName + '` should handle double-converting strings', 1, function() {
var actual = _.map(strings, function(string) { var actual = _.map(strings, function(string) {
if (caseName === 'start' && string === 'FOO BAR') { var expected = (caseName === 'start' && string === 'FOO BAR') ? string : converted;
return func(func(string)) === 'FOO BAR'; return func(func(string)) === expected;
} else {
return func(func(string)) === expected;
}
}); });
deepEqual(actual, _.map(strings, _.constant(true))); deepEqual(actual, _.map(strings, _.constant(true)));
@@ -1488,8 +1482,8 @@
test('`_.' + methodName + '` should deburr letters', 1, function() { test('`_.' + methodName + '` should deburr letters', 1, function() {
var actual = _.map(burredLetters, function(burred, index) { var actual = _.map(burredLetters, function(burred, index) {
var letter = deburredLetters[index].toLowerCase(); var letter = deburredLetters[index];
letter = caseName === 'start' ? _.capitalize(letter) : letter; letter = caseName == 'start' ? _.capitalize(letter) : letter.toLowerCase();
return func(burred) === letter; return func(burred) === letter;
}); });
@@ -1502,14 +1496,14 @@
}); });
test('`_.' + methodName + '` should coerce `string` to a string', 2, function() { test('`_.' + methodName + '` should coerce `string` to a string', 2, function() {
var string = 'Foo Bar'; var string = 'foo bar';
strictEqual(func(Object(string)), expected); strictEqual(func(Object(string)), converted);
strictEqual(func({ 'toString': _.constant(string) }), expected); strictEqual(func({ 'toString': _.constant(string) }), converted);
}); });
test('`_.' + methodName + '` should return an unwrapped value when chaining', 1, function() { test('`_.' + methodName + '` should return an unwrapped value when chaining', 1, function() {
if (!isNpm) { if (!isNpm) {
strictEqual(_('foo bar')[methodName](), expected); strictEqual(_('foo bar')[methodName](), converted);
} }
else { else {
skipTest(); skipTest();