Download & Extend
Feed Element Mapper » Issues

Tests will be broken after default mapping is in FeedAPI Node

Posted by Aron Novak on August 24, 2009 at 2:54pm
Project:Feed Element Mapper
Version:6.x-2.x-dev
Component:Code
Category:task
Priority:critical
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Let's patch test that it should not try to add already existing mapping entries. Or let's override the mapping first and delete everything.

Login or register to post comments

Comments

#1

Posted by Aron Novak on August 25, 2009 at 12:07pm
Status:active» needs review

Only commit after #557512: Add default mapping for FEMP 2.x

#2

Posted by Aron Novak on August 25, 2009 at 12:08pm

The file was missing :)

AttachmentSize
558028_not_add_default_mapping.patch 5.4 KB

#3

Posted by alex_b on September 17, 2009 at 9:14pm

Could you explain _why_ #557512: Add default mapping for FEMP 2.x breaks tests and makes this patch necessary? Otherwise that's looking good.

#4

Posted by Aron Novak on September 18, 2009 at 1:19pm

Yes, because it tries to add an already existing mapping, the option is not even the dropdown list. This definitely fails then.
Going to commit this patch.

#5

Posted by Aron Novak on September 18, 2009 at 1:20pm
Status:needs review» fixed

committed.

#6

Posted by System Message on October 2, 2009 at 1:30pm
Status:fixed» closed (fixed)

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

nobody click here