From 513cfb87cf835c64472b3b5215963e88fedff7bd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=BDeljko=20Filipin?= <zeljko.filipin@gmail.com>
Date: Wed, 18 Nov 2015 12:04:04 +0100
Subject: [PATCH] Fixed Style/HashSyntax RuboCop offense

Bug: 117989
Change-Id: Iecdcc3202adf9458f11310d188ab93176b03194f
---
 .rubocop_todo.yml                                       | 6 ------
 tests/browser/features/step_definitions/common_steps.rb | 4 ++--
 tests/browser/features/step_definitions/ime_steps.rb    | 2 +-
 tests/browser/features/step_definitions/panel_steps.rb  | 2 +-
 4 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index 6ae373fb..2892d28f 100644
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -28,9 +28,3 @@ Style/CaseIndentation:
 Style/GlobalVars:
   Exclude:
     - 'tests/browser/features/step_definitions/accept_language_steps.rb'
-
-# Offense count: 7
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles, UseHashRocketsWithSymbolValues.
-Style/HashSyntax:
-  Enabled: false
diff --git a/tests/browser/features/step_definitions/common_steps.rb b/tests/browser/features/step_definitions/common_steps.rb
index 7660fd97..25c6a0c0 100644
--- a/tests/browser/features/step_definitions/common_steps.rb
+++ b/tests/browser/features/step_definitions/common_steps.rb
@@ -7,14 +7,14 @@ end
 
 Given(/^I set "(.*?)" as the interface language$/) do |language|
   code = on(PanelPage).language_to_code(language)
-  visit(PanelPage, :using_params => { :extra => "setlang=#{code}" })
+  visit(PanelPage, using_params: { extra: "setlang=#{code}" })
   @original_content_font = on(PanelPage).content_font
   @original_interface_font = on(PanelPage).interface_font
 end
 
 Given(/^I temporarily use "(.*?)" as the interface language$/) do |language|
   code = on(PanelPage).language_to_code(language)
-  visit(PanelPage, :using_params => { :extra => "uselang=#{code}" })
+  visit(PanelPage, using_params: { extra: "uselang=#{code}" })
 end
 
 Then(/^my interface language is "(.*?)"$/) do |language|
diff --git a/tests/browser/features/step_definitions/ime_steps.rb b/tests/browser/features/step_definitions/ime_steps.rb
index 96a2e2bd..02bdcb1d 100644
--- a/tests/browser/features/step_definitions/ime_steps.rb
+++ b/tests/browser/features/step_definitions/ime_steps.rb
@@ -69,7 +69,7 @@ Then(/^in it there must be an element with Malayalam text$/) do
 end
 
 Given(/^I visit a random page with (.+) skin and (.+) as the interface language$/) do |skin, language|
-  visit(IMEPage, :using_params => { :extra => "useskin=#{skin.downcase}&uselang=#{on(IMEPage).language_to_code(language)}" })
+  visit(IMEPage, using_params: { extra: "useskin=#{skin.downcase}&uselang=#{on(IMEPage).language_to_code(language)}" })
 end
 
 Then(/^I should see the input method menu is not offscreen$/) do
diff --git a/tests/browser/features/step_definitions/panel_steps.rb b/tests/browser/features/step_definitions/panel_steps.rb
index 3f077544..fbb6ab09 100644
--- a/tests/browser/features/step_definitions/panel_steps.rb
+++ b/tests/browser/features/step_definitions/panel_steps.rb
@@ -1,5 +1,5 @@
 Then(/^I see "(.*?)" as the name of the content language$/) do |text|
-  browser.span(:text => "#{text}").should be_visible
+  browser.span(text: "#{text}").should be_visible
 end
 
 When(/^I open the Universal Language Selector$/) do