punk 0.3.6 → 0.4.1

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)

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