Fix for #1226 and unit tests. (#1227)

src/main/java/com/zaxxer/hikari/HikariDataSource.java

@@ -30,7 +30,7 @@ import java.sql.SQLException;
 import java.sql.SQLFeatureNotSupportedException;
 import java.util.concurrent.atomic.AtomicBoolean;
 
-import static com.zaxxer.hikari.pool.HikariPool.POOL_SHUTDOWN;
+import static com.zaxxer.hikari.pool.HikariPool.POOL_NORMAL;
 
 /**
  * The HikariCP pooled DataSource.
@@ -294,7 +294,7 @@ public class HikariDataSource extends HikariConfig implements DataSource, Closea
     */
    public boolean isRunning()
    {
-      return pool != null && pool.poolState != POOL_SHUTDOWN;
+      return pool != null && pool.poolState == POOL_NORMAL;
    }
 
    /**

src/test/java/com/zaxxer/hikari/pool/TestIsRunning.java

@@ -0,0 +1,83 @@
+package com.zaxxer.hikari.pool;
+
+import static com.zaxxer.hikari.pool.TestElf.getPool;
+import static com.zaxxer.hikari.pool.TestElf.newHikariConfig;
+import static com.zaxxer.hikari.pool.TestElf.newHikariDataSource;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertTrue;
+
+import java.util.concurrent.TimeUnit;
+
+import org.junit.Test;
+
+import com.zaxxer.hikari.HikariConfig;
+import com.zaxxer.hikari.HikariDataSource;
+
+/**
+ * Tests for {@link HikariDataSource#isRunning()}.
+ */
+public class TestIsRunning
+{
+    @Test
+    public void testRunningNormally()
+    {
+        try (HikariDataSource ds = new HikariDataSource(basicConfig()))
+        {
+            assertTrue(ds.isRunning());
+        }
+    }
+
+
+    @Test
+    public void testNoPool()
+    {
+        try (HikariDataSource ds = newHikariDataSource())
+        {
+            assertNull("Pool should not be initialized.", getPool(ds));
+            assertFalse(ds.isRunning());
+        }
+    }
+
+
+    @Test
+    public void testSuspendAndResume()
+    {
+        try (HikariDataSource ds = new HikariDataSource(basicConfig()))
+        {
+            ds.getHikariPoolMXBean().suspendPool();
+            assertFalse(ds.isRunning());
+
+            ds.getHikariPoolMXBean().resumePool();
+            assertTrue(ds.isRunning());
+        }
+    }
+
+
+    @Test
+    public void testShutdown()
+    {
+        try (HikariDataSource ds = new HikariDataSource(basicConfig()))
+        {
+            ds.close();
+            assertFalse(ds.isRunning());
+        }
+    }
+
+
+    private HikariConfig basicConfig()
+    {
+        HikariConfig config = newHikariConfig();
+        config.setMinimumIdle(1);
+        config.setMaximumPoolSize(1);
+        config.setConnectionTestQuery("VALUES 1");
+        config.setConnectionInitSql("SELECT 1");
+        config.setReadOnly(true);
+        config.setConnectionTimeout(2500);
+        config.setLeakDetectionThreshold(TimeUnit.SECONDS.toMillis(30));
+        config.setDataSourceClassName("com.zaxxer.hikari.mocks.StubDataSource");
+        config.setAllowPoolSuspension(true);
+
+        return config;
+    }
+}