punk 0.3.2 → 0.4.1

data/spec/models/punk/session_spec.rb

@@ -8,7 +8,7 @@ describe PUNK::Session do
   it "is assigned a uuid on save" do
     session = build(:session)
     expect(session.id).to be_nil
-    session.save
+    session.save_changes
     expect(valid_uuid?(session.id)).to be(true)
   end
 
@@ -21,7 +21,7 @@ describe PUNK::Session do
   it "must belong to an identity" do
     session = build(:session, identity: nil)
     expect(session.valid?).to be(false)
-    expect(session.errors[:identity].first).to eq('is not present')
+    expect(session.errors[:identity].first).to eq("is not present")
   end
 
   it "may have a user" do
@@ -32,8 +32,8 @@ describe PUNK::Session do
   end
 
   it "can contain client data" do
-    identity = create(:session, data: { foo: 'bar' })
-    expect(identity.data[:foo]).to eq('bar')
+    identity = create(:session, data: {foo: "bar"})
+    expect(identity.data[:foo]).to eq("bar")
   end
 
   it "permits only three validation attempts" do

data/spec/models/punk/tenant_spec.rb

@@ -8,7 +8,7 @@ describe PUNK::Tenant do
   it "is assigned a uuid on save" do
     tenant = build(:tenant)
     expect(tenant.id).to be_nil
-    tenant.save
+    tenant.save_changes
     expect(valid_uuid?(tenant.id)).to be(true)
   end
 
@@ -21,7 +21,7 @@ describe PUNK::Tenant do
   it "is invalid without a name" do
     tenant = build(:tenant, name: nil)
     expect(tenant.valid?).to be(false)
-    expect(tenant.errors[:name].first).to eq('is not present')
+    expect(tenant.errors[:name].first).to eq("is not present")
   end
 
   it "is valid without an icon" do
@@ -32,7 +32,7 @@ describe PUNK::Tenant do
   it "is invalid if the icon is not a URL" do
     tenant = build(:tenant, icon: Faker::Alphanumeric.alpha)
     expect(tenant.valid?).to be(false)
-    expect(tenant.errors[:icon].first).to eq('is not a URL')
+    expect(tenant.errors[:icon].first).to eq("is not a URL")
   end
 
   it "can have multiple users" do

data/spec/models/punk/tenant_user_metadata_spec.rb

@@ -8,13 +8,13 @@ describe PUNK::TenantUserMetadata do
   it "is invalid without a tenant" do
     tenant_user_metadata = build(:tenant_user_metadata, tenant: nil)
     expect(tenant_user_metadata.valid?).to be(false)
-    expect(tenant_user_metadata.errors[:tenant].first).to eq('is not present')
+    expect(tenant_user_metadata.errors[:tenant].first).to eq("is not present")
   end
 
   it "is invalid without a user" do
     tenant_user_metadata = build(:tenant_user_metadata, user: nil)
     expect(tenant_user_metadata.valid?).to be(false)
-    expect(tenant_user_metadata.errors[:user].first).to eq('is not present')
+    expect(tenant_user_metadata.errors[:user].first).to eq("is not present")
   end
 
   it "displays as the two IDs concatenated" do

data/spec/models/punk/user_spec.rb

@@ -8,7 +8,7 @@ describe PUNK::User do
   it "is assigned a uuid on save" do
     user = build(:user)
     expect(user.id).to be_nil
-    user.save
+    user.save_changes
     expect(valid_uuid?(user.id)).to be(true)
   end
 
@@ -21,7 +21,7 @@ describe PUNK::User do
   it "is invalid without a name" do
     user = build(:user, name: nil)
     expect(user.valid?).to be(false)
-    expect(user.errors[:name].first).to eq('is not present')
+    expect(user.errors[:name].first).to eq("is not present")
   end
 
   it "validates name presence in the database" do
@@ -37,7 +37,7 @@ describe PUNK::User do
   it "is invalid if the icon is not a URL" do
     user = build(:user, icon: Faker::Alphanumeric.alpha)
     expect(user.valid?).to be(false)
-    expect(user.errors[:icon].first).to eq('is not a URL')
+    expect(user.errors[:icon].first).to eq("is not a URL")
   end
 
   it "is valid without an email" do
@@ -48,14 +48,14 @@ describe PUNK::User do
   it "is invalid if the email is not an email address" do
     user = build(:user, email: Faker::Alphanumeric.alpha)
     expect(user.valid?).to be(false)
-    expect(user.errors[:email].first).to eq('is not an email address')
+    expect(user.errors[:email].first).to eq("is not an email address")
   end
 
   it "has a unique email" do
     email = create(:user).email
     user = build(:user, email: email)
     expect(user.valid?).to be(false)
-    expect(user.errors[:email].first).to eq('is already taken')
+    expect(user.errors[:email].first).to eq("is already taken")
   end
 
   it "validates email uniqueness in the database" do
@@ -72,14 +72,14 @@ describe PUNK::User do
   it "is invalid if the phone is not a phone number" do
     user = build(:user, phone: Faker::Alphanumeric.alpha)
     expect(user.valid?).to be(false)
-    expect(user.errors[:phone].first).to eq('is not a phone number')
+    expect(user.errors[:phone].first).to eq("is not a phone number")
   end
 
   it "has a unique phone" do
     phone = create(:user).phone
     user = build(:user, phone: phone)
     expect(user.valid?).to be(false)
-    expect(user.errors[:phone].first).to eq('is already taken')
+    expect(user.errors[:phone].first).to eq("is already taken")
   end
 
   it "validates phone uniqueness in the database" do
@@ -91,8 +91,8 @@ describe PUNK::User do
   it "is invalid without either an email or a phone" do
     user = build(:user, email: nil, phone: nil)
     user.valid?
-    expect(user.errors[:email].first).to eq('is not present')
-    expect(user.errors[:phone].first).to eq('is not present')
+    expect(user.errors[:email].first).to eq("is not present")
+    expect(user.errors[:phone].first).to eq("is not present")
   end
 
   it "can belong to multiple tenants" do

data/spec/routes/groups/get_groups_spec.rb

@@ -3,17 +3,17 @@
 describe PUNK, "GET /groups" do
   include_context "Punk"
 
-  context 'when the user is not authenticated' do
+  context "when the user is not authenticated" do
     before do
-      get '/groups'
+      get "/groups"
     end
 
     it { is_expected.not_to be_successful }
   end
 
-  context 'when the user is authenticated' do
+  context "when the user is authenticated" do
     let(:tenant) { create(:tenant) }
-    let(:identity) { create(:identity, claim_type: 'phone') }
+    let(:identity) { create(:identity, claim_type: "phone") }
     let(:group) { create(:group, tenant: tenant) }
 
     before do

data/spec/routes/plivo/get_plivo_spec.rb

@@ -4,7 +4,7 @@ describe PUNK, "GET /plivo" do
   include_context "Punk"
 
   before do
-    get '/plivo.html'
+    get "/plivo.html"
   end
 
   it { is_expected.to be_successful }

data/spec/routes/sessions/delete_session_spec.rb

@@ -4,7 +4,7 @@ describe PUNK, "DELETE /sessions" do
   include_context "Punk"
 
   before do
-    delete '/sessions'
+    delete "/sessions"
   end
 
   it { is_expected.not_to be_successful }

data/spec/routes/sessions/get_sessions_spec.rb

@@ -3,21 +3,21 @@
 describe PUNK, "GET /sessions" do
   include_context "Punk"
 
-  context 'when the user is not authenticated' do
+  context "when the user is not authenticated" do
     before do
-      get '/sessions'
+      get "/sessions"
     end
 
     it { is_expected.not_to be_successful }
   end
 
-  context 'when the user is authenticated' do
+  context "when the user is authenticated" do
     let(:user) { create(:user) }
-    let(:identity) { create(:identity, user: user, claim_type: 'phone') }
+    let(:identity) { create(:identity, user: user, claim_type: "phone") }
 
     before do
       login(identity.claim)
-      get '/sessions'
+      get "/sessions"
     end
 
     after do

data/spec/routes/sessions/patch_session_spec.rb

@@ -4,7 +4,7 @@ describe PUNK, "PATCH /sessions" do
   include_context "Punk"
 
   before do
-    patch '/sessions'
+    patch "/sessions"
   end
 
   it { is_expected.not_to be_successful }

data/spec/routes/sessions/post_session_spec.rb

@@ -4,7 +4,7 @@ describe PUNK, "POST /sessions" do
   include_context "Punk"
 
   before do
-    post '/sessions'
+    post "/sessions"
   end
 
   it { is_expected.not_to be_successful }

data/spec/routes/swagger/get_swagger_spec.rb

@@ -4,7 +4,7 @@ describe PUNK, "GET /swagger" do
   include_context "Punk"
 
   before do
-    get '/swagger'
+    get "/swagger"
   end
 
   it { is_expected.to be_successful }

data/spec/routes/tenants/get_tenants_spec.rb

@@ -3,22 +3,22 @@
 describe PUNK, "GET /tenants" do
   include_context "Punk"
 
-  context 'when the user is not authenticated' do
+  context "when the user is not authenticated" do
     before do
-      get '/tenants'
+      get "/tenants"
     end
 
     it { is_expected.not_to be_successful }
   end
 
-  context 'when the user is authenticated' do
+  context "when the user is authenticated" do
     let(:tenant) { create(:tenant) }
-    let(:identity) { create(:identity, claim_type: 'phone') }
+    let(:identity) { create(:identity, claim_type: "phone") }
 
     before do
       identity.user.add_tenant(tenant)
       login(identity.claim)
-      get '/tenants'
+      get "/tenants"
     end
 
     after do

data/spec/routes/users/get_users_spec.rb

@@ -3,17 +3,17 @@
 describe PUNK, "GET /users" do
   include_context "Punk"
 
-  context 'when the user is not authenticated' do
+  context "when the user is not authenticated" do
     before do
-      get '/users'
+      get "/users"
     end
 
     it { is_expected.not_to be_successful }
   end
 
-  context 'when the user is authenticated' do
+  context "when the user is authenticated" do
     let(:tenant) { create(:tenant) }
-    let(:identity) { create(:identity, claim_type: 'phone') }
+    let(:identity) { create(:identity, claim_type: "phone") }
     let(:group) { create(:group, tenant: tenant) }
     let(:tenant_user) { create(:user) }
     let(:group_user) { create(:user) }
@@ -31,7 +31,7 @@ describe PUNK, "GET /users" do
       logout
     end
 
-    context 'without a group' do
+    context "without a group" do
       before do
         get "/users?tenant_id=#{tenant.id}"
       end
@@ -44,7 +44,7 @@ describe PUNK, "GET /users" do
       end
     end
 
-    context 'with a group' do
+    context "with a group" do
       before do
         get "/users?tenant_id=#{tenant.id}&group_id=#{group.id}"
       end

data/spec/services/punk/challenge_claim_service_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 describe PUNK::ChallengeClaimService do
-  context 'with no specs written' do
-    it 'displays a warning'
+  context "with no specs written" do
+    it "displays a warning"
   end
 end

data/spec/services/punk/create_identities_service_spec.rb

@@ -6,7 +6,7 @@ describe PUNK::CreateIdentitiesService do
     create(:user, email: nil)
   end
 
-  it 'creates identities for users that are missing them' do
+  it "creates identities for users that are missing them" do
     expect(PUNK::Identity.count).to eq(0)
     described_class.run
     expect(PUNK::Identity.count).to eq(2)

data/spec/services/punk/generate_swagger_service_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 describe PUNK::GenerateSwaggerService do
-  context 'with no specs written' do
-    it 'displays a warning'
+  context "with no specs written" do
+    it "displays a warning"
   end
 end

data/spec/services/punk/prove_claim_service_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 describe PUNK::ProveClaimService do
-  context 'with no specs written' do
-    it 'displays a warning'
+  context "with no specs written" do
+    it "displays a warning"
   end
 end

data/spec/services/punk/secret_service_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 describe PUNK::SecretService do
-  context 'with no specs written' do
-    it 'displays a warning'
+  context "with no specs written" do
+    it "displays a warning"
   end
 end

data/spec/spec_helper.rb

@@ -1,46 +1,46 @@
 # frozen_string_literal: true
 
-require 'simplecov'
-require 'coveralls'
+require "simplecov"
+require "coveralls"
 SimpleCov.start do
-  add_filter 'spec'
-  add_filter 'lib/punk/commands/auth.rb'
-  add_filter 'lib/punk/commands/generate.rb'
-  add_filter 'lib/punk/commands/http.rb'
-  add_filter 'lib/punk/commands/list.rb'
-  add_filter 'lib/punk/core/commands.rb'
-  add_filter 'lib/punk/framework/command.rb'
-  add_filter 'lib/punk/plugins/cors.rb'
-  add_filter 'lib/punk/plugins/ssl.rb'
-  add_filter 'lib/punk/startup/logger.rb'
+  add_filter "spec"
+  add_filter "lib/punk/commands/auth.rb"
+  add_filter "lib/punk/commands/generate.rb"
+  add_filter "lib/punk/commands/http.rb"
+  add_filter "lib/punk/commands/list.rb"
+  add_filter "lib/punk/core/commands.rb"
+  add_filter "lib/punk/framework/command.rb"
+  add_filter "lib/punk/plugins/cors.rb"
+  add_filter "lib/punk/plugins/ssl.rb"
+  add_filter "lib/punk/startup/logger.rb"
 end
 Coveralls.wear!
 
-require_relative '../lib/punk'
+require_relative "../lib/punk"
 
-require 'factory_bot'
-require 'faker'
-require 'timecop'
-require 'securerandom'
-require 'vcr'
-require 'rack/test'
+require "factory_bot"
+require "faker"
+require "timecop"
+require "securerandom"
+require "vcr"
+require "rack/test"
 
-Faker::Config.locale = 'en-US'
+Faker::Config.locale = "en-US"
 
 FactoryBot.use_parent_strategy = false
 
 VCR.configure do |c|
   c.ignore_localhost = true
-  c.cassette_library_dir = 'spec/vcr_cassettes'
+  c.cassette_library_dir = "spec/vcr_cassettes"
   c.hook_into :webmock
   c.configure_rspec_metadata!
 end
 
-PUNK.init(task: 'spec', config: { app: { name: 'Punk Test' } }).exec
+PUNK.init(task: "spec", config: {app: {name: "Punk Test"}}).exec
 
-Sidekiq.logger = SemanticLogger['PUNK::SKQ']
+Sidekiq.logger = SemanticLogger["PUNK::SKQ"]
 
-RSpec.shared_context 'Punk' do # rubocop:disable RSpec/ContextWording
+RSpec.shared_context "Punk" do # rubocop:disable RSpec/ContextWording
   include Rack::Test::Methods
 
   subject { last_response }
@@ -57,7 +57,7 @@ RSpec.shared_context 'Punk' do # rubocop:disable RSpec/ContextWording
         "PATH_INFO" => "/sessions",
         "CONTENT_TYPE" => "text/json",
         "SCRIPT_NAME" => "",
-        "rack.input" => StringIO.new({ claim: claim }.to_json)
+        "rack.input" => StringIO.new({claim: claim}.to_json)
       )
     response = ActiveSupport::JSON.decode(response[-1].first).deep_symbolize_keys
     slug = response[:slug]
@@ -68,7 +68,7 @@ RSpec.shared_context 'Punk' do # rubocop:disable RSpec/ContextWording
       "PATH_INFO" => "/sessions/#{slug}",
       "CONTENT_TYPE" => "text/json",
       "SCRIPT_NAME" => "",
-      "rack.input" => StringIO.new({ secret: sms[:body][-7..-2] }.to_json)
+      "rack.input" => StringIO.new({secret: sms[:body][-7..-2]}.to_json)
     )
   end
 
@@ -84,7 +84,7 @@ end
 
 module Helpers
   def valid_uuid?(data)
-    data.split('-').map { |s| Integer(s, 16) }.length == 5
+    data.split("-").map { |s| Integer(s, 16) }.length == 5
   end
 end