From 1987fce01898cd593f2b0dfab2a5492f59387061 Mon Sep 17 00:00:00 2001 From: Alan Guo Xiang Tan Date: Wed, 21 Jun 2023 08:09:39 +0800 Subject: [PATCH] DEV: Improve skip test comment (#22215) Why this change? The comment consists of an output that was copied from RSpec's default output. This has the potential to mess with systems that are parsing RSpec's output to fetch the spec failures as those systems are usually looking for the first occurence of `Failures:` --- spec/system/network_disconnected_spec.rb | 3 --- 1 file changed, 3 deletions(-) diff --git a/spec/system/network_disconnected_spec.rb b/spec/system/network_disconnected_spec.rb index be70bd25778..8749525692b 100644 --- a/spec/system/network_disconnected_spec.rb +++ b/spec/system/network_disconnected_spec.rb @@ -6,9 +6,6 @@ RSpec.describe "Network Disconnected", type: :system do before { skip(<<~TEXT) } This group of tests is flaky and needs to be fixed. Example of error: - Failures: - - 1) Network Disconnected Doesn't show the offline indicator when the site setting isn't present Failure/Error: expect(page).to have_css("html.message-bus-offline") expected to find css "html.message-bus-offline" but there were no matches TEXT