Skip to content

Commit 8ecc42d

Browse files
committed
confirms issue 1024 is resolveD
1 parent 58bcb73 commit 8ecc42d

File tree

1 file changed

+23
-1
lines changed

1 file changed

+23
-1
lines changed

tests/test-cases.js

Lines changed: 23 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -641,6 +641,7 @@ var CORE_PARSER_TESTS = [
641641
}
642642
}
643643
},
644+
644645
];
645646

646647
describe('Core Parser Tests', function() {
@@ -1652,7 +1653,8 @@ var PARSE_TESTS = [
16521653
data: [['1','2','3','4']],
16531654
errors: []
16541655
}
1655-
}
1656+
},
1657+
16561658
];
16571659

16581660
describe('Parse Tests', function() {
@@ -1671,6 +1673,26 @@ describe('Parse Tests', function() {
16711673
for (var i = 0; i < PARSE_TESTS.length; i++) {
16721674
generateTest(PARSE_TESTS[i]);
16731675
}
1676+
1677+
// Custom test for Issue 1024 - renamedHeaders regression test
1678+
it('Issue 1024: renamedHeaders returned for simple duplicate headers (regression test)', function() {
1679+
var result = Papa.parse('Column,Column\n1-1,1-2\n2-1,2-2\n3-1,3-2', { header: true });
1680+
1681+
// Test data structure
1682+
assert.deepEqual(result.data, [
1683+
{Column: '1-1', Column_1: '1-2'},
1684+
{Column: '2-1', Column_1: '2-2'},
1685+
{Column: '3-1', Column_1: '3-2'}
1686+
]);
1687+
1688+
// Test errors
1689+
assert.deepEqual(result.errors, []);
1690+
1691+
// Test that renamedHeaders is present and correct
1692+
assert.isNotNull(result.meta.renamedHeaders, 'renamedHeaders should not be null');
1693+
assert.isObject(result.meta.renamedHeaders, 'renamedHeaders should be an object');
1694+
assert.deepEqual(result.meta.renamedHeaders, {Column_1: 'Column'}, 'renamedHeaders should contain the renamed header mapping');
1695+
});
16741696
});
16751697

16761698

0 commit comments

Comments
 (0)