Problem/Motivation

Behaviour for FeedsCSVParser::getTemplate() is not yet covered. As there are sometimes feature request and bug reports for it, it would be good to have some test covering for it.

Issues for the CSV template:

Proposed resolution

Implement a test for it, covering the following cases:

  1. Each delimiter: ,;TAB|+. When using the delimiter in a source key, that source key should be wrapped with double quotes.
  2. Multiple mappings of the same source key: each source key should be printed once in the CSV template.
  3. Special characters in source keys: & should not be converted to & for example.

Patch will follow.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MegaChriz created an issue. See original summary.

MegaChriz’s picture

Assigned: MegaChriz » Unassigned
Status: Active » Needs review
FileSize
5.56 KB

  • MegaChriz committed b7884a5 on 7.x-2.x
    Issue #2771803 by MegaChriz: Added some tests for the CSV template.
    
MegaChriz’s picture

Issue summary: View changes
Status: Needs review » Fixed

Great, tests passed!

Committed #2.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.