mirror of
				https://github.com/zulip/zulip.git
				synced 2025-11-04 05:53:43 +00:00 
			
		
		
		
	Recognize humbug-user-uploads-test for images/attachements.
(imported from commit 8596a5c4bad6e1adbfe97cbe057d1bd6d8e7f530)
This commit is contained in:
		
				
					committed by
					
						
						Leo Franchi
					
				
			
			
				
	
			
			
			
						parent
						
							d2f773a99d
						
					
				
				
					commit
					72964a2194
				
			@@ -945,11 +945,11 @@ class Message(models.Model):
 | 
				
			|||||||
 | 
					
 | 
				
			||||||
    @staticmethod
 | 
					    @staticmethod
 | 
				
			||||||
    def content_has_attachment(content):
 | 
					    def content_has_attachment(content):
 | 
				
			||||||
        return re.search('[/\-]user[\-_]uploads[/\.]', content)
 | 
					        return re.search('[/\-]user[\-_]uploads[/\.-]', content)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    @staticmethod
 | 
					    @staticmethod
 | 
				
			||||||
    def content_has_image(content):
 | 
					    def content_has_image(content):
 | 
				
			||||||
        return bool(re.search('[/\-]user[\-_]uploads[/\.]\S+\.(bmp|gif|jpg|jpeg|png|webp)', content, re.IGNORECASE))
 | 
					        return bool(re.search('[/\-]user[\-_]uploads[/\.-]\S+\.(bmp|gif|jpg|jpeg|png|webp)', content, re.IGNORECASE))
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    @staticmethod
 | 
					    @staticmethod
 | 
				
			||||||
    def content_has_link(content):
 | 
					    def content_has_link(content):
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -1356,6 +1356,7 @@ class AttachmentTest(TestCase):
 | 
				
			|||||||
        self.assertTrue(Message.content_has_attachment('yo\n https://staging.zulip.com/user_uploads/'))
 | 
					        self.assertTrue(Message.content_has_attachment('yo\n https://staging.zulip.com/user_uploads/'))
 | 
				
			||||||
        self.assertTrue(Message.content_has_attachment('yo\n /user_uploads/1/wEAnI-PEmVmCjo15xxNaQbnj/photo-10.jpg foo'))
 | 
					        self.assertTrue(Message.content_has_attachment('yo\n /user_uploads/1/wEAnI-PEmVmCjo15xxNaQbnj/photo-10.jpg foo'))
 | 
				
			||||||
        self.assertTrue(Message.content_has_attachment('https://humbug-user-uploads.s3.amazonaws.com/sX_TIQx/screen-shot.jpg'))
 | 
					        self.assertTrue(Message.content_has_attachment('https://humbug-user-uploads.s3.amazonaws.com/sX_TIQx/screen-shot.jpg'))
 | 
				
			||||||
 | 
					        self.assertTrue(Message.content_has_attachment('https://humbug-user-uploads-test.s3.amazonaws.com/sX_TIQx/screen-shot.jpg'))
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        self.assertFalse(Message.content_has_image('whatever'))
 | 
					        self.assertFalse(Message.content_has_image('whatever'))
 | 
				
			||||||
        self.assertFalse(Message.content_has_image('yo http://foo.com'))
 | 
					        self.assertFalse(Message.content_has_image('yo http://foo.com'))
 | 
				
			||||||
@@ -1364,12 +1365,14 @@ class AttachmentTest(TestCase):
 | 
				
			|||||||
            content = 'yo\n /user_uploads/1/wEAnI-PEmVmCjo15xxNaQbnj/photo-10.%s foo' % (ext,)
 | 
					            content = 'yo\n /user_uploads/1/wEAnI-PEmVmCjo15xxNaQbnj/photo-10.%s foo' % (ext,)
 | 
				
			||||||
            self.assertTrue(Message.content_has_image(content))
 | 
					            self.assertTrue(Message.content_has_image(content))
 | 
				
			||||||
        self.assertTrue(Message.content_has_image('https://humbug-user-uploads.s3.amazonaws.com/sX_TIQx/screen-shot.jpg'))
 | 
					        self.assertTrue(Message.content_has_image('https://humbug-user-uploads.s3.amazonaws.com/sX_TIQx/screen-shot.jpg'))
 | 
				
			||||||
 | 
					        self.assertTrue(Message.content_has_image('https://humbug-user-uploads-test.s3.amazonaws.com/sX_TIQx/screen-shot.jpg'))
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        self.assertFalse(Message.content_has_link('whatever'))
 | 
					        self.assertFalse(Message.content_has_link('whatever'))
 | 
				
			||||||
        self.assertTrue(Message.content_has_link('yo\n http://foo.com'))
 | 
					        self.assertTrue(Message.content_has_link('yo\n http://foo.com'))
 | 
				
			||||||
        self.assertTrue(Message.content_has_link('yo\n https://example.com?spam=1&eggs=2'))
 | 
					        self.assertTrue(Message.content_has_link('yo\n https://example.com?spam=1&eggs=2'))
 | 
				
			||||||
        self.assertTrue(Message.content_has_link('yo /user_uploads/1/wEAnI-PEmVmCjo15xxNaQbnj/photo-10.pdf foo'))
 | 
					        self.assertTrue(Message.content_has_link('yo /user_uploads/1/wEAnI-PEmVmCjo15xxNaQbnj/photo-10.pdf foo'))
 | 
				
			||||||
        self.assertTrue(Message.content_has_link('https://humbug-user-uploads.s3.amazonaws.com/sX_TIQx/screen-shot.jpg'))
 | 
					        self.assertTrue(Message.content_has_link('https://humbug-user-uploads.s3.amazonaws.com/sX_TIQx/screen-shot.jpg'))
 | 
				
			||||||
 | 
					        self.assertTrue(Message.content_has_link('https://humbug-user-uploads-test.s3.amazonaws.com/sX_TIQx/screen-shot.jpg'))
 | 
				
			||||||
 | 
					
 | 
				
			||||||
class CheckMessageTest(AuthedTestCase):
 | 
					class CheckMessageTest(AuthedTestCase):
 | 
				
			||||||
    def test_basic_check_message_call(self):
 | 
					    def test_basic_check_message_call(self):
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user