mirror of
https://github.com/bookwyrm-social/bookwyrm.git
synced 2024-11-24 10:31:05 +00:00
Adds storygraph tests
This commit is contained in:
parent
aeef472ee1
commit
1b9d08414f
4 changed files with 131 additions and 1 deletions
|
@ -21,7 +21,7 @@ class StorygraphImporter(Importer):
|
||||||
data["ISBN13"] = entry["ISBN"]
|
data["ISBN13"] = entry["ISBN"]
|
||||||
data["My Review"] = entry["Review"]
|
data["My Review"] = entry["Review"]
|
||||||
if entry["Star Rating"]:
|
if entry["Star Rating"]:
|
||||||
data["My Rating"] = math.ceil(float(entry["Star Rating"]))
|
data["My Rating"] = float(entry["Star Rating"])
|
||||||
else:
|
else:
|
||||||
data["My Rating"] = ""
|
data["My Rating"] = ""
|
||||||
|
|
||||||
|
|
5
bookwyrm/tests/data/generic.csv
Normal file
5
bookwyrm/tests/data/generic.csv
Normal file
|
@ -0,0 +1,5 @@
|
||||||
|
id,title,author,ISBN,rating,shelf,review,added
|
||||||
|
38,Gideon the Ninth (The Locked Tomb #1),Tamsyn Muir,"9781250313195",,read,,2021-11-10
|
||||||
|
48,Harrow the Ninth (The Locked Tomb #2),Tamsyn Muir,,3,read,,2021-11-10
|
||||||
|
23,Subcutanean,Aaron A. Reed,,,read,,2021-11-10
|
||||||
|
10,Patisserie at Home,Mélanie Dupuis,"9780062445315",2,read,"mixed feelings",2021-11-10
|
|
3
bookwyrm/tests/data/storygraph.csv
Normal file
3
bookwyrm/tests/data/storygraph.csv
Normal file
|
@ -0,0 +1,3 @@
|
||||||
|
Title,Authors,Contributors,ISBN,Format,Read Status,Date Added,Last Date Read,Dates Read,Read Count,Moods,Pace,Character- or Plot-Driven?,Strong Character Development?,Loveable Characters?,Diverse Characters?,Flawed Characters?,Star Rating,Review,Content Warnings,Content Warning Description,Tags,Owned?
|
||||||
|
Always Coming Home,"Ursula K. Le Guin, Todd Barton, Margaret Chodos-Irvine","",,,to-read,2021/05/10,"","",0,"",,,,,,,,,"",,"",No
|
||||||
|
Subprime Attention Crisis,Tim Hwang,"",,,read,2021/05/10,"","",1,informative,fast,,,,,,5.0,"","","","",No
|
|
122
bookwyrm/tests/importers/test_storygraph_import.py
Normal file
122
bookwyrm/tests/importers/test_storygraph_import.py
Normal file
|
@ -0,0 +1,122 @@
|
||||||
|
""" testing import """
|
||||||
|
import csv
|
||||||
|
import pathlib
|
||||||
|
from unittest.mock import patch
|
||||||
|
import datetime
|
||||||
|
import pytz
|
||||||
|
|
||||||
|
from django.test import TestCase
|
||||||
|
|
||||||
|
from bookwyrm import models
|
||||||
|
from bookwyrm.importers import StorygraphImporter
|
||||||
|
from bookwyrm.importers.importer import handle_imported_book
|
||||||
|
|
||||||
|
|
||||||
|
def make_date(*args):
|
||||||
|
"""helper function to easily generate a date obj"""
|
||||||
|
return datetime.datetime(*args, tzinfo=pytz.UTC)
|
||||||
|
|
||||||
|
|
||||||
|
# pylint: disable=consider-using-with
|
||||||
|
@patch("bookwyrm.suggested_users.rerank_suggestions_task.delay")
|
||||||
|
@patch("bookwyrm.activitystreams.populate_stream_task.delay")
|
||||||
|
@patch("bookwyrm.activitystreams.add_book_statuses_task.delay")
|
||||||
|
class StorygraphImport(TestCase):
|
||||||
|
"""importing from storygraph csv"""
|
||||||
|
|
||||||
|
def setUp(self):
|
||||||
|
"""use a test csv"""
|
||||||
|
self.importer = StorygraphImporter()
|
||||||
|
datafile = pathlib.Path(__file__).parent.joinpath("../data/storygraph.csv")
|
||||||
|
self.csv = open(datafile, "r", encoding=self.importer.encoding)
|
||||||
|
with patch("bookwyrm.suggested_users.rerank_suggestions_task.delay"), patch(
|
||||||
|
"bookwyrm.activitystreams.populate_stream_task.delay"
|
||||||
|
):
|
||||||
|
self.user = models.User.objects.create_user(
|
||||||
|
"mouse", "mouse@mouse.mouse", "password", local=True
|
||||||
|
)
|
||||||
|
|
||||||
|
work = models.Work.objects.create(title="Test Work")
|
||||||
|
self.book = models.Edition.objects.create(
|
||||||
|
title="Example Edition",
|
||||||
|
remote_id="https://example.com/book/1",
|
||||||
|
parent_work=work,
|
||||||
|
)
|
||||||
|
|
||||||
|
def test_create_job(self, *_):
|
||||||
|
"""creates the import job entry and checks csv"""
|
||||||
|
import_job = self.importer.create_job(self.user, self.csv, False, "public")
|
||||||
|
|
||||||
|
import_items = models.ImportItem.objects.filter(job=import_job).all()
|
||||||
|
self.assertEqual(len(import_items), 2)
|
||||||
|
self.assertEqual(import_items[0].index, 0)
|
||||||
|
self.assertEqual(import_items[0].data["Title"], "Always Coming Home")
|
||||||
|
self.assertEqual(import_items[1].index, 1)
|
||||||
|
self.assertEqual(import_items[1].data["Title"], "Subprime Attention Crisis")
|
||||||
|
self.assertEqual(import_items[1].data["My Rating"], 5.0)
|
||||||
|
|
||||||
|
def test_create_retry_job(self, *_):
|
||||||
|
"""trying again with items that didn't import"""
|
||||||
|
import_job = self.importer.create_job(self.user, self.csv, False, "unlisted")
|
||||||
|
import_items = models.ImportItem.objects.filter(job=import_job).all()[:2]
|
||||||
|
|
||||||
|
retry = self.importer.create_retry_job(self.user, import_job, import_items)
|
||||||
|
self.assertNotEqual(import_job, retry)
|
||||||
|
self.assertEqual(retry.user, self.user)
|
||||||
|
self.assertEqual(retry.include_reviews, False)
|
||||||
|
self.assertEqual(retry.privacy, "unlisted")
|
||||||
|
|
||||||
|
retry_items = models.ImportItem.objects.filter(job=retry).all()
|
||||||
|
self.assertEqual(len(retry_items), 2)
|
||||||
|
self.assertEqual(retry_items[0].index, 0)
|
||||||
|
self.assertEqual(retry_items[0].data["Title"], "Always Coming Home")
|
||||||
|
self.assertEqual(retry_items[1].index, 1)
|
||||||
|
self.assertEqual(retry_items[1].data["Title"], "Subprime Attention Crisis")
|
||||||
|
|
||||||
|
def test_handle_imported_book(self, *_):
|
||||||
|
"""storygraph import added a book, this adds related connections"""
|
||||||
|
shelf = self.user.shelf_set.filter(identifier="to-read").first()
|
||||||
|
self.assertIsNone(shelf.books.first())
|
||||||
|
|
||||||
|
import_job = models.ImportJob.objects.create(user=self.user)
|
||||||
|
datafile = pathlib.Path(__file__).parent.joinpath("../data/storygraph.csv")
|
||||||
|
csv_file = open(datafile, "r") # pylint: disable=unspecified-encoding
|
||||||
|
for index, entry in enumerate(list(csv.DictReader(csv_file))):
|
||||||
|
entry = self.importer.parse_fields(entry)
|
||||||
|
import_item = models.ImportItem.objects.create(
|
||||||
|
job_id=import_job.id, index=index, data=entry, book=self.book
|
||||||
|
)
|
||||||
|
break
|
||||||
|
|
||||||
|
with patch("bookwyrm.models.activitypub_mixin.broadcast_task.delay"):
|
||||||
|
handle_imported_book(
|
||||||
|
self.importer.service, self.user, import_item, False, "public"
|
||||||
|
)
|
||||||
|
|
||||||
|
shelf.refresh_from_db()
|
||||||
|
self.assertEqual(shelf.books.first(), self.book)
|
||||||
|
self.assertEqual(
|
||||||
|
shelf.shelfbook_set.first().shelved_date, make_date(2021, 5, 10)
|
||||||
|
)
|
||||||
|
|
||||||
|
@patch("bookwyrm.activitystreams.add_status_task.delay")
|
||||||
|
def test_handle_imported_book_rating(self, *_):
|
||||||
|
"""storygraph rating import"""
|
||||||
|
import_job = models.ImportJob.objects.create(user=self.user)
|
||||||
|
datafile = pathlib.Path(__file__).parent.joinpath("../data/storygraph.csv")
|
||||||
|
csv_file = open(datafile, "r") # pylint: disable=unspecified-encoding
|
||||||
|
entry = list(csv.DictReader(csv_file))[1]
|
||||||
|
entry = self.importer.parse_fields(entry)
|
||||||
|
import_item = models.ImportItem.objects.create(
|
||||||
|
job_id=import_job.id, index=0, data=entry, book=self.book
|
||||||
|
)
|
||||||
|
|
||||||
|
with patch("bookwyrm.models.activitypub_mixin.broadcast_task.delay"):
|
||||||
|
handle_imported_book(
|
||||||
|
self.importer.service, self.user, import_item, True, "unlisted"
|
||||||
|
)
|
||||||
|
review = models.ReviewRating.objects.get(book=self.book, user=self.user)
|
||||||
|
self.assertIsInstance(review, models.ReviewRating)
|
||||||
|
self.assertEqual(review.rating, 5.0)
|
||||||
|
self.assertEqual(review.published_date, make_date(2021, 5, 10))
|
||||||
|
self.assertEqual(review.privacy, "unlisted")
|
Loading…
Reference in a new issue