1271328 - Add a test to ensure NotifyLayersUpdated copies all the right fields

Status: RESOLVED WONTFIX

(Core :: Panning and Zooming, defect, P5)

Description

[Kartikaya Gupta (email:[e-mail])]

In AsyncPanZoomController::NotifyLayersUpdated we do this thing where we copy a bunch of fields from one scroll metadata object to another. We should add some gtests to make sure that all the right things are getting copied in the different branches, so that if a new field is added we don't accidentally miss copying it.

Comment 1

[Botond Ballo [:botond] [away until July 9]]

I don't really understand what the intention is to do in this test. After triggering the code that's supposed to do the copying, the test would need to either check that each field was copied individually (in which case, the test doesn't serve much purpose since after adding a new field you could forget to add the check to the *test*), or it uses operator==, in which case you're relying on the user remembering to add the field to operator==.

Comment 2

[Kartikaya Gupta (email:[e-mail])]

Yeah.. I'd be ok closing this bug as invalid or wontfix.