Skip to content

Commit

Permalink
✅ Add initial test for configuration
Browse files Browse the repository at this point in the history
  • Loading branch information
ianhomer committed Sep 25, 2024
1 parent 75185cc commit 5835a9a
Show file tree
Hide file tree
Showing 5 changed files with 16 additions and 5 deletions.
4 changes: 2 additions & 2 deletions ask/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@
import os


def load_config():
def load_config(config_file_name="~/.config/ask/ask.ini"):
config = configparser.ConfigParser()
config_file_path = os.path.expanduser("~/.config/ask/ask.ini")
config_file_path = os.path.expanduser(config_file_name)
config.read(config_file_path)

return config
Empty file added ask/tests/config/__init__.py
Empty file.
2 changes: 2 additions & 0 deletions ask/tests/config/simple.ini
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
[transcribe]
filename = /tmp/simple-transcribe.txt
10 changes: 10 additions & 0 deletions ask/tests/config/test_config.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
import os
from ...config import load_config

TESTS_DIRECTORY = os.path.dirname(__file__)


def test_simple_config():
config = load_config(f"{TESTS_DIRECTORY}/simple.ini")

assert config.get("transcribe", "filename") == "/tmp/simple-transcribe.txt"
5 changes: 2 additions & 3 deletions ask/tests/test_ask_transcribe.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
from typing import Optional, Callable
from collections import deque
from typing import Optional

from ask.tests.transcribe_prompt_inputter import TranscribePromptInputter
import argparse
Expand All @@ -16,7 +15,7 @@ def parse_args():
dry=False,
inputs=[],
line_target=0,
transcribe_loop_sleep=0.001,
transcribe_loop_sleep=0.01,
no_markdown=True,
no_transcribe=False,
transcribe_filename=transcribe_filename,
Expand Down

0 comments on commit 5835a9a

Please sign in to comment.