punk 0.3.2 → 0.4.1

data/spec/actions/sessions/punk/list_sessions_action_spec.rb

@@ -1,26 +1,26 @@
 # frozen_string_literal: true
 
 describe PUNK::ListSessionsAction do
-  context 'with no user provided' do
-    it 'returns a validation error' do
+  context "with no user provided" do
+    it "returns a validation error" do
       view = described_class.run.result.render(:json)
-      expect(view).to match('user is not present')
+      expect(view).to match("user is not present")
     end
   end
 
-  context 'with a user provided' do
+  context "with a user provided" do
     let(:user) { create(:user) }
     let(:identity) { create(:identity, user: user) }
 
     before do
-      create_list(:session, 3, state: 'active')
-      create_list(:session, 3, state: 'active', identity: identity)
+      create_list(:session, 3, state: "active")
+      create_list(:session, 3, state: "active", identity: identity)
     end
 
-    it 'returns active sessions that the user belongs to' do
+    it "returns active sessions that the user belongs to" do
       expect(PUNK::Session.count).to eq(6)
       view = JSON.parse(described_class.run(user: user).result.render(:json))
-      expect(view.map { |h| h['id'] }.sort).to eq(user.active_sessions.map(&:id).sort)
+      expect(view.map { |h| h["id"] }.sort).to eq(user.active_sessions.map(&:id).sort)
     end
   end
 end

data/spec/actions/sessions/punk/verify_session_action_spec.rb

@@ -1,31 +1,31 @@
 # frozen_string_literal: true
 
 describe PUNK::VerifySessionAction do
-  context 'with no session provided' do
-    it 'returns a validation error' do
+  context "with no session provided" do
+    it "returns a validation error" do
       view = described_class.run.result.render(:json)
-      expect(view).to match('session is not present')
+      expect(view).to match("session is not present")
     end
   end
 
-  context 'with no secret provided' do
-    it 'returns a validation error' do
+  context "with no secret provided" do
+    it "returns a validation error" do
       session = create(:session)
       view = described_class.run(session: session).result.render(:json)
-      expect(view).to match('secret is not present')
+      expect(view).to match("secret is not present")
     end
   end
 
-  context 'with an inactive session provided' do
-    it 'returns a validation error' do
+  context "with an inactive session provided" do
+    it "returns a validation error" do
       session = create(:session)
-      view = described_class.run(session: session, secret: 'xyzzy').result.render(:json)
-      expect(view).to match('session is not in pending state')
-      expect(view).to match('session may not verify')
+      view = described_class.run(session: session, secret: "xyzzy").result.render(:json)
+      expect(view).to match("session is not in pending state")
+      expect(view).to match("session may not verify")
     end
   end
 
-  context 'with a pending session provided' do
+  context "with a pending session provided" do
     let(:session) { create(:session) }
 
     before do
@@ -33,26 +33,26 @@ describe PUNK::VerifySessionAction do
       PUNK::ChallengeClaimService.run(session: session)
     end
 
-    it 'returns an error if the secret is not correct' do
-      expect { described_class.run(session: session, secret: 'qwerty') }.to raise_error(PUNK::BadRequest, "Secret is incorrect")
+    it "returns an error if the secret is not correct" do
+      expect { described_class.run(session: session, secret: "qwerty") }.to raise_error(PUNK::BadRequest, "Secret is incorrect")
     end
 
-    it 'expires the session after three failed attempts' do
-      described_class.run(session: session, secret: 'qwerty') rescue nil # rubocop:disable Style/RescueModifier
-      described_class.run(session: session, secret: 'qwerty') rescue nil # rubocop:disable Style/RescueModifier
-      expect { described_class.run(session: session, secret: 'qwerty') }.to raise_error(PUNK::BadRequest, "Too many attempts")
+    it "expires the session after three failed attempts" do
+      described_class.run(session: session, secret: "qwerty") rescue nil # rubocop:disable Style/RescueModifier
+      described_class.run(session: session, secret: "qwerty") rescue nil # rubocop:disable Style/RescueModifier
+      expect { described_class.run(session: session, secret: "qwerty") }.to raise_error(PUNK::BadRequest, "Too many attempts")
     end
 
-    it 'expires the session if it is more than five minutes old' do
+    it "expires the session if it is more than five minutes old" do
       Timecop.travel(5.minutes.from_now)
-      view = described_class.run(session: session, secret: 'xyzzy').result.render(:json)
-      expect(view).to match('session may not verify')
+      view = described_class.run(session: session, secret: "xyzzy").result.render(:json)
+      expect(view).to match("session may not verify")
       expect(session.expired?).to be(true)
     end
 
-    it 'verifies the session if the secret is correct' do
-      view = described_class.run(session: session, secret: 'xyzzy').result.render(:json)
-      expect(view).to match('We have succesfully verified your identity')
+    it "verifies the session if the secret is correct" do
+      view = described_class.run(session: session, secret: "xyzzy").result.render(:json)
+      expect(view).to match("We have succesfully verified your identity")
       expect(session.active?).to be(true)
     end
   end

data/spec/actions/tenants/punk/list_tenants_action_spec.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
 describe PUNK::ListTenantsAction do
-  context 'with no user provided' do
-    it 'returns a validation error' do
+  context "with no user provided" do
+    it "returns a validation error" do
       view = described_class.run.result.render(:json)
-      expect(view).to match('user is not present')
+      expect(view).to match("user is not present")
     end
   end
 
-  context 'with a user provided' do
+  context "with a user provided" do
     let(:user) { create(:user) }
 
     before do
@@ -16,10 +16,10 @@ describe PUNK::ListTenantsAction do
       3.times { create(:tenant).add_user(user) }
     end
 
-    it 'returns tenants that the user belongs to' do
+    it "returns tenants that the user belongs to" do
       expect(PUNK::Tenant.count).to eq(6)
       view = JSON.parse(described_class.run(user: user).result.render(:json))
-      expect(view.map { |h| h['id'] }.sort).to eq(user.tenants.map(&:id).sort)
+      expect(view.map { |h| h["id"] }.sort).to eq(user.tenants.map(&:id).sort)
     end
   end
 end

data/spec/actions/users/punk/list_group_users_action_spec.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
 describe PUNK::ListGroupUsersAction do
-  context 'with no group provided' do
-    it 'returns a validation error' do
+  context "with no group provided" do
+    it "returns a validation error" do
       view = described_class.run.result.render(:json)
-      expect(view).to match('group is not present')
+      expect(view).to match("group is not present")
     end
   end
 
-  context 'with valid arguments' do
+  context "with valid arguments" do
     let(:group) { create(:group) }
     let(:users) { create_list(:user, 3) }
 
@@ -17,10 +17,10 @@ describe PUNK::ListGroupUsersAction do
       users.each { |user| group.add_user(user) }
     end
 
-    it 'returns users that are members of the given group' do
+    it "returns users that are members of the given group" do
       expect(PUNK::User.count).to eq(5)
       view = JSON.parse(described_class.run(group: group).result.render(:json))
-      expect(view.map { |h| h['id'] }.sort).to eq(group.users.map(&:id).sort)
+      expect(view.map { |h| h["id"] }.sort).to eq(group.users.map(&:id).sort)
     end
   end
 end

data/spec/actions/users/punk/list_tenant_users_action_spec.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
 describe PUNK::ListTenantUsersAction do
-  context 'with no tenant provided' do
-    it 'returns a validation error' do
+  context "with no tenant provided" do
+    it "returns a validation error" do
       view = described_class.run.result.render(:json)
-      expect(view).to match('tenant is not present')
+      expect(view).to match("tenant is not present")
     end
   end
 
-  context 'with valid arguments' do
+  context "with valid arguments" do
     let(:tenant) { create(:tenant) }
     let(:users) { create_list(:user, 3) }
 
@@ -17,10 +17,10 @@ describe PUNK::ListTenantUsersAction do
       users.each { |user| tenant.add_user(user) }
     end
 
-    it 'returns users that are members of the given tenant' do
+    it "returns users that are members of the given tenant" do
       expect(PUNK::User.count).to eq(5)
       view = JSON.parse(described_class.run(tenant: tenant).result.render(:json))
-      expect(view.map { |h| h['id'] }.sort).to eq(tenant.users.map(&:id).sort)
+      expect(view.map { |h| h["id"] }.sort).to eq(tenant.users.map(&:id).sort)
     end
   end
 end

data/spec/factories/group.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :group, class: 'PUNK::Group' do
+  factory :group, class: "PUNK::Group" do
     to_create(&:save)
 
     tenant

data/spec/factories/group_user_metadata.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :group_user_metadata, class: 'PUNK::GroupUserMetadata' do
+  factory :group_user_metadata, class: "PUNK::GroupUserMetadata" do
     to_create(&:save)
 
     group

data/spec/factories/identity.rb

@@ -1,17 +1,17 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :identity, class: 'PUNK::Identity' do
+  factory :identity, class: "PUNK::Identity" do
     to_create(&:save)
 
     user
 
-    claim_type { ['email', 'phone'].sample }
+    claim_type { ["email", "phone"].sample }
     claim do
       case claim_type
-      when 'email'
+      when "email"
         Faker::Internet.email
-      when 'phone'
+      when "phone"
         generate(:phone)
       end
     end

data/spec/factories/session.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :session, class: 'PUNK::Session' do
+  factory :session, class: "PUNK::Session" do
     to_create(&:save)
 
     identity

data/spec/factories/tenant.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :tenant, class: 'PUNK::Tenant' do
+  factory :tenant, class: "PUNK::Tenant" do
     to_create(&:save)
 
     name { Faker::Name.name }

data/spec/factories/tenant_user_metadata.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :tenant_user_metadata, class: 'PUNK::TenantUserMetadata' do
+  factory :tenant_user_metadata, class: "PUNK::TenantUserMetadata" do
     to_create(&:save)
 
     tenant

data/spec/factories/user.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :user, class: 'PUNK::User' do
+  factory :user, class: "PUNK::User" do
     to_create(&:save)
 
     name { Faker::Name.name }

data/spec/lib/commands/generate_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-describe 'generate command', type: :feature do
-  context 'when no tests have been written' do
+describe "generate command", type: :feature do
+  context "when no tests have been written" do
     it 'displays "not yet implemented"'
   end
 end

data/spec/lib/commands/list_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-describe 'list command', type: :feature do
-  context 'when no tests have been written' do
+describe "list command", type: :feature do
+  context "when no tests have been written" do
     it 'displays "not yet implemented"'
   end
 end

data/spec/lib/commands/swagger_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-describe 'swagger command', type: :feature do
-  context 'when no tests have been written' do
+describe "swagger command", type: :feature do
+  context "when no tests have been written" do
     it 'displays "not yet implemented"'
   end
 end

data/spec/lib/engine/punk_env_spec.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
 describe PUNK do
-  describe '.env' do
-    it 'loads environment' do
-      expect(described_class.get.trace).to eq('spec_test')
+  describe ".env" do
+    it "loads environment" do
+      expect(described_class.get.trace).to eq("spec_test")
     end
 
-    it 'loads configuration' do
+    it "loads configuration" do
       expect(described_class.env.to_sym).to eq(:test)
     end
   end

data/spec/lib/engine/punk_exec_spec.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 describe PUNK do
-  describe '.exec' do
-    it 'starts the PUNK engine' do
+  describe ".exec" do
+    it "starts the PUNK engine" do
       expect(described_class.store.state).to eq(:started)
     end
   end

data/spec/lib/engine/punk_init_spec.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 describe PUNK do
-  describe '.init' do
-    it 'cannot be called twice' do
+  describe ".init" do
+    it "cannot be called twice" do
       expect { described_class.init }.to raise_error(PUNK::InternalServerError, "Cannot call PUNK.init multiple times!")
     end
   end

data/spec/lib/engine/punk_store_spec.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 describe PUNK do
-  describe '.store' do
-    it 'acts like a dotted hash' do
+  describe ".store" do
+    it "acts like a dotted hash" do
       described_class.store["foo"] = "bar"
       expect(described_class.store.foo).to eq("bar")
     end

data/spec/models/punk/group_spec.rb

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

data/spec/models/punk/group_user_metadata_spec.rb

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

data/spec/models/punk/identity_spec.rb

@@ -1,58 +1,58 @@
 # frozen_string_literal: true
 
 describe PUNK::Identity do
-  it 'is valid with valid attributes' do
+  it "is valid with valid attributes" do
     expect { create(:identity) }.not_to raise_error
   end
 
-  it 'must claim an email or a phone' do
-    identity = build(:identity, claim_type: 'zork', claim: 'xyzzy')
+  it "must claim an email or a phone" do
+    identity = build(:identity, claim_type: "zork", claim: "xyzzy")
     expect(identity.valid?).to be(false)
-    expect(identity.errors[:claim_type].first).to eq('is not in range or set: [:email, :phone]')
+    expect(identity.errors[:claim_type].first).to eq("is not in range or set: [:email, :phone]")
   end
 
-  it 'is assigned a uuid on save' do
+  it "is assigned a uuid on save" do
     identity = build(:identity)
     expect(identity.id).to be_nil
-    identity.save
+    identity.save_changes
     expect(valid_uuid?(identity.id)).to be(true)
   end
 
-  it 'can be saved with a custom uuid' do
+  it "can be saved with a custom uuid" do
     uuid = generate(:uuid)
     identity = create(:identity, id: uuid)
     expect(identity.id).to eq(uuid)
   end
 
-  it 'may belong to a user' do
+  it "may belong to a user" do
     identity_with_user = create(:identity)
     expect(identity_with_user.user).to exist
     identity_without_user = create(:identity, user: nil)
     expect(identity_without_user.user).to be_nil
   end
 
-  it 'has an email? accessor' do
-    identity_with_email = build(:identity, claim_type: 'email')
+  it "has an email? accessor" do
+    identity_with_email = build(:identity, claim_type: "email")
     expect(identity_with_email.email?).to be(true)
-    identity_without_email = build(:identity, claim_type: 'phone')
+    identity_without_email = build(:identity, claim_type: "phone")
     expect(identity_without_email.email?).to be(false)
   end
 
-  it 'has a phone? accessor' do
-    identity_with_phone = create(:identity, claim_type: 'phone')
+  it "has a phone? accessor" do
+    identity_with_phone = create(:identity, claim_type: "phone")
     expect(identity_with_phone.phone?).to be(true)
-    identity_without_phone = create(:identity, claim_type: 'email')
+    identity_without_phone = create(:identity, claim_type: "email")
     expect(identity_without_phone.phone?).to be(false)
   end
 
-  it 'has a unique claim' do
+  it "has a unique claim" do
     identity = create(:identity)
     duplicate_identity = build(:identity, claim_type: identity.claim_type, claim: identity.claim)
     expect(duplicate_identity.valid?).to be(false)
-    expect(duplicate_identity.errors[:claim].first).to eq('is already taken')
+    expect(duplicate_identity.errors[:claim].first).to eq("is already taken")
   end
 
-  it 'can have multiple sessions' do
+  it "can have multiple sessions" do
     identity = create(:identity)
     expect(identity.sessions.count).to eq(0)
     create_list(:session, 3, identity: identity)