summaryrefslogtreecommitdiff
path: root/test/unit/account_test.rb
diff options
context:
space:
mode:
authorAzul <azul@riseup.net>2017-11-16 13:17:22 +0100
committerAzul <azul@riseup.net>2017-11-16 13:17:22 +0100
commitae0aa02d038f16bb0919eebee805624bb891f48f (patch)
tree647812d41a0a04e1505120655642372fb73ce0b9 /test/unit/account_test.rb
parentf250a11b4fd98bb9f4ef50c501b72f5ae2a97d1d (diff)
minor: clean up account test
also ensures that created user is cleaned up even if test fails
Diffstat (limited to 'test/unit/account_test.rb')
-rw-r--r--test/unit/account_test.rb163
1 files changed, 76 insertions, 87 deletions
diff --git a/test/unit/account_test.rb b/test/unit/account_test.rb
index 61b40b5..ebfff6b 100644
--- a/test/unit/account_test.rb
+++ b/test/unit/account_test.rb
@@ -8,164 +8,153 @@ class AccountTest < ActiveSupport::TestCase
end
teardown do
+ @user.account.destroy if @user && @user.persisted?
Identity.destroy_all_orphaned
end
test "create a new account when invited" do
- user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
- assert user.valid?, "unexpected errors: #{user.errors.inspect}"
- assert user.persisted?
- assert id = user.identity
- assert_equal user.email_address, id.address
- assert_equal user.email_address, id.destination
- user.account.destroy
+ @user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
+ assert @user.valid?, "unexpected errors: #{@user.errors.inspect}"
+ assert @user.persisted?
+ assert id = @user.identity
+ assert_equal @user.email_address, id.address
+ assert_equal @user.email_address, id.destination
end
test "fail to create account without invite" do
with_config invite_required: true do
- user = Account.create(user_attributes)
- assert !user.valid?, "user should not be valid"
- assert !user.persisted?, "user should not have been saved"
- assert_has_errors user, invite_code: "This is not a valid code"
+ @user = Account.create(user_attributes)
+ assert !@user.valid?, "user should not be valid"
+ assert !@user.persisted?, "user should not have been saved"
+ assert_has_errors @user, invite_code: "This is not a valid code"
end
end
test "allow invite_required override" do
with_config invite_required: true do
- user = Account.create(user_attributes, :invite_required => false)
- assert user.valid?, "unexpected errors: #{user.errors.inspect}"
- assert user.persisted?, "user should have been saved"
- user.account.destroy
+ @user = Account.create(user_attributes, :invite_required => false)
+ assert @user.valid?, "unexpected errors: #{@user.errors.inspect}"
+ assert @user.persisted?, "user should have been saved"
end
end
test "create a new account" do
with_config invite_required: false do
- user = Account.create(user_attributes)
- assert user.valid?, "unexpected errors: #{user.errors.inspect}"
- assert user.persisted?
- user.account.destroy
+ @user = Account.create(user_attributes)
+ assert @user.valid?, "unexpected errors: #{@user.errors.inspect}"
+ assert @user.persisted?
end
end
test "error on reused username" do
with_config invite_required: false do
attributes = user_attributes
- user = Account.create attributes
+ @user = Account.create attributes
dup = Account.create attributes
assert !dup.valid?
assert_has_errors dup, login: "has already been taken"
- user.account.destroy
end
end
test "error on invalid username" do
with_config invite_required: false do
attributes = FactoryGirl.attributes_for :user, login: "a"
- user = Account.create attributes
- assert !user.valid?
- assert_has_errors user, login: "Must have at least two characters"
+ @user = Account.create attributes
+ assert !@user.valid?
+ assert_has_errors @user, login: "Must have at least two characters"
end
end
- test "create and remove a user account" do
+ test "create and remove a @user account" do
# We keep an identity that will block the handle from being reused.
assert_difference "Identity.count" do
assert_no_difference "User.count" do
- user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
- user.account.destroy
+ @user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
+ @user.account.destroy
end
end
end
test "change username and create alias" do
- user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
- old_id = user.identity
- old_email = user.email_address
- user.account.update(user_attributes)
- user.reload
+ @user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
+ old_id = @user.identity
+ old_email = @user.email_address
+ @user.account.update(user_attributes)
+ @user.reload
old_id.reload
- assert user.valid?
- assert user.persisted?
- assert id = user.identity
+ assert @user.valid?
+ assert @user.persisted?
+ assert id = @user.identity
assert id.persisted?
- assert_equal user.email_address, id.address
- assert_equal user.email_address, id.destination
- assert_equal user.email_address, old_id.destination
+ assert_equal @user.email_address, id.address
+ assert_equal @user.email_address, id.destination
+ assert_equal @user.email_address, old_id.destination
assert_equal old_email, old_id.address
- user.account.destroy
end
test "create recovery code if it does not exist" do
- user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
- user.account.update(:recovery_code_verifier => "abc", :recovery_code_salt => "123")
- user.reload
+ @user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
+ @user.account.update(:recovery_code_verifier => "abc", :recovery_code_salt => "123")
+ @user.reload
- assert_equal "abc", user.recovery_code_verifier
- assert_equal "123", user.recovery_code_salt
-
- user.account.destroy
+ assert_equal "abc", @user.recovery_code_verifier
+ assert_equal "123", @user.recovery_code_salt
end
test "update recovery code that already exists" do
- user = Account.create(user_attributes(
+ @user = Account.create(user_attributes(
:invite_code => @testcode.invite_code,
:recovery_code_verifier => "000",
:recovery_code_salt => "111"))
- user.account.update(:recovery_code_verifier => "abc", :recovery_code_salt => "123")
- user.reload
-
- assert_equal "abc", user.recovery_code_verifier
- assert_equal "123", user.recovery_code_salt
+ @user.account.update(:recovery_code_verifier => "abc", :recovery_code_salt => "123")
+ @user.reload
- user.account.destroy
+ assert_equal "abc", @user.recovery_code_verifier
+ assert_equal "123", @user.recovery_code_salt
end
test "update password" do
- user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
- user.account.update(:password_verifier => "551A8B", :password_salt => "551A8B")
-
- assert_equal "551A8B", user.password_verifier
- assert_equal "551A8B", user.password_salt
+ @user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
+ @user.account.update(:password_verifier => "551A8B", :password_salt => "551A8B")
- user.account.destroy
+ assert_equal "551A8B", @user.password_verifier
+ assert_equal "551A8B", @user.password_salt
end
test "Invite code count goes up by 1 when the invite code is entered" do
with_config invite_required: true do
- user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
- user_code = InviteCode.find_by_invite_code user.invite_code
- user.save
- assert user.persisted?
+ @user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
+ user_code = InviteCode.find_by_invite_code @user.invite_code
+ @user.save
+ assert @user.persisted?
assert_equal 1, user_code.invite_count
end
end
- test "Single use invite code is destroyed when used by test user" do
+ test "Single use invite code is destroyed when used by test @user" do
with_config invite_required: true do
attrs = user_attributes invite_code: @testcode.invite_code
attrs[:login] = 'test_user_' + attrs[:login]
- user = Account.create(attrs)
- user.save
- assert user.persisted?, user.errors.inspect
- assert_nil InviteCode.find_by_invite_code user.invite_code
+ @user = Account.create(attrs)
+ @user.save
+ assert @user.persisted?, @user.errors.inspect
+ assert_nil InviteCode.find_by_invite_code @user.invite_code
end
end
- test "Single use invite code is destroyed when used by tmp user" do
+ test "Single use invite code is destroyed when used by tmp @user" do
with_config invite_required: true do
attrs = user_attributes invite_code: @testcode.invite_code
attrs[:login] = 'tmp_user_' + attrs[:login]
- user = Account.create(attrs)
- user.save
- assert user.persisted?, user.errors.inspect
- assert_nil InviteCode.find_by_invite_code user.invite_code
+ @user = Account.create(attrs)
+ @user.save
+ assert @user.persisted?, @user.errors.inspect
+ assert_nil InviteCode.find_by_invite_code @user.invite_code
end
end
test "Invite code stays zero when invite code is not used" do
- #user = Account.create(user_attributes( :invite_code => @testcode.invite_code))
invalid_user = FactoryGirl.build(:user, :invite_code => @testcode.invite_code)
invalid_user.save
user_code = InviteCode.find_by_invite_code invalid_user.invite_code
@@ -175,16 +164,16 @@ class AccountTest < ActiveSupport::TestCase
end
test "disabled accounts have no cert fingerprints" do
- user = Account.create(user_attributes)
+ @user = Account.create(user_attributes)
cert = stub(expiry: 1.month.from_now, fingerprint: SecureRandom.hex)
- user.identity.register_cert cert
- user.identity.save
- assert_equal(cert.fingerprint, Identity.for(user).cert_fingerprints.keys.first)
- user.account.disable
- assert_equal({}, Identity.for(user).cert_fingerprints)
- assert_equal(cert.fingerprint, Identity.for(user).read_attribute(:disabled_cert_fingerprints).keys.first)
- user.account.enable
- assert_equal(cert.fingerprint, Identity.for(user).cert_fingerprints.keys.first)
+ @user.identity.register_cert cert
+ @user.identity.save
+ assert_equal(cert.fingerprint, Identity.for(@user).cert_fingerprints.keys.first)
+ @user.account.disable
+ assert_equal({}, Identity.for(@user).cert_fingerprints)
+ assert_equal(cert.fingerprint, Identity.for(@user).read_attribute(:disabled_cert_fingerprints).keys.first)
+ @user.account.enable
+ assert_equal(cert.fingerprint, Identity.for(@user).cert_fingerprints.keys.first)
end
# Pixelated relies on the ability to test invite codes without sending a
@@ -192,15 +181,15 @@ class AccountTest < ActiveSupport::TestCase
# So we better make sure we return the appropriate errors
test "errors trying to create account with invite only" do
with_config invite_required: true do
- user = Account.create invite_code: @testcode.invite_code
- assert user.errors[:invite_code].blank?
+ @user = Account.create invite_code: @testcode.invite_code
+ assert @user.errors[:invite_code].blank?
end
end
test "errors trying to create account with invalid invite only" do
with_config invite_required: true do
- user = Account.create invite_code: "wrong_invite_code"
- assert_has_errors user, invite_code: "This is not a valid code"
+ @user = Account.create invite_code: "wrong_invite_code"
+ assert_has_errors @user, invite_code: "This is not a valid code"
end
end