unrolled test nesting to better demonstrate async nature

Author: bmc

test/integration/client/prepared-statement-tests.js

@@ -20,76 +20,76 @@ test("simple, unnamed prepared statement", function(){
 test("named prepared statement", function() {
 
   var client = helper.client();
-  var queryName = "user by age and like name";
-  var query = client.query({
-    text: 'select name from person where age <= $1 and name LIKE $2',
-    values: [20, 'Bri%'],
-    name: queryName
-  });
+  client.on('drain', client.end.bind(client));
 
+  var queryName = "user by age and like name";
   var parseCount = 0;
-  client.connection.on('parseComplete', function() {
-    parseCount++;
-  });
-
-  assert.raises(query, 'row', function(row) {
-    assert.equal(row.fields[0], 'Brian');
-  });
 
-  assert.raises(query, 'end', function() {
-    test("query was parsed", function() {
-      assert.equal(parseCount, 1);
+  test("first named prepared statement",function() {
+    var query = client.query({
+      text: 'select name from person where age <= $1 and name LIKE $2',
+      values: [20, 'Bri%'],
+      name: queryName
     });
 
-    test("with same name & text", function() {
-      var cachedQuery = client.query({
-        text: 'select name from person where age <= $1 and name LIKE $2',
-        name: queryName,
-        values: [10, 'A%']
+    test("is parsed", function() {
+      client.connection.on('parseComplete', function() {
+        parseCount++;
       });
+    });
 
-      assert.raises(cachedQuery, 'row', function(row) {
-        assert.equal(row.fields[0], 'Aaron');
-      });
+    assert.raises(query, 'row', function(row) {
+      assert.equal(row.fields[0], 'Brian');
+    });
 
-      assert.raises(cachedQuery, 'end', function() {
-        test("query was only parsed one time", function() {
-          assert.equal(parseCount, 1, "Should not have reparsed query");
-        });
+    assert.raises(query, 'end', function() {
+      test("query was parsed", function() {
+        assert.equal(parseCount, 1);
       });
     });
+  });
 
-    test("with same name, but the query text not even there batman!", function() {
-      var q = client.query({
-        name: queryName,
-        values: [30, '%n%']
-      });
+  test("second named prepared statement with same name & text", function() {
+    var cachedQuery = client.query({
+      text: 'select name from person where age <= $1 and name LIKE $2',
+      name: queryName,
+      values: [10, 'A%']
+    });
+
+    assert.raises(cachedQuery, 'row', function(row) {
+      assert.equal(row.fields[0], 'Aaron');
+    });
 
-      test("gets first row", function() {
+    assert.raises(cachedQuery, 'end', function() {
+      test("query was only parsed one time", function() {
+        assert.equal(parseCount, 1, "Should not have reparsed query");
+      });
+    });
+  });
 
-        assert.raises(q, 'row', function(row) {
-          assert.equal(row.fields[0], "Aaron");
+  test("with same name, but the query text not even there batman!", function() {
+    var q = client.query({
+      name: queryName,
+      values: [30, '%n%']
+    });
 
-          test("gets second row", function() {
+    test("gets first row", function() {
+      assert.raises(q, 'row', function(row) {
+        assert.equal(row.fields[0], "Aaron");
 
-            assert.raises(q, 'row', function(row) {
-              assert.equal(row.fields[0], "Brian");
-            });
+        test("gets second row", function() {
+          assert.raises(q, 'row', function(row) {
+            assert.equal(row.fields[0], "Brian");
           });
-
         });
-      });
 
-      test("only parsed query once total", function() {
-        assert.equal(parseCount, 1);
-        q.on('end', function() {
-          client.end();
-        });
       });
+    });
 
+    assert.raises(q, 'end', function() {
+      assert.equal(parseCount, 1);
     });
   });
-
 });
 
 test("prepared statements on different clients", function() {