X-Git-Url: https://git.mdrn.pl/wolnelektury.git/blobdiff_plain/c9a668f42a4a821a920baf1df421045cc9a6e82d..4f985994c54a53688a15c21ba599d7bcdc1e4974:/apps/catalogue/tests.py diff --git a/apps/catalogue/tests.py b/apps/catalogue/tests.py index fae4ba2e9..c2eb33675 100644 --- a/apps/catalogue/tests.py +++ b/apps/catalogue/tests.py @@ -1,53 +1,256 @@ # -*- coding: utf-8 -*- from django.test import TestCase from catalogue import models, views +from django.core.files.base import ContentFile from django.contrib.auth.models import User, AnonymousUser +from django.test.client import Client + +from nose.tools import raises +from StringIO import StringIO + +class BasicSearchLogicTests(TestCase): -class SimpleSearchTest(TestCase): def setUp(self): - t = models.Tag(name=u'Tadeusz Żeleński (Boy)', category=u'author', slug=u'a1') - t.save() - self.t = t - self.user = AnonymousUser() - - def tearDown(self): - self.t.delete() - - def test_too_short(self): - self.assertRaises(ValueError, views.find_best_matches, u'', self.user) - self.assertRaises(ValueError, views.find_best_matches, u't', self.user) - - def test_match_beginning(self): - self.assertEqual(views.find_best_matches(u'Tad', self.user), (self.t,)) - - def test_match_case(self): - self.assertEqual(views.find_best_matches(u'TAD', self.user), (self.t,)) + self.author_tag = models.Tag.objects.create( + name=u'Adam Mickiewicz [SubWord]', + category=u'author', slug="one") + + self.unicode_tag = models.Tag.objects.create( + name=u'Tadeusz Żeleński (Boy)', + category=u'author', slug="two") + + self.polish_tag = models.Tag.objects.create( + name=u'ĘÓĄŚŁŻŹĆŃęóąśłżźćń', + category=u'author', slug="three") + + @raises(ValueError) + def test_empty_query(self): + """ Check that empty queries raise an error. """ + views.find_best_matches(u'') + + @raises(ValueError) + def test_one_letter_query(self): + """ Check that one letter queries aren't permitted. """ + views.find_best_matches(u't') + + def test_match_by_prefix(self): + """ Tags should be matched by prefix of words within it's name. """ + self.assertEqual(views.find_best_matches(u'Ada'), (self.author_tag,)) + self.assertEqual(views.find_best_matches(u'Mic'), (self.author_tag,)) + self.assertEqual(views.find_best_matches(u'Mickiewicz'), (self.author_tag,)) + + def test_match_case_insensitive(self): + """ Tag names should match case insensitive. """ + self.assertEqual(views.find_best_matches(u'adam mickiewicz'), (self.author_tag,)) + + def test_match_case_insensitive_unicode(self): + """ Tag names should match case insensitive (unicode). """ + self.assertEqual(views.find_best_matches(u'tadeusz żeleński (boy)'), (self.unicode_tag,)) def test_word_boundary(self): - self.assertEqual(views.find_best_matches(u'Boy', self.user), (self.t,)) - self.assertEqual(views.find_best_matches(u'(Boy', self.user), (self.t,)) + self.assertEqual(views.find_best_matches(u'SubWord'), (self.author_tag,)) + self.assertEqual(views.find_best_matches(u'[SubWord'), (self.author_tag,)) + + def test_unrelated_search(self): + self.assertEqual(views.find_best_matches(u'alamakota'), tuple()) + self.assertEqual(views.find_best_matches(u'Adama'), ()) + + def test_infix_doesnt_match(self): + """ Searching for middle of a word shouldn't match. """ + self.assertEqual(views.find_best_matches(u'deusz'), tuple()) + + def test_diactricts_removal_pl(self): + """ Tags should match both with and without national characters. """ + self.assertEqual(views.find_best_matches(u'ĘÓĄŚŁŻŹĆŃęóąśłżźćń'), (self.polish_tag,)) + self.assertEqual(views.find_best_matches(u'EOASLZZCNeoaslzzcn'), (self.polish_tag,)) + self.assertEqual(views.find_best_matches(u'eoaslzzcneoaslzzcn'), (self.polish_tag,)) + + def test_diactricts_query_removal_pl(self): + """ Tags without national characters shouldn't be matched by queries with them. """ + self.assertEqual(views.find_best_matches(u'Adąm'), ()) - def test_not_found(self): - self.assertEqual(views.find_best_matches(u'andrzej', self.user), ()) - self.assertEqual(views.find_best_matches(u'deusz', self.user), ()) - - def test_locale(self): - self.assertEqual(views.find_best_matches(u'ele', self.user), ()) - self.assertEqual(views.find_best_matches(u'Żele', self.user), (self.t,)) - self.assertEqual(views.find_best_matches(u'żele', self.user), (self.t,)) - def test_sloppy(self): - self.assertEqual(views.find_best_matches(u'Żelenski', self.user), (self.t,)) - self.assertEqual(views.find_best_matches(u'zelenski', self.user), (self.t,)) - + self.assertEqual(views.find_best_matches(u'Żelenski'), (self.unicode_tag,)) + self.assertEqual(views.find_best_matches(u'zelenski'), (self.unicode_tag,)) + + +class PersonStub(object): + + def __init__(self, first_names, last_name): + self.first_names = first_names + self.last_name = last_name + +class BookInfoStub(object): + def __init__(self, **kwargs): + self.__dict = kwargs + + def __setattr__(self, key, value): + if not key.startswith('_'): + self.__dict[key] = value + return object.__setattr__(self, key, value) + + def __getattr__(self, key): + return self.__dict[key] + + def to_dict(self): + return dict((key, unicode(value)) for key, value in self.__dict.items()) + +class BookImportLogicTests(TestCase): -class SetSearchTest(TestCase): def setUp(self): - self.me = User(name='me') - self.other = User(name='other') + self.book_info = BookInfoStub( + url=u"http://wolnelektury.pl/example/default_book", + about=u"http://wolnelektury.pl/example/URI/default_book", + title=u"Default Book", + author=PersonStub(("Jim",), "Lazy"), + kind="X-Kind", + genre="X-Genre", + epoch="X-Epoch", + ) + + self.expected_tags = [ + ('author', 'jim-lazy'), + ('book', 'l-default_book'), + ('genre', 'x-genre'), + ('epoch', 'x-epoch'), + ('kind', 'x-kind'), + ] + self.expected_tags.sort() + + def test_empty_book(self): + BOOK_TEXT = "" + book = models.Book.from_text_and_meta(ContentFile(BOOK_TEXT), self.book_info) + + self.assertEqual(book.title, "Default Book") + self.assertEqual(book.slug, "default_book") + self.assert_(book.parent is None) + self.assertTrue(book.has_html_file()) + + # no fragments generated + self.assertEqual(book.fragments.count(), 0) + + # TODO: this should be filled out probably... + self.assertEqual(book.wiki_link, '') + self.assertEqual(book.gazeta_link, '') + self.assertEqual(book._short_html, '') + self.assertEqual(book.description, '') + + tags = [ (tag.category, tag.slug) for tag in book.tags ] + tags.sort() + + self.assertEqual(tags, self.expected_tags) + + def test_book_with_fragment(self): + BOOK_TEXT = """ + + LoveAla ma kota + + """ + + book = models.Book.from_text_and_meta(ContentFile(BOOK_TEXT), self.book_info) + + self.assertEqual(book.fragments.count(), 1) + self.assertEqual(book.fragments.all()[0].text, u'

Ala ma kota

\n') + + self.assert_(('theme', 'love') in [ (tag.category, tag.slug) for tag in book.tags ]) + + def test_book_replace_title(self): + BOOK_TEXT = """""" + self.book_info.title = u"Extraordinary" + book = models.Book.from_text_and_meta(ContentFile(BOOK_TEXT), self.book_info) + + tags = [ (tag.category, tag.slug) for tag in book.tags ] + tags.sort() + + self.assertEqual(tags, self.expected_tags) + + def test_book_replace_author(self): + BOOK_TEXT = """""" + self.book_info.author = PersonStub(("Hans", "Christian"), "Andersen") + book = models.Book.from_text_and_meta(ContentFile(BOOK_TEXT), self.book_info) + + tags = [ (tag.category, tag.slug) for tag in book.tags ] + tags.sort() + + self.expected_tags.remove(('author', 'jim-lazy')) + self.expected_tags.append(('author', 'hans-christian-andersen')) + self.expected_tags.sort() + + self.assertEqual(tags, self.expected_tags) + + # the old tag should disappear + self.assertRaises(models.Tag.DoesNotExist, models.Tag.objects.get, + slug="jim-lazy", category="author") + + - def tearDown(self): - self.me.delete() - self.other.delete() +class BooksByTagFlat(TestCase): + def setUp(self): + self.tag_empty = models.Tag(name='Empty tag', slug='empty', category='author') + self.tag_common = models.Tag(name='Common author', slug='common', category='author') + + self.tag_kind1 = models.Tag(name='Type 1', slug='type1', category='kind') + self.tag_kind2 = models.Tag(name='Type 2', slug='type2', category='kind') + self.tag_kind3 = models.Tag(name='Type 3', slug='type3', category='kind') + for tag in self.tag_empty, self.tag_common, self.tag_kind1, self.tag_kind2, self.tag_kind3: + tag.save() + + + self.parent = models.Book(title='Parent', slug='parent') + self.parent.save() + + self.similar_child = models.Book(title='Similar child', + slug='similar_child', + parent=self.parent) + self.similar_child.save() + self.similar_grandchild = models.Book(title='Similar grandchild', + slug='similar_grandchild', + parent=self.similar_child) + self.similar_grandchild.save() + for book in self.parent, self.similar_child, self.similar_grandchild: + book.tags = [self.tag_common, self.tag_kind1] + book.save() + + self.different_child = models.Book(title='Different child', + slug='different_child', + parent=self.parent) + self.different_child.save() + self.different_child.tags = [self.tag_common, self.tag_kind2] + self.different_child.save() + self.different_grandchild = models.Book(title='Different grandchild', + slug='different_grandchild', + parent=self.different_child) + self.different_grandchild.save() + self.different_grandchild.tags = [self.tag_common, self.tag_kind3] + self.different_grandchild.save() + + for book in models.Book.objects.all(): + l_tag = models.Tag(name=book.title, slug='l-'+book.slug, category='book') + l_tag.save() + book.tags = list(book.tags) + [l_tag] + + + self.client = Client() + + def test_nonexistent_tag(self): + """ Looking for a non-existent tag should yield 404 """ + self.assertEqual(404, self.client.get('/katalog/czeslaw_milosz/').status_code) + + def test_book_tag(self): + """ Looking for a book tag isn't permitted """ + self.assertEqual(404, self.client.get('/katalog/parent/').status_code) + def test_tag_empty(self): + """ Tag with no books should return no books and no related tags """ + context = self.client.get('/katalog/empty/').context + self.assertEqual(0, len(context['object_list'])) + self.assertEqual(0, len(context['categories'])) + + def test_tag_common(self): + """ Filtering by tag should only yield top-level books """ + context = self.client.get('/katalog/%s/' % self.tag_common.slug).context + self.assertEqual(list(context['object_list']), + [self.parent]) + +