Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
configuration entity system
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
16 Jan 2013 at 06:05 UTC
Updated:
29 Jul 2014 at 21:46 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
tim.plunkettOkay, this needs a test using the config_test config entity, so I'll work on that, but at least I have something.
Comment #3
tim.plunkettOh, that actually wasn't that hard.
This is a pretty big one, since Views cannot be updated at all.
Comment #5
tim.plunkettHad to adjust the expectations of other tests, and reorder the properties.
Comment #7
tim.plunkettAhh, had to adjust for the config entity query commits.
Comment #8
eclipsegc commentedeyup
Comment #9
sunWhy is this part of the UI test?
Let's move it into the API import test.
Can we rename this property to 'protected_property', please?
Comment #10
sunAlso, can we make sure to check for existing issues first? :)
Marked #1892558: Fatal error when importing a view, using set() rather than accessing properties directly might work as duplicate.
Comment #11
tim.plunkettIt turned out we already had test coverage for this in ConfigImportTest, and my addition to ConfigImportUITest was redundant (I put it in there initially because I was writing test coverage to mirror my interaction in the UI).
I also did s/description/protected_property, that's a nice clarification.
Comment #12
tim.plunkettThat was tagged neither Configurables nor VDC, I can't look everywhere.
Comment #13
sunThanks, looks great. :)
Comment #15
tim.plunkettRetested through qa.d.o
Comment #16
webchickCommitted and pushed to 8.x. Thanks!