Tests will be broken after default mapping is in FeedAPI Node

Aron Novak - August 24, 2009 - 14:54
Project:Feed Element Mapper
Version:6.x-2.x-dev
Component:Code
Category:task
Priority:critical
Assigned:Unassigned
Status:closed
Description

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.

#1

Aron Novak - August 25, 2009 - 12:07
Status:active» needs review

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

#2

Aron Novak - August 25, 2009 - 12:08

The file was missing :)

AttachmentSize
558028_not_add_default_mapping.patch 5.4 KB

#3

alex_b - September 17, 2009 - 21:14

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

Aron Novak - September 18, 2009 - 13:19

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

Aron Novak - September 18, 2009 - 13:20
Status:needs review» fixed

committed.

#6

System Message - October 2, 2009 - 13:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.