On Tue, 2010-08-03 at 15:44 +0200, jprov...@redhat.com wrote: > From: Jan Provaznik <jprov...@redhat.com> > > Till now provider new form was displayed after successful delete. > --- > src/app/controllers/provider_controller.rb | 2 -- > src/app/views/provider/_providers.haml | 7 +++++++ > src/app/views/provider/index.haml | 1 + > src/app/views/provider/index.html.erb | 1 - > src/app/views/settings/index.haml | 9 +-------- > 5 files changed, 9 insertions(+), 11 deletions(-) > create mode 100644 src/app/views/provider/_providers.haml > create mode 100644 src/app/views/provider/index.haml > delete mode 100644 src/app/views/provider/index.html.erb
ACK, though with my usual response of 'where is the test?'. This should be a cucumber scenario. It could be as simple as verifying that the user is redirected to the list, and that the just-deleted provider is no longer shown. Not related to your patch, but that javascript alert of 'do you really want to delete provider' is icky. -j _______________________________________________ deltacloud-devel mailing list deltacloud-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/deltacloud-devel
