The simplest thing to do, now that the view can be opened, is to close it, and to do it without applying any changes, that is, to click the cancel button. So I add a test for that.
[Test] public void Clicking_cancel_closes_add_song_window() { Application .Start(view => { view.ClickAddButton(addView => { addView.ClickCancelButton(); Assert.That(addView.HasBeenClosed, Is.True); }); view.ClickCloseButton(); }); }This test follows the same pattern as the one where I verify that clicking the close button closes the SongsView.
I need to add a few members to the AddSongViewFake in order for the code to compile.
public class AddSongViewFake : IAddSongView { public event Action OnCancelButtonClick; public bool HasBeenClosed { get; private set; } // ... public AddSongViewFake(Action<AddSongViewFake> showAction) { ShowAction = showAction; HasBeenClosed = false; } // ... public void CloseView() { HasBeenClosed = true; } public void ClickCancelButton() { OnCancelButtonClick.Raise(); } }Just like with the songs window, I delegate the handling of the button click to somewhere else by raising the OnCancelButtonClick event. Then I expect whoever is handling that event to call the CloseView() method to close it.
The test failure is the same as always, that I don't have any listeners to the event.
So I pull the OnCancelButtonClick event and the CloseView() method up to the interface, and also add it to the real view.
public interface IAddSongView { event Action OnCancelButtonClick; void ShowView(); void CloseView(); } public partial class AddSongView : Form, IAddSongView { public event Action OnCancelButtonClick; // ... public void CloseView() { Close(); } private void CancelButtonClick( object sender, EventArgs e) { OnCancelButtonClick.Raise(); } }And then I do the smallest thing possible to fix the failure, which is to start subscribing to the event with an empty event handler.
public class SongsShower { // ... private void OnAddButtonClick() { var view = AddSongViewFactory.Create(); view.OnCancelButtonClick += () => { }; view.ShowView(); } }Now I get another failure message when running the test.
That's a good message. So now I can implement the solution.
private void OnAddButtonClick() { var view = AddSongViewFactory.Create(); view.OnCancelButtonClick += view.CloseView; view.ShowView(); }And the test is green. I run the application to see that it is working for real as well. And then I check in my code.
Here is what the beautiful user interface looks like after adding the cancel button.
