Adjust test results display to render to a ListView with color

Author: developernotes

res/layout/main.xml

@@ -13,12 +13,9 @@
                 android:text="@string/test_suite_button"
                 android:onClick="onButtonClick"/>
     </LinearLayout>
-    <TextView
+    <ListView
             android:id="@+id/test_suite_results"
             android:layout_width="fill_parent"
             android:layout_height="fill_parent"
-            android:gravity="top|left"
-            android:scrollbars="vertical"
-            android:singleLine="false"
             android:layout_below="@+id/container"/>
 </RelativeLayout>

res/layout/test_result_row.xml

@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="utf-8"?>
+
+<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
+              android:orientation="vertical"
+              android:layout_width="fill_parent"
+              android:layout_height="fill_parent">
+    <TextView android:id="@+id/test_status"
+              android:layout_height="wrap_content"
+              android:layout_width="wrap_content"
+              android:padding="5dip"
+              android:gravity="right"
+              android:textSize="18sp"
+              android:layout_alignParentRight="true"/>
+    <TextView android:id="@+id/test_name"
+              android:layout_height="wrap_content"
+              android:layout_width="fill_parent"
+              android:paddingTop="5dip"
+              android:gravity="left"
+              android:textSize="18sp"
+              android:singleLine="true"
+              android:ellipsize="end"
+              android:layout_toLeftOf="@+id/test_status"/>
+</RelativeLayout>

src/main/java/net/zetetic/TestResultAdapter.java

@@ -0,0 +1,46 @@
+package net.zetetic;
+
+import android.content.Context;
+import android.graphics.Color;
+import android.view.LayoutInflater;
+import android.view.View;
+import android.view.ViewGroup;
+import android.widget.ArrayAdapter;
+import android.widget.TextView;
+import net.zetetic.tests.TestResult;
+
+import java.util.List;
+
+public class TestResultAdapter extends ArrayAdapter<TestResult> {
+
+    public TestResultAdapter(Context context, List<TestResult> results) {
+        super(context, 0, results);
+    }
+
+    @Override
+    public View getView(int position, View view, ViewGroup parent) {
+
+        ViewHolder holder;
+        if(view == null){
+            LayoutInflater viewInflater = (LayoutInflater) ZeteticApplication.getInstance().getSystemService(Context.LAYOUT_INFLATER_SERVICE);
+            view = viewInflater.inflate(R.layout.test_result_row, null);
+            holder = new ViewHolder();
+            holder.testName = (TextView) view.findViewById(R.id.test_name);
+            holder.testStatus = (TextView) view.findViewById(R.id.test_status);
+            view.setTag(holder);
+        } else {
+            holder = (ViewHolder)view.getTag();
+        }
+        TestResult result = getItem(position);
+        holder.testName.setText(result.getName());
+        holder.testStatus.setText(result.toString());
+        int displayColor = result.isSuccess() ? Color.GREEN : Color.RED;
+        holder.testStatus.setTextColor(displayColor);
+        return view;
+    }
+
+    static class ViewHolder {
+        TextView testName;
+        TextView testStatus;
+    }
+}

src/main/java/net/zetetic/TestSuiteActivity.java

@@ -2,18 +2,26 @@
 
 import android.app.Activity;
 import android.os.Bundle;
-import android.text.method.ScrollingMovementMethod;
 import android.util.Log;
 import android.view.View;
-import android.widget.TextView;
+import android.widget.ArrayAdapter;
+import android.widget.ListView;
 import net.zetetic.tests.ResultNotifier;
 import net.zetetic.tests.TestResult;
 import net.zetetic.tests.TestSuiteRunner;
 
+import java.util.ArrayList;
+import java.util.List;
+
 public class TestSuiteActivity extends Activity implements ResultNotifier {
 
     private static String TAG = "net.zetetic.sqlcipher.test";
-    TextView resultsView;
+    ListView resultsView;
+    List<TestResult> results;
+
+    public TestSuiteActivity(){
+        results = new ArrayList<TestResult>();
+    }
 
     @Override
     public void onCreate(Bundle savedInstanceState) {
@@ -25,15 +33,21 @@ public void onCreate(Bundle savedInstanceState) {
     public void onButtonClick(View view) {
 
         findViewById(R.id.executeSuite).setEnabled(false);
-        resultsView = (TextView) findViewById(R.id.test_suite_results);
-        resultsView.setMovementMethod(ScrollingMovementMethod.getInstance());
+        resultsView = (ListView) findViewById(R.id.test_suite_results);
         ZeteticApplication.getInstance().setCurrentActivity(this);
         new TestSuiteRunner().execute(this);
     }
 
     @Override
     public void send(TestResult result) {
-        resultsView.append(result.toString());
+
+        results.add(result);
+        ArrayAdapter<TestResult> adapter = (ArrayAdapter<TestResult>) resultsView.getAdapter();
+        if(adapter == null){
+            resultsView.setAdapter(new TestResultAdapter(ZeteticApplication.getInstance(), results));
+        } else {
+            adapter.notifyDataSetChanged();
+        }
     }
 
     @Override

src/main/java/net/zetetic/tests/TestResult.java

@@ -24,7 +24,6 @@ public boolean isSuccess() {
 
     @Override
     public String toString() {
-        String status = isSuccess() ? "OK" : "FAILED";
-        return String.format("%s : %s\n", getName(), status);
+        return isSuccess() ? "OK" : "FAILED";
     }
 }