From f16b31f0773c2fd1122ff0ad98cb392e762f0d0b Mon Sep 17 00:00:00 2001 From: Eugen Rochko Date: Fri, 26 Feb 2016 15:28:08 +0100 Subject: [PATCH] Adding a bunch of tests --- Gemfile | 1 + Gemfile.lock | 2 + app/helpers/application_helper.rb | 4 +- app/models/account.rb | 8 ++ app/models/status.rb | 7 +- app/models/stream_entry.rb | 2 +- app/services/setup_local_account_service.rb | 4 - spec/fabricators/account_fabricator.rb | 3 + spec/fabricators/favourite_fabricator.rb | 3 + spec/fabricators/follow_fabricator.rb | 3 + spec/fabricators/status_fabricator.rb | 3 + spec/helpers/application_helper_spec.rb | 37 ++++++ spec/helpers/atom_helper_spec.rb | 122 ++++++++++++++++++-- spec/helpers/home_helper_spec.rb | 12 +- spec/helpers/profile_helper_spec.rb | 12 +- spec/helpers/routing_helper.rb | 12 +- spec/helpers/xrd_helper_spec.rb | 12 +- spec/models/account_spec.rb | 95 ++++++++++++--- spec/models/favourite_spec.rb | 39 +++++-- spec/models/follow_spec.rb | 29 ++++- spec/models/status_spec.rb | 98 ++++++++++++++-- spec/models/stream_entry_spec.rb | 36 +++++- 22 files changed, 439 insertions(+), 105 deletions(-) create mode 100644 spec/fabricators/account_fabricator.rb create mode 100644 spec/fabricators/favourite_fabricator.rb create mode 100644 spec/fabricators/follow_fabricator.rb create mode 100644 spec/fabricators/status_fabricator.rb create mode 100644 spec/helpers/application_helper_spec.rb diff --git a/Gemfile b/Gemfile index a922e7a9dd..7beff4ea61 100644 --- a/Gemfile +++ b/Gemfile @@ -33,6 +33,7 @@ group :development, :test do gem 'pry-rails' gem 'nyan-cat-formatter' gem 'fuubar' + gem 'fabrication' end group :test do diff --git a/Gemfile.lock b/Gemfile.lock index 44d14929f8..14c0f29c0a 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -75,6 +75,7 @@ GEM equalizer (0.0.11) erubis (2.7.0) execjs (2.6.0) + fabrication (2.14.1) fuubar (2.0.0) rspec (~> 3.0) ruby-progressbar (~> 1.4) @@ -302,6 +303,7 @@ DEPENDENCIES binding_of_caller coffee-rails (~> 4.1.0) dotenv-rails + fabrication fuubar goldfinger grape diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb index 7a74a4d962..4c65a78bbf 100644 --- a/app/helpers/application_helper.rb +++ b/app/helpers/application_helper.rb @@ -6,8 +6,8 @@ module ApplicationHelper end def unique_tag_to_local_id(tag, expected_type) - Regexp.new("objectId=([\d]+):objectType=#{expected_type}").match(tag) - return match[1] unless match.nil? + matches = Regexp.new("objectId=([\\d]+):objectType=#{expected_type}").match(tag) + return matches[1] unless matches.nil? end def local_id?(id) diff --git a/app/models/account.rb b/app/models/account.rb index 2cbec488bd..1b4f4ebdf9 100644 --- a/app/models/account.rb +++ b/app/models/account.rb @@ -59,4 +59,12 @@ class Account < ActiveRecord::Base def subscription(webhook_url) @subscription ||= OStatus2::Subscription.new(self.remote_url, secret: self.secret, token: self.verify_token, webhook: webhook_url, hub: self.hub_url) end + + before_create do + if local? + keypair = OpenSSL::PKey::RSA.new(Rails.env.test? ? 48 : 2048) + self.private_key = keypair.to_pem + self.public_key = keypair.public_key.to_pem + end + end end diff --git a/app/models/status.rb b/app/models/status.rb index d958703936..3ba58cc914 100644 --- a/app/models/status.rb +++ b/app/models/status.rb @@ -54,10 +54,9 @@ class Status < ActiveRecord::Base unless reblog? self.text.scan(Account::MENTION_RE).each do |match| - uri = match.first - username = uri.split('@').first - domain = uri.split('@').size == 2 ? uri.split('@').last : nil - account = Account.find_by(username: username, domain: domain) + uri = match.first + username, domain = uri.split('@') + account = Account.find_by(username: username, domain: domain) m << account unless account.nil? end diff --git a/app/models/stream_entry.rb b/app/models/stream_entry.rb index a3ae099a12..e19dcc6c6d 100644 --- a/app/models/stream_entry.rb +++ b/app/models/stream_entry.rb @@ -29,7 +29,7 @@ class StreamEntry < ActiveRecord::Base end def threaded? - [:favorite, :comment].include? verb + verb == :favorite || object_type == :comment end def thread diff --git a/app/services/setup_local_account_service.rb b/app/services/setup_local_account_service.rb index a5ef689969..16de87e508 100644 --- a/app/services/setup_local_account_service.rb +++ b/app/services/setup_local_account_service.rb @@ -9,10 +9,6 @@ class SetupLocalAccountService < BaseService user.account.username = username user.account.domain = nil - keypair = OpenSSL::PKey::RSA.new(2048) - user.account.private_key = keypair.to_pem - user.account.public_key = keypair.public_key.to_pem - user.save! end end diff --git a/spec/fabricators/account_fabricator.rb b/spec/fabricators/account_fabricator.rb new file mode 100644 index 0000000000..5f87ea522a --- /dev/null +++ b/spec/fabricators/account_fabricator.rb @@ -0,0 +1,3 @@ +Fabricator(:account) do + +end diff --git a/spec/fabricators/favourite_fabricator.rb b/spec/fabricators/favourite_fabricator.rb new file mode 100644 index 0000000000..e598d38385 --- /dev/null +++ b/spec/fabricators/favourite_fabricator.rb @@ -0,0 +1,3 @@ +Fabricator(:favourite) do + +end diff --git a/spec/fabricators/follow_fabricator.rb b/spec/fabricators/follow_fabricator.rb new file mode 100644 index 0000000000..9d9d06f125 --- /dev/null +++ b/spec/fabricators/follow_fabricator.rb @@ -0,0 +1,3 @@ +Fabricator(:follow) do + +end diff --git a/spec/fabricators/status_fabricator.rb b/spec/fabricators/status_fabricator.rb new file mode 100644 index 0000000000..df222fc9d2 --- /dev/null +++ b/spec/fabricators/status_fabricator.rb @@ -0,0 +1,3 @@ +Fabricator(:status) do + text "Lorem ipsum dolor sit amet" +end diff --git a/spec/helpers/application_helper_spec.rb b/spec/helpers/application_helper_spec.rb new file mode 100644 index 0000000000..ad4284768d --- /dev/null +++ b/spec/helpers/application_helper_spec.rb @@ -0,0 +1,37 @@ +require 'rails_helper' + +RSpec.describe ApplicationHelper, type: :helper do + let(:local_domain) { 'local.tld' } + + before do + stub_const('LOCAL_DOMAIN', local_domain) + end + + describe '#unique_tag' do + it 'returns a string' do + expect(helper.unique_tag(Time.now, 12, 'Status')).to be_a String + end + end + + describe '#unique_tag_to_local_id' do + it 'returns the ID part' do + expect(helper.unique_tag_to_local_id("tag:#{local_domain};objectId=12:objectType=Status", 'Status')).to eql '12' + end + end + + describe '#local_id?' do + it 'returns true for a local ID' do + expect(helper.local_id?("tag:#{local_domain};objectId=12:objectType=Status")).to be true + end + + it 'returns false for a foreign ID' do + expect(helper.local_id?('tag:foreign.tld;objectId=12:objectType=Status')).to be false + end + end + + describe '#add_base_url_prefix' do + it 'returns full API URL from base to suffix' do + expect(helper.add_base_url_prefix('test')).to eql "#{root_url}api/test" + end + end +end diff --git a/spec/helpers/atom_helper_spec.rb b/spec/helpers/atom_helper_spec.rb index 57b12de673..009c20c965 100644 --- a/spec/helpers/atom_helper_spec.rb +++ b/spec/helpers/atom_helper_spec.rb @@ -1,15 +1,115 @@ require 'rails_helper' -# Specs in this file have access to a helper object that includes -# the AtomHelper. For example: -# -# describe AtomHelper do -# describe "string concat" do -# it "concats two strings with spaces" do -# expect(helper.concat_strings("this","that")).to eq("this that") -# end -# end -# end RSpec.describe AtomHelper, type: :helper do - pending "add some examples to (or delete) #{__FILE__}" + describe '#stream_updated_at' do + pending + end + + describe '#entry' do + pending + end + + describe '#feed' do + pending + end + + describe '#unique_id' do + pending + end + + describe '#simple_id' do + pending + end + + describe '#published_at' do + pending + end + + describe '#updated_at' do + pending + end + + describe '#verb' do + pending + end + + describe '#content' do + pending + end + + describe '#title' do + pending + end + + describe '#author' do + pending + end + + describe '#target' do + pending + end + + describe '#object_type' do + pending + end + + describe '#uri' do + pending + end + + describe '#name' do + pending + end + + describe '#summary' do + pending + end + + describe '#subtitle' do + pending + end + + describe '#link_alternate' do + pending + end + + describe '#link_self' do + pending + end + + describe '#link_hub' do + pending + end + + describe '#link_salmon' do + pending + end + + describe '#portable_contact' do + pending + end + + describe '#in_reply_to' do + pending + end + + describe '#link_mention' do + pending + end + + describe '#disambiguate_uri' do + pending + end + + describe '#disambiguate_url' do + pending + end + + describe '#include_author' do + pending + end + + describe '#include_entry' do + pending + end end diff --git a/spec/helpers/home_helper_spec.rb b/spec/helpers/home_helper_spec.rb index e537d8d9a8..8c96387e68 100644 --- a/spec/helpers/home_helper_spec.rb +++ b/spec/helpers/home_helper_spec.rb @@ -1,15 +1,5 @@ require 'rails_helper' -# Specs in this file have access to a helper object that includes -# the HomeHelper. For example: -# -# describe HomeHelper do -# describe "string concat" do -# it "concats two strings with spaces" do -# expect(helper.concat_strings("this","that")).to eq("this that") -# end -# end -# end RSpec.describe HomeHelper, type: :helper do - pending "add some examples to (or delete) #{__FILE__}" + end diff --git a/spec/helpers/profile_helper_spec.rb b/spec/helpers/profile_helper_spec.rb index 154c7dfb74..c73d67af27 100644 --- a/spec/helpers/profile_helper_spec.rb +++ b/spec/helpers/profile_helper_spec.rb @@ -1,15 +1,5 @@ require 'rails_helper' -# Specs in this file have access to a helper object that includes -# the ProfileHelper. For example: -# -# describe ProfileHelper do -# describe "string concat" do -# it "concats two strings with spaces" do -# expect(helper.concat_strings("this","that")).to eq("this that") -# end -# end -# end RSpec.describe ProfileHelper, type: :helper do - pending "add some examples to (or delete) #{__FILE__}" + end diff --git a/spec/helpers/routing_helper.rb b/spec/helpers/routing_helper.rb index 9ccb4a38ad..3cd397397b 100644 --- a/spec/helpers/routing_helper.rb +++ b/spec/helpers/routing_helper.rb @@ -1,15 +1,5 @@ require 'rails_helper' -# Specs in this file have access to a helper object that includes -# the RoutingHelper. For example: -# -# describe RoutingHelper do -# describe "string concat" do -# it "concats two strings with spaces" do -# expect(helper.concat_strings("this","that")).to eq("this that") -# end -# end -# end RSpec.describe RoutingHelper, type: :helper do - pending "add some examples to (or delete) #{__FILE__}" + end diff --git a/spec/helpers/xrd_helper_spec.rb b/spec/helpers/xrd_helper_spec.rb index 63ca2268a4..0bc71b6577 100644 --- a/spec/helpers/xrd_helper_spec.rb +++ b/spec/helpers/xrd_helper_spec.rb @@ -1,15 +1,5 @@ require 'rails_helper' -# Specs in this file have access to a helper object that includes -# the XrdHelper. For example: -# -# describe XrdHelper do -# describe "string concat" do -# it "concats two strings with spaces" do -# expect(helper.concat_strings("this","that")).to eq("this that") -# end -# end -# end RSpec.describe XrdHelper, type: :helper do - pending "add some examples to (or delete) #{__FILE__}" + end diff --git a/spec/models/account_spec.rb b/spec/models/account_spec.rb index fbcc972cf0..a5f24a9905 100644 --- a/spec/models/account_spec.rb +++ b/spec/models/account_spec.rb @@ -1,47 +1,110 @@ require 'rails_helper' RSpec.describe Account, type: :model do - describe '#follow!' do - pending - end + subject { Fabricate(:account, username: 'alice') } - describe '#unfollow!' do - pending - end + context do + let(:bob) { Fabricate(:account, username: 'bob') } - describe '#following?' do - pending + describe '#follow!' do + it 'creates a follow' do + follow = subject.follow!(bob) + + expect(follow).to be_instance_of Follow + expect(follow.account).to eq subject + expect(follow.target_account).to eq bob + end + end + + describe '#unfollow!' do + before do + subject.follow!(bob) + end + + it 'destroys a follow' do + unfollow = subject.unfollow!(bob) + + expect(unfollow).to be_instance_of Follow + expect(unfollow.account).to eq subject + expect(unfollow.target_account).to eq bob + expect(unfollow.destroyed?).to be true + end + end + + describe '#following?' do + it 'returns true when the target is followed' do + subject.follow!(bob) + expect(subject.following?(bob)).to be true + end + + it 'returns false if the target is not followed' do + expect(subject.following?(bob)).to be false + end + end end describe '#local?' do - pending + it 'returns true when the account is local' do + expect(subject.local?).to be true + end + + it 'returns false when the account is on a different domain' do + subject.domain = 'foreign.tld' + expect(subject.local?).to be false + end end describe '#acct' do - pending + it 'returns username for local users' do + expect(subject.acct).to eql 'alice' + end + + it 'returns username@domain for foreign users' do + subject.domain = 'foreign.tld' + expect(subject.acct).to eql 'alice@foreign.tld' + end end describe '#subscribed?' do - pending + it 'returns false when no secrets and tokens have been set' do + expect(subject.subscribed?).to be false + end + + it 'returns true when the secret and token have been set' do + subject.secret = 'a' + subject.verify_token = 'b' + + expect(subject.subscribed?).to be true + end end describe '#keypair' do - pending + it 'returns an RSA key pair' do + expect(subject.keypair).to be_instance_of OpenSSL::PKey::RSA + end end describe '#subscription' do - pending + it 'returns an OStatus subscription' do + expect(subject.subscription('')).to be_instance_of OStatus2::Subscription + end end describe '#object_type' do - pending + it 'is always a person' do + expect(subject.object_type).to be :person + end end describe '#title' do - pending + it 'is the same as the username' do + expect(subject.title).to eql subject.username + end end describe '#content' do - pending + it 'is the same as the note' do + expect(subject.content).to eql subject.note + end end end diff --git a/spec/models/favourite_spec.rb b/spec/models/favourite_spec.rb index ad803ec4b4..c778b13d49 100644 --- a/spec/models/favourite_spec.rb +++ b/spec/models/favourite_spec.rb @@ -1,31 +1,56 @@ require 'rails_helper' RSpec.describe Favourite, type: :model do + let(:alice) { Fabricate(:account, username: 'alice') } + let(:bob) { Fabricate(:account, username: 'bob') } + let(:status) { Fabricate(:status, account: bob) } + + subject { Favourite.new(account: alice, status: status) } + describe '#verb' do - pending + it 'is always favorite' do + expect(subject.verb).to be :favorite + end end describe '#title' do - pending + it 'describes the favourite' do + expect(subject.title).to eql 'alice favourited a status by bob' + end end describe '#content' do - pending + it 'equals the title' do + expect(subject.content).to eq subject.title + end end describe '#object_type' do - pending + it 'is a note when the target is a note' do + expect(subject.object_type).to be :note + end + + it 'is a comment when the target is a comment' do + status.in_reply_to_id = 2 + expect(subject.object_type).to be :comment + end end describe '#target' do - pending + it 'is the status that was favourited' do + expect(subject.target).to eq status + end end describe '#mentions' do - pending + it 'is always empty' do + expect(subject.mentions).to be_empty + end end describe '#thread' do - pending + it 'equals the target' do + expect(subject.thread).to eq subject.target + end end end diff --git a/spec/models/follow_spec.rb b/spec/models/follow_spec.rb index edb0847018..28bd41dfdc 100644 --- a/spec/models/follow_spec.rb +++ b/spec/models/follow_spec.rb @@ -1,27 +1,44 @@ require 'rails_helper' RSpec.describe Follow, type: :model do + let(:alice) { Fabricate(:account, username: 'alice') } + let(:bob) { Fabricate(:account, username: 'bob') } + + subject { Follow.new(account: alice, target_account: bob) } + describe '#verb' do - pending + it 'is follow' do + expect(subject.verb).to be :follow + end end describe '#title' do - pending + it 'describes the follow' do + expect(subject.title).to eql 'alice started following bob' + end end describe '#content' do - pending + it 'is the same as the title' do + expect(subject.content).to eql subject.title + end end describe '#object_type' do - pending + it 'is a person' do + expect(subject.object_type).to be :person + end end describe '#target' do - pending + it 'is the person being followed' do + expect(subject.target).to eq bob + end end describe '#mentions' do - pending + it 'is empty' do + expect(subject.mentions).to be_empty + end end end diff --git a/spec/models/status_spec.rb b/spec/models/status_spec.rb index 070e7b87f4..db7b671172 100644 --- a/spec/models/status_spec.rb +++ b/spec/models/status_spec.rb @@ -1,35 +1,117 @@ require 'rails_helper' RSpec.describe Status, type: :model do + let(:alice) { Fabricate(:account, username: 'alice') } + let(:bob) { Fabricate(:account, username: 'bob') } + let(:other) { Fabricate(:status, account: bob, text: 'Skulls for the skull god! The enemy\'s gates are sideways!')} + + subject { Fabricate(:status, account: alice) } + describe '#local?' do - pending + it 'returns true when no remote URI is set' do + expect(subject.local?).to be true + end + + it 'returns false if a remote URI is set' do + subject.uri = 'a' + expect(subject.local?).to be false + end end describe '#reblog?' do - pending + it 'returns true when the status reblogs another status' do + subject.reblog = other + expect(subject.reblog?).to be true + end + + it 'returns false if the status is self-contained' do + expect(subject.reblog?).to be false + end end describe '#reply?' do - pending + it 'returns true if the status references another' do + subject.thread = other + expect(subject.reply?).to be true + end + + it 'returns false if the status is self-contained' do + expect(subject.reply?).to be false + end end describe '#mentions' do - pending + before do + bob # make sure the account exists + end + + it 'is empty if the status is self-contained and does not mention anyone' do + expect(subject.mentions).to be_empty + end + + it 'returns mentioned accounts' do + subject.text = 'Hello @bob!' + expect(subject.mentions).to include bob + end + + it 'returns account of the replied-to status' do + subject.thread = other + expect(subject.mentions).to include bob + end + + it 'returns the account of the shared status' do + subject.reblog = other + expect(subject.mentions).to include bob + end end describe '#verb' do - pending + it 'is always post' do + expect(subject.verb).to be :post + end end describe '#object_type' do - pending + it 'is note when the status is self-contained' do + expect(subject.object_type).to be :note + end + + it 'is comment when the status replies to another' do + subject.thread = other + expect(subject.object_type).to be :comment + end end describe '#title' do - pending + it 'is a shorter version of the content' do + expect(subject.title).to be_a String + end + end + + describe '#content' do + it 'returns the text of the status if it is not a reblog' do + expect(subject.content).to eql subject.text + end + + it 'returns the text of the reblogged status' do + subject.reblog = other + expect(subject.content).to eql other.text + end end describe '#target' do - pending + it 'returns nil if the status is self-contained' do + expect(subject.target).to be_nil + end + + it 'returns nil if the status is a reply' do + subject.thread = other + expect(subject.target).to be_nil + end + + it 'returns the reblogged status' do + subject.reblog = other + expect(subject.target).to eq other + end end end diff --git a/spec/models/stream_entry_spec.rb b/spec/models/stream_entry_spec.rb index 6386cac0f6..9ecf6412a5 100644 --- a/spec/models/stream_entry_spec.rb +++ b/spec/models/stream_entry_spec.rb @@ -1,11 +1,43 @@ require 'rails_helper' RSpec.describe StreamEntry, type: :model do + let(:alice) { Fabricate(:account, username: 'alice') } + let(:bob) { Fabricate(:account, username: 'bob') } + let(:follow) { Fabricate(:follow, account: alice, target_account: bob) } + let(:status) { Fabricate(:status, account: alice) } + let(:reblog) { Fabricate(:status, account: bob, reblog: status) } + let(:reply) { Fabricate(:status, account: bob, thread: status) } + let(:favourite) { Fabricate(:favourite, account: alice, status: status) } + describe '#targeted?' do - pending + it 'returns true for a follow' do + expect(follow.stream_entry.targeted?).to be true + end + + it 'returns true for a favourite' do + expect(favourite.stream_entry.targeted?).to be true + end + + it 'returns true for a reblog' do + expect(reblog.stream_entry.targeted?).to be true + end + + it 'returns false otherwise' do + expect(status.stream_entry.targeted?).to be false + end end describe '#threaded?' do - pending + it 'returns true for a favourite' do + expect(favourite.stream_entry.threaded?).to be true + end + + it 'returns true for a reply' do + expect(reply.stream_entry.threaded?).to be true + end + + it 'returns false otherwise' do + expect(status.stream_entry.threaded?).to be false + end end end