Skip to content

Fix optional aiohttp import and update tests #20

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
trader-payne merged 1 commit into from
Jun 4, 2025
Merged

Fix optional aiohttp import and update tests #20

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion dtop/views/log_view.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
import os
from ..utils.utils import safe_addstr
import concurrent.futures
import aiohttp
from datetime import datetime, timezone
import json

Expand Down
46 changes: 23 additions & 23 deletions tests/test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,25 +12,25 @@ def test_imports():
print("Testing imports...")

try:
import utils
from dtop.utils import utils
print("✓ utils module")
import config

from dtop.utils import config
print("✓ config module")
import stats

from dtop.core import stats
print("✓ stats module")
import container_actions

from dtop.actions import container_actions
print("✓ container_actions module")
import log_view

from dtop.views import log_view
print("✓ log_view module")
import docker_tui

from dtop.core import docker_tui
print("✓ docker_tui module")
import main

from dtop import main
print("✓ main module")

print("\nAll modules imported successfully!")
Expand All @@ -42,14 +42,14 @@ def test_imports():
def check_files():
"""Check if all necessary files exist"""
required_files = [
"main.py",
"docker_tui.py",
"log_view.py",
"container_actions.py",
"utils.py",
"config.py",
"stats.py",
"normalize_logs.py",
"dtop/main.py",
"dtop/core/docker_tui.py",
"dtop/views/log_view.py",
"dtop/actions/container_actions.py",
"dtop/utils/utils.py",
"dtop/utils/config.py",
"dtop/core/stats.py",
"dtop/utils/normalize_logs.py",
]

print("\nChecking required files...")
Expand All @@ -72,8 +72,8 @@ def check_files():
def check_permissions():
"""Check if files have proper execution permissions"""
executable_files = [
"main.py",
"normalize_logs.py"
"dtop/main.py",
"dtop/utils/normalize_logs.py"
]

print("\nChecking executable permissions...")
Expand Down