From 9a28be7c6698073cadba43046e4397854bce8a6c Mon Sep 17 00:00:00 2001
From: Nemanja Glumac <31325167+nemanjaglumac@users.noreply.github.com>
Date: Wed, 25 May 2022 19:35:40 +0200
Subject: [PATCH] Repro #22929: Circle size not respected on a scatter chart
(#22945)
---
.../visualizations/scatter.cy.spec.js | 37 +++++++++++++++++++
1 file changed, 37 insertions(+)
diff --git a/frontend/test/metabase/scenarios/visualizations/scatter.cy.spec.js b/frontend/test/metabase/scenarios/visualizations/scatter.cy.spec.js
index 83a346fd1e4..83477f7acb9 100644
--- a/frontend/test/metabase/scenarios/visualizations/scatter.cy.spec.js
+++ b/frontend/test/metabase/scenarios/visualizations/scatter.cy.spec.js
@@ -85,6 +85,43 @@ describe("scenarios > visualizations > scatter", () => {
cy.findByText("Visualization");
cy.findAllByText("79").should("not.exist");
});
+
+ it("should respect circle size in a visualization (metabase#22929)", () => {
+ visitQuestionAdhoc({
+ dataset_query: {
+ type: "native",
+ native: {
+ query: `select 1 as size, 1 as x, 5 as y union all
+select 10 as size, 2 as x, 5 as y`,
+ },
+ database: SAMPLE_DB_ID,
+ },
+ display: "scatter",
+ visualization_settings: {
+ "scatter.bubble": "SIZE",
+ "graph.dimensions": ["X"],
+ "graph.metrics": ["Y"],
+ },
+ });
+
+ cy.get("circle").each((circle, index) => {
+ cy.wrap(circle)
+ .invoke("attr", "r")
+ .then(r => {
+ const rFloat = +r;
+
+ expect(rFloat).to.be.greaterThan(0);
+
+ cy.wrap(r).as("radius" + index);
+ });
+ });
+
+ cy.get("@radius0").then(r0 => {
+ cy.get("@radius1").then(r1 => {
+ assert.notEqual(r0, r1);
+ });
+ });
+ });
});
function triggerPopoverForBubble(index = 13) {
--
GitLab