Skip to content

Commit ed217b7

Browse files
authored
Update dagster-apprise lockfile (#300)
* Update dagster-apprise lockfile * Format dagster-apprise after lockfile update * Update Apprise invalid URL expectations
1 parent c4783d5 commit ed217b7

2 files changed

Lines changed: 842 additions & 544 deletions

File tree

libraries/dagster-apprise/dagster_apprise_tests/test_comprehensive_urls.py

Lines changed: 18 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,9 @@ def test_pushover_url_variations(self):
8181
]
8282

8383
for url in pushover_urls:
84-
assert (
85-
validate_apprise_url(url) is True
86-
), f"Pushover URL {url} should be valid"
84+
assert validate_apprise_url(url) is True, (
85+
f"Pushover URL {url} should be valid"
86+
)
8787

8888
def test_discord_url_variations(self):
8989
"""Test various Discord URL formats."""
@@ -95,9 +95,9 @@ def test_discord_url_variations(self):
9595
]
9696

9797
for url in discord_urls:
98-
assert (
99-
validate_apprise_url(url) is True
100-
), f"Discord URL {url} should be valid"
98+
assert validate_apprise_url(url) is True, (
99+
f"Discord URL {url} should be valid"
100+
)
101101

102102
def test_matrix_url_variations(self):
103103
"""Test various Matrix URL formats."""
@@ -109,9 +109,9 @@ def test_matrix_url_variations(self):
109109
]
110110

111111
for url in matrix_urls:
112-
assert (
113-
validate_apprise_url(url) is True
114-
), f"Matrix URL {url} should be valid"
112+
assert validate_apprise_url(url) is True, (
113+
f"Matrix URL {url} should be valid"
114+
)
115115

116116
def test_gotify_url_variations(self):
117117
"""Test various Gotify URL formats."""
@@ -123,9 +123,9 @@ def test_gotify_url_variations(self):
123123
]
124124

125125
for url in gotify_urls:
126-
assert (
127-
validate_apprise_url(url) is True
128-
), f"Gotify URL {url} should be valid"
126+
assert validate_apprise_url(url) is True, (
127+
f"Gotify URL {url} should be valid"
128+
)
129129

130130
def test_ifttt_url_variations(self):
131131
"""Test various IFTTT URL formats."""
@@ -174,9 +174,9 @@ def test_urls_with_parameters(self):
174174
]
175175

176176
for url in param_urls:
177-
assert (
178-
validate_apprise_url(url) is True
179-
), f"Parameterized URL {url} should be valid"
177+
assert validate_apprise_url(url) is True, (
178+
f"Parameterized URL {url} should be valid"
179+
)
180180

181181
def test_invalid_urls(self):
182182
"""Test validation of invalid URLs."""
@@ -213,12 +213,11 @@ def test_invalid_urls(self):
213213
"syslog://hostname", # Invalid Syslog format
214214
"twitter://consumer_key:consumer_secret@access_token:access_token_secret", # Invalid Twitter format
215215
"vonage://api_key:api_secret@from_number", # Invalid Vonage format
216-
"webex://webhook_url", # Invalid Webex format
217216
"xmpp://user:pass@hostname", # Invalid XMPP format
218217
]
219218

220219
for url in invalid_urls:
221220
if url is not None:
222-
assert (
223-
validate_apprise_url(url) is False
224-
), f"URL {url} should be invalid"
221+
assert validate_apprise_url(url) is False, (
222+
f"URL {url} should be invalid"
223+
)

0 commit comments

Comments
 (0)