* Make custom emoji domains case sensitive #9351 * Fixup style in downcase_domain to comply with codeclimate. * switch if! to unless * Don't use transactions, operate in batches. Also revert spurious schema change.
		
			
				
	
	
		
			87 lines
		
	
	
	
		
			1.9 KiB
		
	
	
	
		
			Ruby
		
	
	
	
	
	
			
		
		
	
	
			87 lines
		
	
	
	
		
			1.9 KiB
		
	
	
	
		
			Ruby
		
	
	
	
	
	
| require 'rails_helper'
 | |
| 
 | |
| RSpec.describe CustomEmoji, type: :model do
 | |
|   describe '#search' do
 | |
|     let(:custom_emoji) { Fabricate(:custom_emoji, shortcode: shortcode) }
 | |
| 
 | |
|     subject { described_class.search(search_term) }
 | |
| 
 | |
|     context 'shortcode is exact' do
 | |
|       let(:shortcode) { 'blobpats' }
 | |
|       let(:search_term) { 'blobpats' }
 | |
| 
 | |
|       it 'finds emoji' do
 | |
|         is_expected.to include(custom_emoji)
 | |
|       end
 | |
|     end
 | |
| 
 | |
|     context 'shortcode is partial' do
 | |
|       let(:shortcode) { 'blobpats' }
 | |
|       let(:search_term) { 'blob' }
 | |
| 
 | |
|       it 'finds emoji' do
 | |
|         is_expected.to include(custom_emoji)
 | |
|       end
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   describe '#local?' do
 | |
|     let(:custom_emoji) { Fabricate(:custom_emoji, domain: domain) }
 | |
| 
 | |
|     subject { custom_emoji.local? }
 | |
| 
 | |
|     context 'domain is nil' do
 | |
|       let(:domain) { nil }
 | |
| 
 | |
|       it 'returns true' do
 | |
|         is_expected.to be true
 | |
|       end
 | |
|     end
 | |
| 
 | |
|     context 'domain is present' do
 | |
|       let(:domain) { 'example.com' }
 | |
| 
 | |
|       it 'returns false' do
 | |
|         is_expected.to be false
 | |
|       end
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   describe '#object_type' do
 | |
|     it 'returns :emoji' do
 | |
|       custom_emoji = Fabricate(:custom_emoji)
 | |
|       expect(custom_emoji.object_type).to be :emoji
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   describe '.from_text' do
 | |
|     let!(:emojo) { Fabricate(:custom_emoji) }
 | |
| 
 | |
|     subject { described_class.from_text(text, nil) }
 | |
| 
 | |
|     context 'with plain text' do
 | |
|       let(:text) { 'Hello :coolcat:' }
 | |
| 
 | |
|       it 'returns records used via shortcodes in text' do
 | |
|         is_expected.to include(emojo)
 | |
|       end
 | |
|     end
 | |
| 
 | |
|     context 'with html' do
 | |
|       let(:text) { '<p>Hello :coolcat:</p>' }
 | |
| 
 | |
|       it 'returns records used via shortcodes in text' do
 | |
|         is_expected.to include(emojo)
 | |
|       end
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   describe 'pre_validation' do
 | |
|     let(:custom_emoji) { Fabricate(:custom_emoji, domain: 'wWw.MaStOdOn.CoM') }
 | |
| 
 | |
|     it 'should downcase' do
 | |
|       custom_emoji.valid?
 | |
|       expect(custom_emoji.domain).to eq('www.mastodon.com')
 | |
|     end
 | |
|   end
 | |
| end
 |