From 78f3e68e8bd8008dd9ef3d8fea0761c66c88c05c Mon Sep 17 00:00:00 2001
From: Justin Richer <jricher@mit.edu>
Date: Thu, 31 Jul 2014 23:22:06 -0400
Subject: [PATCH] update unit test for Windows environment, closes #647

---
 .../org/mitre/jose/TestJWKSetKeyStore.java    | 51 +++++++++++--------
 1 file changed, 30 insertions(+), 21 deletions(-)

diff --git a/openid-connect-common/src/test/java/org/mitre/jose/TestJWKSetKeyStore.java b/openid-connect-common/src/test/java/org/mitre/jose/TestJWKSetKeyStore.java
index 02ca64919..524b4a75e 100644
--- a/openid-connect-common/src/test/java/org/mitre/jose/TestJWKSetKeyStore.java
+++ b/openid-connect-common/src/test/java/org/mitre/jose/TestJWKSetKeyStore.java
@@ -161,28 +161,37 @@ public class TestJWKSetKeyStore {
 		File file = new File(ks_file);
 		
 		/* First, test with file without "read" permission */
+
+        boolean set = false;
+
+        if (file.exists()) {
+                set = file.setReadable(false);
+        }
+        
+        // skip this part of the test on systems that don't allow the settable function, like Windows
+        if (set) {
+        	
+	        Resource loc_noread = new FileSystemResource(file);
+	        assertTrue(loc_noread.exists());
+	        // assertTrue(!loc_noread.isReadable());
+	
+	        boolean thrown = false;
+	        try {
+	                ks.setLocation(loc_noread);
+	        } catch (IllegalArgumentException e) {
+	                thrown = true;
+	        }
+	        assertTrue(thrown);
+	
+	        /* Now, make cache file readable */
+	
+	        if (file.exists()) {
+	                file.setReadable(true);
+	        }
+
+        }
 		
-		if (file.exists()) {
-			file.setReadable(false);
-		}
-		Resource loc_noread = new FileSystemResource(file);
-		assertTrue(loc_noread.exists());
-		assertTrue(!loc_noread.isReadable());
-		
-		boolean thrown = false;
-		try {
-			ks.setLocation(loc_noread);
-		} catch (IllegalArgumentException e) {
-			thrown = true;
-		}
-		assertTrue(thrown); 	
-		
-		/* Now, make cache file readable */
-		
-		if (file.exists()) {
-			file.setReadable(true);
-		}
-		Resource loc = new FileSystemResource(file);
+        Resource loc = new FileSystemResource(file);
 		assertTrue(loc.exists());
 		assertTrue(loc.isReadable());