improved testing for 147484

src/com/jacob/com/Variant.java

@@ -813,7 +813,7 @@ public class Variant extends JacobObject {
      * @param pVariant
      * @return
      */
-    private boolean objectIsAConstant(Variant pVariant){
+    protected boolean objectIsAConstant(Variant pVariant){
     	if (pVariant == VT_FALSE ||
     			pVariant == VT_TRUE ||
     			pVariant == VT_MISSING ||

unittest/com/jacob/com/VariantTest.java

@@ -13,7 +13,7 @@ class VariantTest {
 		testJig.testSafeReleaseBoolean();
 		testJig.testSafeReleaseConstant();
 		testJig.testSafeReleaseString();
-		
+		testJig.testObjectIsAConstant();
 		System.out.println("Testing Complete");
 		
 	}
@@ -90,6 +90,34 @@ class VariantTest {
 			System.out.println("Variant value ("+v.getString()+") "
 					+"has survived SafeRelease()");
 		}
+    }
+	
+    /**
+     * verifies objectIsAConstant works as expected
+     *
+     */
+    public void testObjectIsAConstant(){
+    	Variant v = new Variant("d");
+    	if (!v.objectIsAConstant(Variant.VT_FALSE)){
+    		System.out.println("did not recognize VT_FALSE");
+    	}
+    	if (!v.objectIsAConstant(Variant.VT_TRUE)){
+    		System.out.println("did not recognize VT_TRUE");
+    	}
+    	if (!v.objectIsAConstant(Variant.VT_MISSING)){
+    		System.out.println("did not recognize VT_MISSING");
+    	}
+    	if (!v.objectIsAConstant(Variant.DEFAULT)){
+    		System.out.println("did not recognize DEFAULT");
+    	}
+    	if (v.objectIsAConstant(new Variant(true))){
+    		System.out.println("confused a boolean with VT_TRUE");
+    	}
+    	if (v.objectIsAConstant(new Variant(false))){
+    		System.out.println("confused a boolean with VT_FALSE");
+    	}
+    	    	
+    	
     }
     
 	/**