C#

Member 4487083 wrote:

One way is to make them protected instead of private, but then you are changing the application code.

Member 4487083 wrote:

Is there a way to do it without modifying the code above?

Member 4487083 wrote:

Visual Studio can also setup private accessors, but I think they're quite ugly. Or are they they best option?

keefb wrote:

you shouldn't change your code specifically for testing

Luc Pattyn wrote:

I don't quite agree

Luc Pattyn wrote:

my personal preference: change all private attributes to protected ones; then derive a class that simply adds the test code and can access everything; you probably would keep the test class in a separate assembly which you do not distribute.

Luc Pattyn wrote:

change all private attributes to protected ones

public class TestClass
{
	private void PrivateMethod(bool someBool)
	{
		if (someBool)
		{
			// do something
			Console.WriteLine("Reachable Code");
		}
		else
		{
			// do something else
			Console.WriteLine("Dead Code");
		}
	}
	
	public void PublicMethod(double x)
	{
		// do some stuff
		
		// Here we're going to do some calculation with the input and use the result to determine what to pass 
		// PrivateMethod(...).   I'm using Math.Abs just as an example.
		double calculationResult = Math.Abs(x);  
		
		PrivateMethod (calculationResult >= 0.0);
		
		// do some more stuff
	}
}

modified on Monday, February 15, 2010 7:06 AM

Member 4487083 wrote:

only testing public members

Member 4487083 wrote:

a unit eg. a single method

Member 4487083 wrote:

which may include using things in a way that they weren't necessarily supposed to be used

Member 4487083 wrote:

Dead code is potentially a bug waiting to happen IMHO

Member 4487083 wrote:

it may be worth making a few modifications for testing

SimpleData wrote:

you need I can post the code

private void InitializeComponent()
        {
            this.components = new System.ComponentModel.Container();
            System.ComponentModel.ComponentResourceManager resources = new System.ComponentModel.ComponentResourceManager(typeof(login));
            this.label1 = new System.Windows.Forms.Label();
            this.btnLogin = new System.Windows.Forms.Button();
            this.txtUsername = new System.Windows.Forms.TextBox();
            this.label2 = new System.Windows.Forms.Label();
            this.txtPassword = new System.Windows.Forms.TextBox();
            this.label4 = new System.Windows.Forms.Label();
            this.panel1 = new System.Windows.Forms.Panel();
            this.cbTVmode = new System.Windows.Forms.CheckBox();
            this.pbPartnerLogo = new System.Windows.Forms.PictureBox();
            this.label6 = new System.Windows.Forms.Label();
            this.cbWebSite = new System.Windows.Forms.ComboBox();
            this.chAutoLogin = new System.Windows.Forms.CheckBox();
            this.chRememberMe = new System.Windows.Forms.CheckBox();
            this.panel2 = new System.Windows.Forms.Panel();
            this.pictureBox3 = new System.Windows.Forms.PictureBox();
            this.pictureBox2 = new System.Windows.Forms.PictureBox();
            this.pictureBox1 = new System.Windows.Forms.PictureBox();
            this.toolTip1 = new System.Windows.Forms.ToolTip(this.components);
            this.panel1.SuspendLayout();
            ((System.ComponentModel.ISupportInitialize)(this.pbPartnerLogo)).BeginInit();
            this.panel2.SuspendLayout();
            ((System.ComponentModel.ISupportInitialize)(this.pictureBox3)).BeginInit();
            ((System.ComponentModel.ISupportInitialize)(this.pictureBox2)).BeginInit();
            ((System.ComponentModel.ISupportInitialize)(this.pictureBox1)).BeginInit();
            this.SuspendLayout();

// This continues with property declerations of buttons, labels, pictureboxes and listboxes.

public static void Main() {
    try {
        ...
        Application.Run(new Form1());
    } catch(Exception exc) {
        Console.WriteLine(exc.ToString());
    }
}