From d977817fbc99e6752f804cab1d0fd82b96e569e6 Mon Sep 17 00:00:00 2001 From: Mike Crute Date: Thu, 4 Feb 2010 21:56:55 -0500 Subject: Last thoughts about the parser. --- kronos/parser.py | 11 ++++++----- kronos/tests/test_parser.py | 1 + 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/kronos/parser.py b/kronos/parser.py index 8d93391..f0543f9 100644 --- a/kronos/parser.py +++ b/kronos/parser.py @@ -24,8 +24,8 @@ def parse_activity(text): description = description.strip() tokens = description.split(' ') - tags = parse_tags(tokens) - description = strip_tags(tokens) + tags = _parse_tags(tokens) + description = _strip_tags(tokens) category = None if '@' in text: @@ -38,9 +38,10 @@ def parse_activity(text): return activity -def parse_tags(tokens): +def _parse_tags(tokens): return [token.lstrip('#') for token in tokens if token.startswith('#')] -def strip_tags(tokens): - return ' '.join([token for token in tokens if not token.startswith('#')]) +def _strip_tags(tokens): + words = [token for token in tokens if not token.startswith('#')] + return ' '.join(words) diff --git a/kronos/tests/test_parser.py b/kronos/tests/test_parser.py index 015ae06..8a82a65 100644 --- a/kronos/tests/test_parser.py +++ b/kronos/tests/test_parser.py @@ -61,6 +61,7 @@ class TestWhenParsingWithTags(object): assert_equals(self.results.description, "Some cool stuff!") +# TODO: Implement this functionality when I need it. class TestWhenParsingWithTimeOffset(object): def test_should_understand_minutes(self): -- cgit v1.2.3