summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLamont Granquist <lamont@scriptkiddie.org>2018-06-26 13:02:29 -0700
committerLamont Granquist <lamont@scriptkiddie.org>2018-06-26 13:02:29 -0700
commitcfde9c1490bfbaa9f4fc2bc390955117d1b25b31 (patch)
treef05c50c941d5ea2c6d8173c3337a297d3c8707d2
parent0cfadddb41abfafcf5bca47ecb2c7760f1b517a4 (diff)
downloadmixlib-authentication-cfde9c1490bfbaa9f4fc2bc390955117d1b25b31.tar.gz
remove hashrocket syntax
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
-rw-r--r--.rubocop.yml2
-rw-r--r--Rakefile4
-rw-r--r--spec/mixlib/authentication/http_authentication_request_spec.rb24
-rw-r--r--spec/mixlib/authentication/mixlib_authentication_spec.rb54
4 files changed, 43 insertions, 41 deletions
diff --git a/.rubocop.yml b/.rubocop.yml
index e69de29..5b837e2 100644
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -0,0 +1,2 @@
+Style/HashSyntax:
+ Enabled: true
diff --git a/Rakefile b/Rakefile
index dfcf8fd..7be508e 100644
--- a/Rakefile
+++ b/Rakefile
@@ -3,7 +3,7 @@ require "rspec/core/rake_task"
RSpec::Core::RakeTask.new(:spec)
-task :default => :spec
+task default: :spec
begin
require "chefstyle"
@@ -15,4 +15,4 @@ rescue LoadError
puts "chefstyle/rubocop is not available."
end
-task :ci => [:style, :spec]
+task ci: [:style, :spec]
diff --git a/spec/mixlib/authentication/http_authentication_request_spec.rb b/spec/mixlib/authentication/http_authentication_request_spec.rb
index 0025cf4..7c45df1 100644
--- a/spec/mixlib/authentication/http_authentication_request_spec.rb
+++ b/spec/mixlib/authentication/http_authentication_request_spec.rb
@@ -64,18 +64,18 @@ describe Mixlib::Authentication::HTTPAuthenticationRequest do
end
it "normalizes the headers to lowercase symbols" do
- expected = { :host => "127.0.0.1",
- :x_ops_sign => "version=1.0",
- :x_ops_requestid => "127.0.0.1 1258566194.85386",
- :x_ops_timestamp => "2009-01-01T12:00:00Z",
- :x_ops_content_hash => "DFteJZPVv6WKdQmMqZUQUumUyRs=",
- :x_ops_userid => "spec-user",
- :x_ops_authorization_1 => "jVHrNniWzpbez/eGWjFnO6lINRIuKOg40ZTIQudcFe47Z9e/HvrszfVXlKG4",
- :x_ops_authorization_2 => "NMzYZgyooSvU85qkIUmKuCqgG2AIlvYa2Q/2ctrMhoaHhLOCWWoqYNMaEqPc",
- :x_ops_authorization_3 => "3tKHE+CfvP+WuPdWk4jv4wpIkAz6ZLxToxcGhXmZbXpk56YTmqgBW2cbbw4O",
- :x_ops_authorization_4 => "IWPZDHSiPcw//AYNgW1CCDptt+UFuaFYbtqZegcBd2n/jzcWODA7zL4KWEUy",
- :x_ops_authorization_5 => "9q4rlh/+1tBReg60QdsmDRsw/cdO1GZrKtuCwbuD4+nbRdVBKv72rqHX9cu0",
- :x_ops_authorization_6 => "utju9jzczCyB+sSAQWrxSsXB/b8vV2qs0l4VD2ML+w==" }
+ expected = { host: "127.0.0.1",
+ x_ops_sign: "version=1.0",
+ x_ops_requestid: "127.0.0.1 1258566194.85386",
+ x_ops_timestamp: "2009-01-01T12:00:00Z",
+ x_ops_content_hash: "DFteJZPVv6WKdQmMqZUQUumUyRs=",
+ x_ops_userid: "spec-user",
+ x_ops_authorization_1: "jVHrNniWzpbez/eGWjFnO6lINRIuKOg40ZTIQudcFe47Z9e/HvrszfVXlKG4",
+ x_ops_authorization_2: "NMzYZgyooSvU85qkIUmKuCqgG2AIlvYa2Q/2ctrMhoaHhLOCWWoqYNMaEqPc",
+ x_ops_authorization_3: "3tKHE+CfvP+WuPdWk4jv4wpIkAz6ZLxToxcGhXmZbXpk56YTmqgBW2cbbw4O",
+ x_ops_authorization_4: "IWPZDHSiPcw//AYNgW1CCDptt+UFuaFYbtqZegcBd2n/jzcWODA7zL4KWEUy",
+ x_ops_authorization_5: "9q4rlh/+1tBReg60QdsmDRsw/cdO1GZrKtuCwbuD4+nbRdVBKv72rqHX9cu0",
+ x_ops_authorization_6: "utju9jzczCyB+sSAQWrxSsXB/b8vV2qs0l4VD2ML+w==" }
expect(@http_authentication_request.headers).to eq(expected)
end
diff --git a/spec/mixlib/authentication/mixlib_authentication_spec.rb b/spec/mixlib/authentication/mixlib_authentication_spec.rb
index 2c233a2..6985741 100644
--- a/spec/mixlib/authentication/mixlib_authentication_spec.rb
+++ b/spec/mixlib/authentication/mixlib_authentication_spec.rb
@@ -316,45 +316,45 @@ PATH = "/organizations/clownco"
HASHED_CANONICAL_PATH = "YtBWDn1blGGuFIuKksdwXzHU9oE=" # Base64.encode64(Digest::SHA1.digest("/organizations/clownco")).chomp
V1_0_ARGS = {
- :body => BODY,
- :user_id => USER_ID,
- :http_method => :post,
- :timestamp => TIMESTAMP_ISO8601, # fixed timestamp so we get back the same answer each time.
- :file => MockFile.new,
- :path => PATH,
+ body: BODY,
+ user_id: USER_ID,
+ http_method: :post,
+ timestamp: TIMESTAMP_ISO8601, # fixed timestamp so we get back the same answer each time.
+ file: MockFile.new,
+ path: PATH,
}
V1_1_ARGS = {
- :body => BODY,
- :user_id => USER_ID,
- :http_method => :post,
- :timestamp => TIMESTAMP_ISO8601, # fixed timestamp so we get back the same answer each time.
- :file => MockFile.new,
- :path => PATH,
- :proto_version => 1.1,
+ body: BODY,
+ user_id: USER_ID,
+ http_method: :post,
+ timestamp: TIMESTAMP_ISO8601, # fixed timestamp so we get back the same answer each time.
+ file: MockFile.new,
+ path: PATH,
+ proto_version: 1.1,
}
V1_3_ARGS_SHA256 = {
- :body => BODY,
- :user_id => USER_ID,
- :http_method => :post,
- :timestamp => TIMESTAMP_ISO8601, # fixed timestamp so we get back the same answer each time.
- :file => MockFile.new,
- :path => PATH,
- :proto_version => "1.3",
- :headers => {
+ body: BODY,
+ user_id: USER_ID,
+ http_method: :post,
+ timestamp: TIMESTAMP_ISO8601, # fixed timestamp so we get back the same answer each time.
+ file: MockFile.new,
+ path: PATH,
+ proto_version: "1.3",
+ headers: {
"X-OpS-SeRvEr-ApI-VerSiOn" => "1",
}
# This defaults to sha256
}
LONG_PATH_LONG_USER_ARGS = {
- :body => BODY,
- :user_id => "A" * 200,
- :http_method => :put,
- :timestamp => TIMESTAMP_ISO8601, # fixed timestamp so we get back the same answer each time.
- :file => MockFile.new,
- :path => PATH + "/nodes/#{"A" * 250}",
+ body: BODY,
+ user_id: "A" * 200,
+ http_method: :put,
+ timestamp: TIMESTAMP_ISO8601, # fixed timestamp so we get back the same answer each time.
+ file: MockFile.new,
+ path: PATH + "/nodes/#{"A" * 250}",
}
REQUESTING_ACTOR_ID = "c0f8a68c52bffa1020222a56b23cccfa"