Fix unrelated flapping test

Signed-off-by: Andrew Stein <steinlink@gmail.com>

Author: texodus

rust/perspective-client/src/rust/config/view_config.rs

@@ -416,13 +416,13 @@ impl ViewConfig {
                 .map(|x| !x.is_empty())
                 .unwrap_or_default()
         {
-            tracing::warn!("`total` incompatible with `group_by`");
+            tracing::info!("`total` incompatible with `group_by`");
             changed = true;
             update.group_rollup_mode = Some(GroupRollupMode::Rollup);
         }
 
         if update.group_rollup_mode == Some(GroupRollupMode::Total) && !self.group_by.is_empty() {
-            tracing::warn!("`group_by` incompatible with `total`");
+            tracing::info!("`group_by` incompatible with `total`");
             changed = true;
             update.group_by = Some(vec![]);
         }
@@ -436,7 +436,7 @@ impl ViewConfig {
         changed = Self::_apply(&mut self.expressions, update.expressions) || changed;
         changed = Self::_apply(&mut self.group_rollup_mode, update.group_rollup_mode) || changed;
         if self.group_rollup_mode == GroupRollupMode::Total && !self.group_by.is_empty() {
-            tracing::warn!("`total` incompatible with `group_by`");
+            tracing::info!("`total` incompatible with `group_by`");
             changed = true;
             self.group_by = vec![];
         }

rust/perspective-viewer/src/rust/components/column_selector/active_column.rs

@@ -336,7 +336,6 @@ impl Component for ActiveColumn {
                     let event_name = name.to_owned();
                     let dragdrop = ctx.props().dragdrop.clone();
                     move |event: DragEvent| {
-                        tracing::error!("dragstart");
                         dragdrop.set_drag_image(&event).unwrap();
                         dragdrop.notify_drag_start(
                             event_name.to_string(),

rust/perspective-viewer/src/rust/session/column_defaults_update.rs

@@ -43,7 +43,7 @@ pub impl ViewConfigUpdate {
                 .unwrap_or(&GroupRollupMode::Rollup),
         ) {
             self.group_rollup_mode = group_rollups.first().cloned();
-            tracing::error!(
+            tracing::debug!(
                 "Setting plugin-advised rollup mode {:?}",
                 self.group_rollup_mode
             );

rust/perspective-viewer/test/js/column_settings/sidebar.spec.ts

@@ -207,14 +207,20 @@ test.describe("Column Settings Sidebar", () => {
                 .locator(".tab-title")
                 .getAttribute("id");
         };
+
         expect(await selectedTab()).toBe("Attributes");
         await col.activeBtn.click();
+        await view.columnSettingsSidebar.container
+            .locator(".tab-title#Style")
+            .waitFor({ state: "visible" });
+
         await checkTab(view.columnSettingsSidebar, true, true, true);
         expect(await selectedTab()).toBe("Attributes");
         await view.columnSettingsSidebar.attributesTab.expressionEditor.textarea.clear();
         await view.columnSettingsSidebar.attributesTab.expressionEditor.textarea.type(
             "'new expr value'",
         );
+
         await view.columnSettingsSidebar.attributesTab.saveBtn.click();
         expect(await selectedTab()).toBe("Attributes");
     });