Skip to content

Commit 5cd6434

Browse files
author
michaelyaakoby
authored
Merge pull request #14 from SolarEdgeTech/yanay_fix_test_e2e_logfile
Fix false positive in e2e logfile endpoint test
2 parents c98c336 + 8aa179f commit 5cd6434

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/test_pyctuator_e2e.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -215,13 +215,13 @@ def test_logfile_endpoint(endpoints: Endpoints) -> None:
215215
thirsty_str = "These pretzels are making me thirsty"
216216
response: Response = requests.get(
217217
endpoints.root + "logfile_test_repeater",
218-
params={"repeated_string": "thirsty_str"}
218+
params={"repeated_string": thirsty_str}
219219
)
220220
assert response.status_code == HTTPStatus.OK.value
221221

222222
response = requests.get(endpoints.logfile)
223223
assert response.status_code == HTTPStatus.OK.value
224-
assert response.text.find(thirsty_str)
224+
assert response.text.find(thirsty_str) >= 0
225225

226226
response = requests.get(endpoints.logfile, headers={"Range": "bytes=-307200"}) # Immitate SBA's 1st request
227227
assert response.status_code == HTTPStatus.PARTIAL_CONTENT.value

0 commit comments

Comments
 (0)