Add CoreFeedbackDialogModel.

Change-Id: I0b995639103e7511b89e9aee658f91476f2547b1
Reviewed-on: http://gerrit.dmdirc.com/3603
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

src/com/dmdirc/interfaces/ui/FeedbackDialogModel.java

@@ -0,0 +1,172 @@
+/*
+ * Copyright (c) 2006-2014 DMDirc Developers
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+package com.dmdirc.interfaces.ui;
+
+import com.dmdirc.util.validators.Validator;
+
+import com.google.common.base.Optional;
+
+/**
+ * Dialog to connect to send feedback to the developers.
+ */
+public interface FeedbackDialogModel {
+
+    /**
+     * Gets the name in the model.
+     *
+     * @return Optional string containing the name
+     */
+    Optional<String> getName();
+
+    /**
+     * Sets the name in the model.
+     *
+     * @param name Optional string containing the name
+     */
+    void setName(Optional<String> name);
+
+    /**
+     * Is the name in the model valid?
+     *
+     * @return true or false
+     */
+    boolean isNameValid();
+
+    /**
+     * Gets the name validator in the model.
+     *
+     * @return Name validator
+     */
+    Validator<String> getNameValidator();
+
+    /**
+     * Gets the email in the model.
+     *
+     * @return Optional string containing the email
+     */
+    Optional<String> getEmail();
+
+    /**
+     * Sets the email in the model.
+     *
+     * @param email Optional string containing the email
+     */
+    void setEmail(Optional<String> email);
+
+    /**
+     * Is the email in the model valid?
+     *
+     * @return true or false
+     */
+    boolean isEmailValid();
+
+    /**
+     * Gets the validator for the email address
+     *
+     * @return Email validator
+     */
+    Validator<String> getEmailValidator();
+
+    /**
+     * Gets the feedback in the model.
+     *
+     * @return Optional string containing the feedback
+     */
+    Optional<String> getFeedback();
+
+    /**
+     * Sets the feedback in the model.
+     *
+     * @param feedback Optional string containing the feedback
+     */
+    void setFeedback(Optional<String> feedback);
+
+    /**
+     * Is the feedback in the model valid?
+     *
+     * @return true or false
+     */
+    boolean isFeedbackValid();
+
+    /**
+     * Gets the validator for the feedback.
+     *
+     * @return Feedback validator
+     */
+    Validator<String> getFeedbackValidator();
+
+    /**
+     * Should we include the server info with the feedback?
+     *
+     * @return true or false
+     */
+    boolean getIncludeServerInfo();
+
+    /**
+     * Sets whether we should include the server info with the feedback.
+     *
+     * @param includeServerInfo true or false
+     */
+    void setIncludeServerInfo(boolean includeServerInfo);
+
+    /**
+     * Should we include information about DMDirc with the feedback?
+     *
+     * @return true or false
+     */
+    boolean getIncludeDMDircInfo();
+
+    /**
+     * Sets whether we should include information about DMDirc with the feedback.
+     *
+     * @param includeDMDircInfo true or false
+     */
+    void setIncludeDMDircInfo(boolean includeDMDircInfo);
+
+    /**
+     * Are we allowed to save the dialog?
+     *
+     * @return true or false
+     */
+    boolean isSaveAllowed();
+
+    /**
+     * Saves the state of the dialog, sending specified feedback.
+     */
+    void save();
+
+    /**
+     * Adds a listener to this model.
+     *
+     * @param listener Listener to add
+     */
+    void addListener(FeedbackDialogModelListener listener);
+
+    /**
+     * Removes a listener from this model.
+     *
+     * @param listener Listener to remove
+     */
+    void removeListener(FeedbackDialogModelListener listener);
+
+}

src/com/dmdirc/interfaces/ui/FeedbackDialogModelListener.java

@@ -0,0 +1,67 @@
+/*
+ * Copyright (c) 2006-2014 DMDirc Developers
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+package com.dmdirc.interfaces.ui;
+
+import com.google.common.base.Optional;
+
+/**
+ * Listener for various events in a feedback dialog.
+ */
+public interface FeedbackDialogModelListener {
+
+    /**
+     * Called when the name is changed.
+     *
+     * @param name New name
+     */
+    void nameChanged(Optional<String> name);
+
+    /**
+     * Called when the email is changed.
+     *
+     * @param email New email
+     */
+    void emailChanged(Optional<String> email);
+
+    /**
+     * Called when the feedback is changed.
+     *
+     * @param feedback New feedback
+     */
+    void feedbackChanged(Optional<String> feedback);
+
+    /**
+     * Called when the server info checkbox is changed.
+     *
+     * @param includeServerInfo New state
+     */
+    void includeServerInfoChanged(boolean includeServerInfo);
+
+    /**
+     * Called when the DMDirc info checkbox is changed.
+     *
+     * @param includeDMDircInfo New state
+     */
+    void includeDMDircInfoChanged(boolean includeDMDircInfo);
+
+}

src/com/dmdirc/ui/core/feedback/CoreFeedbackDialogModel.java

@@ -0,0 +1,222 @@
+/*
+ * Copyright (c) 2006-2014 DMDirc Developers
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+package com.dmdirc.ui.core.feedback;
+
+import com.dmdirc.ClientModule;
+import com.dmdirc.ServerManager;
+import com.dmdirc.commandline.CommandLineOptionsModule.Directory;
+import com.dmdirc.commandline.CommandLineOptionsModule.DirectoryType;
+import com.dmdirc.interfaces.Connection;
+import com.dmdirc.interfaces.config.AggregateConfigProvider;
+import com.dmdirc.interfaces.ui.FeedbackDialogModel;
+import com.dmdirc.interfaces.ui.FeedbackDialogModelListener;
+import com.dmdirc.util.ClientInfo;
+import com.dmdirc.util.collections.ListenerList;
+import com.dmdirc.util.validators.EmailAddressValidator;
+import com.dmdirc.util.validators.NotEmptyValidator;
+import com.dmdirc.util.validators.PermissiveValidator;
+import com.dmdirc.util.validators.Validator;
+
+import com.google.common.base.Optional;
+
+import javax.inject.Inject;
+
+/**
+ * Default implementation of a feedback dialog module.
+ */
+public class CoreFeedbackDialogModel implements FeedbackDialogModel {
+
+    private final FeedbackSenderFactory feedbackSenderFactory;
+    private final ListenerList listeners;
+    private final AggregateConfigProvider config;
+    private final ServerManager serverManager;
+    private final String configDirectory;
+    private Optional<String> name;
+    private Optional<String> email;
+    private Optional<String> feedback;
+    private boolean includeServerInfo;
+    private boolean includeDMDircInfo;
+
+    @Inject
+    public CoreFeedbackDialogModel(@ClientModule.GlobalConfig final AggregateConfigProvider config,
+            final ServerManager serverManager, final FeedbackSenderFactory feedbackSenderFactory,
+            @Directory(DirectoryType.BASE) final String configDirectory) {
+        this.serverManager = serverManager;
+        this.configDirectory = configDirectory;
+        this.feedbackSenderFactory = feedbackSenderFactory;
+        this.config = config;
+        this.listeners = new ListenerList();
+        name = Optional.absent();
+        email = Optional.absent();
+        feedback = Optional.absent();
+        includeServerInfo = false;
+        includeDMDircInfo = false;
+    }
+
+    @Override
+    public Optional<String> getName() {
+        return name;
+    }
+
+    @Override
+    public void setName(final Optional<String> name) {
+        this.name = name;
+        listeners.getCallable(FeedbackDialogModelListener.class).nameChanged(name);
+    }
+
+    @Override
+    public boolean isNameValid() {
+        return !getNameValidator().validate(name.or("")).isFailure();
+    }
+
+    @Override
+    public Validator<String> getNameValidator() {
+        return new PermissiveValidator<>();
+    }
+
+    @Override
+    public Optional<String> getEmail() {
+        return email;
+    }
+
+    @Override
+    public void setEmail(final Optional<String> email) {
+        this.email = email;
+        listeners.getCallable(FeedbackDialogModelListener.class).emailChanged(email);
+    }
+
+    @Override
+    public boolean isEmailValid() {
+        return !getEmailValidator().validate(email.or("")).isFailure();
+    }
+
+    @Override
+    public Validator<String> getEmailValidator() {
+        if (email.isPresent()) {
+            return new EmailAddressValidator();
+        }
+        return new PermissiveValidator<>();
+    }
+
+    @Override
+    public Optional<String> getFeedback() {
+        return feedback;
+    }
+
+    @Override
+    public void setFeedback(final Optional<String> feedback) {
+        this.feedback = feedback;
+        listeners.getCallable(FeedbackDialogModelListener.class).feedbackChanged(feedback);
+    }
+
+    @Override
+    public boolean isFeedbackValid() {
+        return !getFeedbackValidator().validate(feedback.or("")).isFailure();
+    }
+
+    @Override
+    public Validator<String> getFeedbackValidator() {
+        return new NotEmptyValidator();
+    }
+
+    @Override
+    public boolean getIncludeServerInfo() {
+        return includeServerInfo;
+    }
+
+    @Override
+    public void setIncludeServerInfo(final boolean includeServerInfo) {
+        this.includeServerInfo = includeServerInfo;
+        listeners.getCallable(FeedbackDialogModelListener.class).includeServerInfoChanged(
+                includeServerInfo);
+    }
+
+    @Override
+    public boolean getIncludeDMDircInfo() {
+        return includeDMDircInfo;
+    }
+
+    @Override
+    public void setIncludeDMDircInfo(final boolean includeDMDircInfo) {
+        this.includeDMDircInfo = includeDMDircInfo;
+        listeners.getCallable(FeedbackDialogModelListener.class).includeDMDircInfoChanged(
+                includeDMDircInfo);
+    }
+
+    @Override
+    public boolean isSaveAllowed() {
+        return isNameValid() && isEmailValid() && isFeedbackValid();
+    }
+
+    @Override
+    public void save() {
+        final StringBuilder serverInfo = new StringBuilder(255);
+        final StringBuilder dmdircInfo = new StringBuilder(255);
+        if (getIncludeServerInfo()) {
+            for (Connection connection : serverManager.getServers()) {
+                if (connection.getState().isDisconnected()) {
+                    continue;
+                }
+                serverInfo.append("Actual name: ").append(connection.getParser()
+                        .getServerName()).append("\n");
+                serverInfo.append("Network: ").append(connection.getNetwork())
+                        .append("\n");
+                serverInfo.append("IRCd: ").append(connection.getParser()
+                        .getServerSoftware()).append(" - ");
+                serverInfo.append(connection.getParser().getServerSoftwareType())
+                        .append("\n");
+                serverInfo.append("Modes: ").append(connection.getParser()
+                        .getBooleanChannelModes()).append(" ");
+                serverInfo.append(connection.getParser().getListChannelModes())
+                        .append(" ");
+                serverInfo.append(connection.getParser().getParameterChannelModes())
+                        .append(" ");
+                serverInfo.append(connection.getParser().
+                        getDoubleParameterChannelModes());
+            }
+        }
+        if (getIncludeDMDircInfo()) {
+            dmdircInfo.append("DMDirc version: ").append(ClientInfo.getVersionInformation())
+                    .append("\n");
+            dmdircInfo.append("Profile directory: ").append(configDirectory).append("\n");
+            dmdircInfo.append("Java version: ").append(ClientInfo.getJavaInformation())
+                    .append("\n");
+            dmdircInfo.append("OS Version: ").append(ClientInfo.getOperatingSystemInformation());
+        }
+        final FeedbackSender sender = feedbackSenderFactory.getFeedbackSender(
+                name.or(""), email.or(""), feedback.or(""), config.getOption("version", "version"),
+                serverInfo.toString(), dmdircInfo.toString());
+        new Thread(sender, "Feedback Sender").start();
+    }
+
+    @Override
+    public void addListener(final FeedbackDialogModelListener listener) {
+        listeners.add(FeedbackDialogModelListener.class, listener);
+    }
+
+    @Override
+    public void removeListener(final FeedbackDialogModelListener listener) {
+        listeners.remove(FeedbackDialogModelListener.class, listener);
+    }
+
+}

src/com/dmdirc/ui/core/feedback/FeedbackDialogModelAdapter.java

@@ -0,0 +1,55 @@
+/*
+ * Copyright (c) 2006-2014 DMDirc Developers
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+package com.dmdirc.ui.core.feedback;
+
+import com.dmdirc.interfaces.ui.FeedbackDialogModelListener;
+
+import com.google.common.base.Optional;
+
+/**
+ * An abstract adapter class for receiving feedback dialog model events. The methods in this class
+ * are empty. This class exists as convenience for creating listener objects.
+ */
+public class FeedbackDialogModelAdapter implements FeedbackDialogModelListener {
+
+    @Override
+    public void nameChanged(final Optional<String> name) {
+    }
+
+    @Override
+    public void emailChanged(final Optional<String> email) {
+    }
+
+    @Override
+    public void feedbackChanged(final Optional<String> feedback) {
+    }
+
+    @Override
+    public void includeServerInfoChanged(final boolean includeServerInfo) {
+    }
+
+    @Override
+    public void includeDMDircInfoChanged(final boolean includeDMDircInfo) {
+    }
+
+}

src/com/dmdirc/ui/core/feedback/FeedbackSender.java

@@ -0,0 +1,109 @@
+/*
+ * Copyright (c) 2006-2014 DMDirc Developers
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+package com.dmdirc.ui.core.feedback;
+
+import com.dmdirc.ClientModule.GlobalConfig;
+import com.dmdirc.interfaces.config.AggregateConfigProvider;
+import com.dmdirc.interfaces.ui.StatusBar;
+import com.dmdirc.ui.StatusMessage;
+import com.dmdirc.util.annotations.factory.Factory;
+import com.dmdirc.util.annotations.factory.Unbound;
+import com.dmdirc.util.io.Downloader;
+
+import java.io.IOException;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+
+/**
+ * Sends feedback back to the developers.
+ */
+@Factory(inject = true)
+public class FeedbackSender implements Runnable {
+
+    private final Map<String, String> postData;
+    private final StatusBar statusBar;
+    private final AggregateConfigProvider config;
+    private final Downloader downloader;
+
+    public FeedbackSender(
+            @SuppressWarnings("qualifiers") @GlobalConfig final AggregateConfigProvider config,
+            final Downloader downloader,
+            final StatusBar statusBar,
+            @Unbound final String name,
+            @Unbound final String email,
+            @Unbound final String feedback,
+            @Unbound final String version,
+            @Unbound final String serverInfo,
+            @Unbound final String dmdircInfo) {
+        this.downloader = downloader;
+        this.config = config;
+        this.statusBar = statusBar;
+        this.postData = new HashMap<>(6);
+
+        if (!name.isEmpty()) {
+            postData.put("name", name);
+        }
+        if (!email.isEmpty()) {
+            postData.put("email", email);
+        }
+        if (!feedback.isEmpty()) {
+            postData.put("feedback", feedback);
+        }
+        if (!version.isEmpty()) {
+            postData.put("version", version);
+        }
+        if (!serverInfo.isEmpty()) {
+            postData.put("serverInfo", serverInfo);
+        }
+        if (!dmdircInfo.isEmpty()) {
+            postData.put("dmdircInfo", dmdircInfo);
+        }
+    }
+
+    /**
+     * Sends the error data to the server appending returned information to the global error
+     * variable.
+     *
+     * @param postData Feedback data to send
+     */
+    private String sendData(final Map<String, String> postData) {
+        try {
+            final List<String> response = downloader.getPage("http://www.dmdirc.com/feedback.php",
+                    postData);
+            if (response.size() >= 1) {
+                return "Feedback sent successfully";
+            } else {
+                return "Feedback failure: Unknown response from the server.";
+            }
+        } catch (IOException ex) {
+            return "Feedback failure: " + ex.getMessage();
+        }
+    }
+
+    @Override
+    public void run() {
+        statusBar.setMessage(new StatusMessage(sendData(postData), config));
+    }
+
+}

test/com/dmdirc/ui/core/feedback/CoreFeedbackDialogModelTest.java

@@ -0,0 +1,145 @@
+/*
+ * Copyright (c) 2006-2014 DMDirc Developers
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+package com.dmdirc.ui.core.feedback;
+
+import com.dmdirc.ServerManager;
+import com.dmdirc.interfaces.config.AggregateConfigProvider;
+import com.dmdirc.interfaces.ui.FeedbackDialogModelListener;
+
+import com.google.common.base.Optional;
+
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.Mock;
+import org.mockito.runners.MockitoJUnitRunner;
+
+import static org.junit.Assert.assertEquals;
+import static org.mockito.Mockito.verify;
+
+@RunWith(MockitoJUnitRunner.class)
+public class CoreFeedbackDialogModelTest {
+
+    @Mock private AggregateConfigProvider config;
+    @Mock private ServerManager serverManager;
+    @Mock private FeedbackSenderFactory feedbackSenderFactory;
+    @Mock private FeedbackDialogModelListener listener;
+    private final String name = "Bob Dole";
+    private final String email = "bob@dole.com";
+    private final String feedback = "DMDirc Rocks.";
+
+    @Test
+    public void testName() {
+        final CoreFeedbackDialogModel instance = new CoreFeedbackDialogModel(config, serverManager,
+                feedbackSenderFactory, "configDirectory");
+        assertEquals("testName", Optional.absent(), instance.getName());
+        instance.setName(Optional.fromNullable(name));
+        assertEquals("testName", Optional.fromNullable(name), instance.getName());
+    }
+
+    @Test
+    public void testEmail() {
+        final CoreFeedbackDialogModel instance = new CoreFeedbackDialogModel(config, serverManager,
+                feedbackSenderFactory, "configDirectory");
+        assertEquals("testEmail", Optional.absent(), instance.getEmail());
+        instance.setEmail(Optional.fromNullable(email));
+        assertEquals("testEmail", Optional.fromNullable(email), instance.getEmail());
+    }
+
+    @Test
+    public void testFeedback() {
+        final CoreFeedbackDialogModel instance = new CoreFeedbackDialogModel(config, serverManager,
+                feedbackSenderFactory, "configDirectory");
+        assertEquals("testFeedback", Optional.absent(), instance.getFeedback());
+        instance.setFeedback(Optional.fromNullable(feedback));
+        assertEquals("testFeedback", Optional.fromNullable(feedback), instance.getFeedback());
+    }
+
+    @Test
+    public void testServerInfo() {
+        final CoreFeedbackDialogModel instance = new CoreFeedbackDialogModel(config, serverManager,
+                feedbackSenderFactory, "configDirectory");
+        assertEquals("testServerInfo", false, instance.getIncludeServerInfo());
+        instance.setIncludeServerInfo(true);
+        assertEquals("testServerInfo", true, instance.getIncludeServerInfo());
+    }
+
+    @Test
+    public void testDMDircInfo() {
+        final CoreFeedbackDialogModel instance = new CoreFeedbackDialogModel(config, serverManager,
+                feedbackSenderFactory, "configDirectory");
+        assertEquals("testDMDircInfo", false, instance.getIncludeDMDircInfo());
+        instance.setIncludeDMDircInfo(true);
+        assertEquals("testDMDircInfo", true, instance.getIncludeDMDircInfo());
+    }
+
+    @Test
+    public void testSave() {
+        // TODO: fix ClientInfo being static somehow and test.
+    }
+
+    @Test
+    public void testNameListener() {
+        final CoreFeedbackDialogModel instance = new CoreFeedbackDialogModel(config, serverManager,
+                feedbackSenderFactory, "configDirectory");
+        instance.addListener(listener);
+        instance.setName(Optional.fromNullable("Bob Dole"));
+        verify(listener).nameChanged(Optional.fromNullable("Bob Dole"));
+    }
+
+    @Test
+    public void testEmailListener() {
+        final CoreFeedbackDialogModel instance = new CoreFeedbackDialogModel(config, serverManager,
+                feedbackSenderFactory, "configDirectory");
+        instance.addListener(listener);
+        instance.setEmail(Optional.fromNullable("bob@dole.com"));
+        verify(listener).emailChanged(Optional.fromNullable("bob@dole.com"));
+    }
+
+    @Test
+    public void testFeedbackListener() {
+        final CoreFeedbackDialogModel instance = new CoreFeedbackDialogModel(config, serverManager,
+                feedbackSenderFactory, "configDirectory");
+        instance.addListener(listener);
+        instance.setFeedback(Optional.fromNullable("DMDirc Rocks."));
+        verify(listener).feedbackChanged(Optional.fromNullable("DMDirc Rocks."));
+    }
+
+    @Test
+    public void testServerInfoListener() {
+        final CoreFeedbackDialogModel instance = new CoreFeedbackDialogModel(config, serverManager,
+                feedbackSenderFactory, "configDirectory");
+        instance.addListener(listener);
+        instance.setIncludeServerInfo(true);
+        verify(listener).includeServerInfoChanged(true);
+    }
+
+    @Test
+    public void testDMDircInfoListener() {
+        final CoreFeedbackDialogModel instance = new CoreFeedbackDialogModel(config, serverManager,
+                feedbackSenderFactory, "configDirectory");
+        instance.addListener(listener);
+        instance.setIncludeDMDircInfo(true);
+        verify(listener).includeDMDircInfoChanged(true);
+    }
+
+}