From 337882182bd0ab3e7a147159245a8d1331fbfbf0 Mon Sep 17 00:00:00 2001
From: Richard Wall <richard@largo>
Date: Mon, 30 Aug 2010 15:22:25 +0100
Subject: Add another RRA to demonstrate that jarmon is currently selecting the
 wrong RRA

---
 jarmon/jarmon.test.js   | 13 +++++++------
 jarmonbuild/commands.py |  3 ++-
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/jarmon/jarmon.test.js b/jarmon/jarmon.test.js
index 09fec1a..0aca865 100644
--- a/jarmon/jarmon.test.js
+++ b/jarmon/jarmon.test.js
@@ -47,9 +47,10 @@ YUI({ logInclude: { TestRunner: true } }).use('console', 'test', function(Y) {
     var RRD_STEP = 10;
     var RRD_DSNAME = 'speed';
     var RRD_DSINDEX = 0;
-    var RRD_RRAROWS = 6;
+    var RRD_RRACOUNT = 2;
+    var RRD_RRAROWS = 12;
     var RRD_STARTTIME = new Date('1 jan 1980 00:00:00').getTime();
-    var RRD_ENDTIME = new Date('1 jan 1980 00:01:01').getTime();
+    var RRD_ENDTIME = new Date('1 jan 1980 00:02:01').getTime();
 
     Y.Test.Runner.add(new Y.Test.Case({
         name: "javascriptrrd.RRDFile",
@@ -112,7 +113,7 @@ YUI({ logInclude: { TestRunner: true } }).use('console', 'test', function(Y) {
             this.d.addCallback(
                 function(self, rrd) {
                     self.resume(function() {
-                        Y.Assert.areEqual(1, rrd.getNrRRAs());
+                        Y.Assert.areEqual(RRD_RRACOUNT, rrd.getNrRRAs());
                     });
                 }, this);
             this.wait();
@@ -137,7 +138,7 @@ YUI({ logInclude: { TestRunner: true } }).use('console', 'test', function(Y) {
                         }
                         var error = null
                         try {
-                            rra.getEl(10, 0);
+                            rra.getEl(RRD_RRAROWS+1, 0);
                         } catch(e) {
                             error = e;
                         }
@@ -195,10 +196,10 @@ YUI({ logInclude: { TestRunner: true } }).use('console', 'test', function(Y) {
                     self.resume(function() {
                         var rq = new jarmon.RrdQuery(rrd, '');
                         var data = rq.getData(RRD_STARTTIME, RRD_ENDTIME);
-                        console.log(data.data);
                         Y.Assert.areEqual(RRD_RRAROWS, data.data.length);
                         Y.Assert.areEqual(2, data.data[2][1]);
-                        Y.Assert.areEqual(RRD_STARTTIME+RRD_STEP*1000, data.data[0][0]);
+                        Y.Assert.areEqual(
+                            RRD_STARTTIME+RRD_STEP*1000, data.data[0][0]);
                         //Y.Assert.areEqual(
                         //    RRD_ENDTIME, data.data[RRD_RRAROWS-1][0]);
                     });
diff --git a/jarmonbuild/commands.py b/jarmonbuild/commands.py
index 5823042..9ababd9 100644
--- a/jarmonbuild/commands.py
+++ b/jarmonbuild/commands.py
@@ -230,11 +230,12 @@ class BuildTestDataCommand(BuildCommand):
         rras = []
         filename = os.path.join(self.build_dir, 'test.rrd')
 
-        rows = 6
+        rows = 12
         step = 10
 
         dss.append(DataSource(dsName='speed', dsType='GAUGE', heartbeat=2*step))
         rras.append(RRA(cf='AVERAGE', xff=0.5, steps=1, rows=rows))
+        rras.append(RRA(cf='AVERAGE', xff=0.5, steps=12, rows=rows))
         my_rrd = RRD(filename, ds=dss, rra=rras, start=start, step=step)
         my_rrd.create()
 
-- 
cgit v1.2.3