fix: reverse data correctly #1250 (#1364)

chartist-js · Oct 4, 2022 · 0223b1f · 0223b1f
1 parent 69389e8
commit 0223b1f
Show file tree

Hide file tree

Showing 8 changed files with 491 additions and 410 deletions.
diff --git a/.size-limit.json b/.size-limit.json
@@ -7,7 +7,7 @@
   },
   {
     "path": "dist/index.cjs",
-    "limit": "7.3 kB",
+    "limit": "7.35 kB",
     "import": "{ BarChart }"
   },
   {

diff --git a/src/charts/BarChart/BarChart.spec.ts b/src/charts/BarChart/BarChart.spec.ts
@@ -399,5 +399,33 @@ describe('Charts', () => {
         ).toBeDefined();
       });
     });
+
+    it('should correct apply class names', async () => {
+      data = {
+        labels: ['A', 'B', 'C'],
+        series: [
+          {
+            className: 'series-1',
+            data: [1, 2, 3]
+          },
+          {
+            className: 'series-2',
+            data: [4, 5, 6]
+          }
+        ]
+      };
+      options = {
+        reverseData: true
+      };
+      await createChart();
+
+      const seriesElements = document.querySelectorAll('.ct-series');
+
+      expect(seriesElements[0]).toHaveClass('series-2');
+      expect(seriesElements[0]).toContainHTML('ct:value="6"');
+
+      expect(seriesElements[1]).toHaveClass('series-1');
+      expect(seriesElements[1]).toContainHTML('ct:value="3"');
+    });
   });
 });