mirror of
https://github.com/veekun/pokedex.git
synced 2024-08-20 18:16:34 +00:00
I18n for the database schema
- Helper base class: Named Subclasses: OfficiallyNamed, UnofficiallyNamed for these, a 'name' column is created in the appropriate text table also, they get automatic __str__/__repr__/__unicode__ - Faux columns: ProseColumn, TextColumn these become columns in the appropriate text tables these text tables (*_text, *_prose) are auto-generated at the end the main table gets one property (singular name) that gets the English text and one (plural name) with dict of texts keyed by language - Every named table gets 'identifier' - Languages compare & hash equal to their identifiers - Existing foreign-name tables replaced by the autogenerated ones - order_by: names replaced by identifiers - New function: all_tables(), yields all tables - Markdown move properties removed for now - Schema test suite
This commit is contained in:
parent
59f1e835f4
commit
d813e24b25
3 changed files with 469 additions and 265 deletions
File diff suppressed because it is too large
Load diff
71
pokedex/tests/test_schema.py
Normal file
71
pokedex/tests/test_schema.py
Normal file
|
@ -0,0 +1,71 @@
|
||||||
|
# encoding: utf8
|
||||||
|
from nose.tools import *
|
||||||
|
import unittest
|
||||||
|
from sqlalchemy.orm import class_mapper
|
||||||
|
|
||||||
|
from pokedex.db import tables, markdown
|
||||||
|
|
||||||
|
def test_variable_names():
|
||||||
|
"""We want pokedex.db.tables to export tables using the class name"""
|
||||||
|
for varname in dir(tables):
|
||||||
|
if not varname[0].isupper():
|
||||||
|
continue
|
||||||
|
table = getattr(tables, varname)
|
||||||
|
try:
|
||||||
|
if not issubclass(table, tables.TableBase) or table is tables.TableBase:
|
||||||
|
continue
|
||||||
|
except TypeError:
|
||||||
|
continue
|
||||||
|
classname = table.__name__
|
||||||
|
if classname and varname[0].isupper():
|
||||||
|
assert varname == classname, '%s refers to %s' % (varname, classname)
|
||||||
|
for table in tables.all_tables():
|
||||||
|
assert getattr(tables, table.__name__) is table
|
||||||
|
|
||||||
|
def test_texts():
|
||||||
|
"""Check DB schema for integrity of text columns & translations.
|
||||||
|
|
||||||
|
Mostly protects against copy/paste oversights and rebase hiccups.
|
||||||
|
If there's a reason to relax the tests, do it
|
||||||
|
"""
|
||||||
|
for table in sorted(tables.all_tables(), key=lambda t: t.__name__):
|
||||||
|
if issubclass(table, tables.LanguageSpecific):
|
||||||
|
good_formats = 'markdown plaintext gametext'.split()
|
||||||
|
assert_text = '%s is language-specific'
|
||||||
|
else:
|
||||||
|
good_formats = 'identifier latex'.split()
|
||||||
|
assert_text = '%s is not language-specific'
|
||||||
|
mapper = class_mapper(table)
|
||||||
|
for column in sorted(mapper.c, key=lambda c: c.name):
|
||||||
|
format = column.info.get('format', None)
|
||||||
|
if format is not None:
|
||||||
|
if format not in good_formats:
|
||||||
|
raise AssertionError(assert_text % column)
|
||||||
|
is_markdown = isinstance(column.type, markdown.MarkdownColumn)
|
||||||
|
if is_markdown != (format == 'markdown'):
|
||||||
|
raise AssertionError('%s: markdown format/column type mismatch' % column)
|
||||||
|
if (format != 'identifier') and (column.name == 'identifier'):
|
||||||
|
raise AssertionError('%s: identifier column name/type mismatch' % column)
|
||||||
|
if column.info.get('official', None) and format not in 'gametext plaintext':
|
||||||
|
raise AssertionError('%s: official text with bad format' % column)
|
||||||
|
else:
|
||||||
|
if isinstance(column.type, (markdown.MarkdownColumn, tables.Unicode)):
|
||||||
|
raise AssertionError('%s: text column without format' % column)
|
||||||
|
if column.name == 'name' and format != 'plaintext':
|
||||||
|
raise AssertionError('%s: non-plaintext name' % column)
|
||||||
|
# No mention of English in the description
|
||||||
|
assert 'English' not in column.info['description'], column
|
||||||
|
|
||||||
|
def test_identifiers_with_names():
|
||||||
|
"""Test that named tables have identifiers, and non-named tables don't
|
||||||
|
|
||||||
|
...have either names or identifiers.
|
||||||
|
"""
|
||||||
|
for table in sorted(tables.all_tables(), key=lambda t: t.__name__):
|
||||||
|
if issubclass(table, tables.Named):
|
||||||
|
assert issubclass(table, tables.OfficiallyNamed) or issubclass(table, tables.UnofficiallyNamed), table
|
||||||
|
assert hasattr(table, 'identifier'), table
|
||||||
|
else:
|
||||||
|
assert not hasattr(table, 'identifier'), table
|
||||||
|
if not issubclass(table, tables.LanguageSpecific):
|
||||||
|
assert not hasattr(table, 'name'), table
|
2
setup.py
2
setup.py
|
@ -8,7 +8,7 @@ setup(
|
||||||
'pokedex': ['data/csv/*.csv']
|
'pokedex': ['data/csv/*.csv']
|
||||||
},
|
},
|
||||||
install_requires=[
|
install_requires=[
|
||||||
'SQLAlchemy>=0.6',
|
'SQLAlchemy>=0.6.6',
|
||||||
'whoosh>=1.1.0',
|
'whoosh>=1.1.0',
|
||||||
'markdown',
|
'markdown',
|
||||||
'construct',
|
'construct',
|
||||||
|
|
Loading…
Reference in a new issue