HubTurbo / addressbook

A small app for training and experimentation
Other
1 stars 3 forks source link

Add full GUI testing for tagging #559

Closed damithc closed 8 years ago

damithc commented 8 years ago

Test both the accelerator method and within the person edit dialog.

damithc commented 8 years ago

@m133225 can do this?

m133225 commented 8 years ago

Isn't the testing for accelerators done in KeyBindingsGuiTest already? Never mind, I get what you mean - testing the whole gui flow for tagging.

damithc commented 8 years ago

Yes. You can add/enhance the corresponding Gui unit tests at the same time.

m133225 commented 8 years ago

There's something similar (but not using shortcut to open the tag selection dialog) in FullSystemTest. Should remove that scenario from that class?

damithc commented 8 years ago

Yes, fullsystemtest should be removed after moving its parts to other classes.

Sent from a mobile device.

damithc commented 8 years ago

I noticed at least one bug in the tagging feature. Let's see if the Gui test uncovers it. :-)