mrlan
/
EnglishPal
4
0
Fork 11
Code Issues Pull Requests 22 Projects Releases Wiki Activity

Bug528-TangJiao #103

Merged
tangjiao merged 10 commits from Bug528-TangJiao into Alpha-snapshot20230621 2024-05-05 21:27:39 +08:00
Conversation 5 Commits 10 Files Changed 4
tangjiao commented 2024-04-06 00:26:50 +08:00

修复了bug528
改为了include函数方便阅读和理解
添加了测试文件

修复了bug528 改为了include函数方便阅读和理解 添加了测试文件
tangjiao added 2 commits 2024-04-06 00:26:51 +08:00
mrlan commented 2024-04-07 07:34:35 +08:00
Owner

@tangjiao

不错。

测试文件名请用 test_bug528_tangjiao.py

三个测试文件合并为一个。一个 py 文件中可以定义多个测试用例的。

@tangjiao 不错。 测试文件名请用 test_bug528_tangjiao.py 三个测试文件合并为一个。一个 py 文件中可以定义多个测试用例的。
yaaqob reviewed 2024-04-08 00:25:57 +08:00
app/test/test_login_password_with_space.py Outdated
@ -0,0 +26,4 @@
password_elem.send_keys(Keys.RETURN)
# 等待一段时间确保页面加载完成
time.sleep(2)
yaaqob commented 2024-04-08 00:25:57 +08:00

不要使用time.sleep(2)来等待页面加载，而是考虑将显式等待与WebDriverWait一起使用`
因为web应用程序的加载时间可能会因页面的复杂性等多种因素而异。使用显式等待可以让您的测试脚本通过等待直到满足预期条件来动态适应这些变化。

不要使用`time.sleep(2)`来等待页面加载，而是考虑将显式等待与`WebDriverWait`一起使用` 因为web应用程序的加载时间可能会因页面的复杂性等多种因素而异。使用显式等待可以让您的测试脚本通过等待直到满足预期条件来动态适应这些变化。
yaaqob reviewed 2024-04-08 00:26:00 +08:00
app/test/test_login_password_with_space.py Outdated
@ -0,0 +30,4 @@
# 检查是否弹出警告框
alert = driver.switch_to.alert
assert "输入不能包含空格!" in alert.text
yaaqob commented 2024-04-08 00:26:00 +08:00

此外，您还应该考虑处理测试过程中可能出现的NoSuchElementException或TimeoutException等错误。
它可以增强测试用例的健壮性，并提供更有意义的错误消息。

此外，您还应该考虑处理测试过程中可能出现的`NoSuchElementException`或`TimeoutException`等错误。 它可以增强测试用例的健壮性，并提供更有意义的错误消息。
yaaqob approved these changes 2024-04-08 00:31:07 +08:00
yaaqob left a comment

我对一些应该进一步增强测试脚本的地方发表了一些意见。
如果可能的话，可以将注释应用到测试脚本中所有类似的行。

我对一些应该进一步增强测试脚本的地方发表了一些意见。 如果可能的话，可以将注释应用到测试脚本中所有类似的行。
tangjiao added 3 commits 2024-04-15 06:18:03 +08:00
tangjiao added 1 commit 2024-04-15 06:24:04 +08:00
tangjiao commented 2024-04-17 10:48:59 +08:00
Poster

我更新了提交，合并了测试文件，修改了等待方式及URL

我更新了提交，合并了测试文件，修改了等待方式及URL
mrlan reviewed 2024-04-18 20:16:41 +08:00
app/test/test_bug528_tangjiao.py
@ -0,0 +10,4 @@
@pytest.fixture(scope="module")
def driver():
# 初始化WebDriver
driver = webdriver.Edge(executable_path=r'D:\codeapp\python-3.8.5\Lib\site-packages\selenium\webdriver\edge\MicrosoftWebDriver.exe')
mrlan commented 2024-04-18 20:16:41 +08:00
Owner

@tangjiao

  • 这个路径 'D:\codeapp\python-3.8.5\Lib\site-packages\selenium\webdriver\edge\MicrosoftWebDriver.exe' 会导致你的测试脚本无法在别人的电脑上运行，因为别人没有这个路径。如何设置 PATH 变量，请参考 README.md 中的 End-to-end testing 那节。

  • 不需要额外定义 driver 这个 fixture，在 contest.py 中已有定义。

@tangjiao - 这个路径 'D:\codeapp\python-3.8.5\Lib\site-packages\selenium\webdriver\edge\MicrosoftWebDriver.exe' 会导致你的测试脚本无法在别人的电脑上运行，因为别人没有这个路径。如何设置 PATH 变量，请参考 README.md 中的 [End-to-end testing](http://118.25.96.118:3000/mrlan/EnglishPal/src/branch/Alpha-snapshot20230621/README.md#end-to-end-testing) 那节。 - 不需要额外定义 driver 这个 fixture，在 contest.py 中已有定义。
👍 1
tangjiao added 2 commits 2024-04-22 05:00:22 +08:00
tangjiao commented 2024-04-22 11:00:26 +08:00
Poster

我删除了多余的代码，并且测试成功运行。

我删除了多余的代码，并且测试成功运行。
测试成功.png
229 KiB
测试成功.png
tangjiao added 1 commit 2024-04-23 17:35:58 +08:00
mrlan commented 2024-04-26 18:02:32 +08:00
Owner

@tangjiao

Alpha-snapshot20230621 分支已经更新，请与 Alpha-snapshot20230621 分支同步一下，再推送。

@tangjiao Alpha-snapshot20230621 分支已经更新，请与 Alpha-snapshot20230621 分支同步一下，再推送。
tangjiao added 1 commit 2024-04-29 13:41:36 +08:00
mrlan commented 2024-04-29 15:08:08 +08:00
Owner

APPROVED

APPROVED
mrlan changed title from WIP: Bug528-TangJiao to Bug528-TangJiao 2024-04-29 15:08:22 +08:00
tangjiao merged commit a4e64ee4a0 into Alpha-snapshot20230621 2024-05-05 21:27:39 +08:00
Sign in to join this conversation.
Reviewers
No reviewers
yaaqob
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: mrlan/EnglishPal#103
There is no content yet.
Delete Branch "Bug528-TangJiao"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?