mirror of
https://github.com/Hopiu/bowser.git
synced 2026-03-16 19:10:24 +00:00
d3119f0b10
Feature: Smart protocol handling - If no protocol is provided, assume https:// - URLs like 'example.com' become 'https://example.com' - 'example.com/path' becomes 'https://example.com/path' - Special 'about:' URLs are preserved as-is Implementation: - Added _normalize_url() method to Browser class - Checks for '://' in URL to detect existing protocol - Strips whitespace from URLs - Applied in both new_tab() and navigate_to() methods - Supports all URL formats (subdomains, ports, paths, queries) URL Normalization Logic: 1. Strip leading/trailing whitespace 2. Check if URL already has a protocol ('://') 3. Check for special about: URLs 4. Otherwise prepend 'https://' Examples: - 'example.com' → 'https://example.com' - 'https://example.com' → 'https://example.com' (unchanged) - 'about:startpage' → 'about:startpage' (unchanged) - 'www.example.com:8080' → 'https://www.example.com:8080' - 'localhost:3000' → 'https://localhost:3000' Tests added (10 test cases): - test_normalize_url_with_https - test_normalize_url_with_http - test_normalize_url_without_protocol - test_normalize_url_with_path - test_normalize_url_with_about - test_normalize_url_strips_whitespace - test_normalize_url_with_query_string - test_normalize_url_with_subdomain - test_normalize_url_with_port - test_normalize_url_localhost Existing tests still passing (15/15)
70 lines
2.7 KiB
Python
70 lines
2.7 KiB
Python
"""Tests for URL normalization."""
|
|
|
|
import pytest
|
|
from src.browser.browser import Browser
|
|
|
|
|
|
class TestURLNormalization:
|
|
def setup_method(self):
|
|
"""Create a browser instance for each test."""
|
|
self.browser = Browser()
|
|
|
|
def test_normalize_url_with_https(self):
|
|
"""Test that URLs with https:// protocol are unchanged."""
|
|
url = "https://example.com"
|
|
normalized = self.browser._normalize_url(url)
|
|
assert normalized == "https://example.com"
|
|
|
|
def test_normalize_url_with_http(self):
|
|
"""Test that URLs with http:// protocol are unchanged."""
|
|
url = "http://example.com"
|
|
normalized = self.browser._normalize_url(url)
|
|
assert normalized == "http://example.com"
|
|
|
|
def test_normalize_url_without_protocol(self):
|
|
"""Test that URLs without protocol get https:// added."""
|
|
url = "example.com"
|
|
normalized = self.browser._normalize_url(url)
|
|
assert normalized == "https://example.com"
|
|
|
|
def test_normalize_url_with_path(self):
|
|
"""Test that URLs with path but no protocol get https:// added."""
|
|
url = "example.com/path/to/page"
|
|
normalized = self.browser._normalize_url(url)
|
|
assert normalized == "https://example.com/path/to/page"
|
|
|
|
def test_normalize_url_with_about(self):
|
|
"""Test that about: URLs are not modified."""
|
|
url = "about:startpage"
|
|
normalized = self.browser._normalize_url(url)
|
|
assert normalized == "about:startpage"
|
|
|
|
def test_normalize_url_strips_whitespace(self):
|
|
"""Test that leading/trailing whitespace is stripped."""
|
|
url = " example.com "
|
|
normalized = self.browser._normalize_url(url)
|
|
assert normalized == "https://example.com"
|
|
|
|
def test_normalize_url_with_query_string(self):
|
|
"""Test that URLs with query strings work correctly."""
|
|
url = "example.com/search?q=test"
|
|
normalized = self.browser._normalize_url(url)
|
|
assert normalized == "https://example.com/search?q=test"
|
|
|
|
def test_normalize_url_with_subdomain(self):
|
|
"""Test that subdomains work correctly."""
|
|
url = "www.example.com"
|
|
normalized = self.browser._normalize_url(url)
|
|
assert normalized == "https://www.example.com"
|
|
|
|
def test_normalize_url_with_port(self):
|
|
"""Test that ports are preserved."""
|
|
url = "example.com:8080"
|
|
normalized = self.browser._normalize_url(url)
|
|
assert normalized == "https://example.com:8080"
|
|
|
|
def test_normalize_url_localhost(self):
|
|
"""Test that localhost URLs work correctly."""
|
|
url = "localhost:3000"
|
|
normalized = self.browser._normalize_url(url)
|
|
assert normalized == "https://localhost:3000"
|