enable some skipped tests again

This commit is contained in:
Benjamin Neff 2016-05-28 04:38:18 +02:00
parent 7db9d7733c
commit ecbf5fa3e7
3 changed files with 4 additions and 9 deletions

View file

@ -108,12 +108,10 @@ describe NotificationsController, :type => :controller do
end end
it "should provide a contacts menu for start sharing notifications" do it "should provide a contacts menu for start sharing notifications" do
skip # TODO
eve.share_with(alice.person, eve.aspects.first) eve.share_with(alice.person, eve.aspects.first)
get :index, "per_page" => 5 get :index, "per_page" => 5
expect(Nokogiri(response.body).css('.aspect_membership')).not_to be_empty expect(Nokogiri(response.body).css(".aspect_membership")).not_to be_empty
end end
it 'succeeds on mobile' do it 'succeeds on mobile' do

View file

@ -70,10 +70,9 @@ describe 'making sure the spec runner works' do
end end
end end
describe '#post' do describe "#post" do
it 'creates a notification with a mention' do it "creates a notification with a mention" do
skip("TODO: handle local receive") # TODO expect {
expect{
alice.post(:status_message, :text => "@{Bob Grimn; #{bob.person.diaspora_handle}} you are silly", :to => alice.aspects.find_by_name('generic')) alice.post(:status_message, :text => "@{Bob Grimn; #{bob.person.diaspora_handle}} you are silly", :to => alice.aspects.find_by_name('generic'))
}.to change(Notification, :count).by(1) }.to change(Notification, :count).by(1)
end end

View file

@ -123,8 +123,6 @@ describe PostService do
end end
it "marks a corresponding mention notification as read" do it "marks a corresponding mention notification as read" do
skip("TODO: create local notifications") # TODO
status_text = "this is a text mentioning @{Mention User ; #{alice.diaspora_handle}} ... have fun testing!" status_text = "this is a text mentioning @{Mention User ; #{alice.diaspora_handle}} ... have fun testing!"
mention_post = bob.post(:status_message, text: status_text, public: true) mention_post = bob.post(:status_message, text: status_text, public: true)