Rename to FilenameBuilder, with a main method named fill.
I think these names better convey what's actually going on. I updated the documentation a bit while I was at it.
This commit is contained in:
@@ -5,16 +5,16 @@ from nose.tools import assert_equal, assert_true, assert_false
|
||||
from mediagoblin import processing
|
||||
|
||||
class TestProcessing(object):
|
||||
def run_munge(self, input, format, output=None):
|
||||
munger = processing.FilenameMunger(input)
|
||||
result = munger.munge(format)
|
||||
def run_fill(self, input, format, output=None):
|
||||
builder = processing.FilenameBuilder(input)
|
||||
result = builder.fill(format)
|
||||
if output is None:
|
||||
return result
|
||||
assert_equal(output, result)
|
||||
|
||||
def test_easy_filename_munge(self):
|
||||
self.run_munge('/home/user/foo.TXT', '{basename}bar{ext}', 'foobar.txt')
|
||||
def test_easy_filename_fill(self):
|
||||
self.run_fill('/home/user/foo.TXT', '{basename}bar{ext}', 'foobar.txt')
|
||||
|
||||
def test_long_filename_munge(self):
|
||||
self.run_munge('{0}.png'.format('A' * 300), 'image-{basename}{ext}',
|
||||
'image-{0}.png'.format('A' * 245))
|
||||
def test_long_filename_fill(self):
|
||||
self.run_fill('{0}.png'.format('A' * 300), 'image-{basename}{ext}',
|
||||
'image-{0}.png'.format('A' * 245))
|
||||
|
||||
Reference in New Issue
Block a user