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

Bug553_LinShan #104

Merged
linshan merged 12 commits from Bug553_LinShan into Alpha-snapshot20230621 2024-04-25 15:12:55 +08:00
Conversation 13 Commits 12 Files Changed 3
linshan commented 2024-04-08 13:40:45 +08:00

修复了Bug553-单引号或双引号会被转义成&
我们通过html.unescape()将字符串中的HTML实体转换回原始字符。
在HTML中，一些特殊字符（如单引号和双引号）会被转义为实体编码，例如"代表双引号，'代表单引号。
通过使用html.unescape()函数，可以将这些实体编码转换回原始字符，这样做的目的是为了避免在处理HTML文本时出现错误或混淆。

修复了Bug553-单引号或双引号会被转义成& 我们通过`html.unescape()`将字符串中的HTML实体转换回原始字符。 在HTML中，一些特殊字符（如单引号和双引号）会被转义为实体编码，例如`"`代表双引号，`'`代表单引号。 通过使用`html.unescape()`函数，可以将这些实体编码转换回原始字符，这样做的目的是为了避免在处理HTML文本时出现错误或混淆。
linshan added 3 commits 2024-04-08 13:40:46 +08:00
linshan added 1 commit 2024-04-08 16:37:18 +08:00
linshan commented 2024-04-08 16:50:14 +08:00
Poster

添加了自动化测试文件

添加了自动化测试文件
mrlan commented 2024-04-10 07:54:22 +08:00
Owner

@linshan

谢谢

请将 test_file.json 中的内容放入 test_bug553_LinShan.py，然后删除 test_file.json。

这样可以少管理一个文件。而且更容易理解。不然测试脚本多了，会觉得奇怪，test_file.json 是做什么用的？增加了理解成本。

@linshan 谢谢 请将 test_file.json 中的内容放入 test_bug553_LinShan.py，然后删除 test_file.json。 这样可以少管理一个文件。而且更容易理解。不然测试脚本多了，会觉得奇怪，test_file.json 是做什么用的？增加了理解成本。
linshan commented 2024-04-12 00:32:08 +08:00
Poster

@linshan

谢谢

请将 test_file.json 中的内容放入 test_bug553_LinShan.py，然后删除 test_file.json。

这样可以少管理一个文件。而且更容易理解。不然测试脚本多了，会觉得奇怪，test_file.json 是做什么用的？增加了理解成本。

@mrlan
test_file.json是用来存放测试数据的，我将测试数据和对应的测试结果放在json文件中，这样测试的时候著需要通过加载json文件就可以获取到多个测试数据，而无需编写多个相同步骤的测试代码。

> @linshan > > 谢谢 > > 请将 test_file.json 中的内容放入 test_bug553_LinShan.py，然后删除 test_file.json。 > > 这样可以少管理一个文件。而且更容易理解。不然测试脚本多了，会觉得奇怪，test_file.json 是做什么用的？增加了理解成本。 > > > > > > @mrlan test_file.json是用来存放测试数据的，我将测试数据和对应的测试结果放在json文件中，这样测试的时候著需要通过加载json文件就可以获取到多个测试数据，而无需编写多个相同步骤的测试代码。
mrlan commented 2024-04-12 21:13:25 +08:00
Owner

@linshan

有道理。可以试用一下 pytest.mark.parametrize 这个装饰器。

例子 https://docs.pytest.org/en/7.3.x/how-to/parametrize.html

@linshan 有道理。可以试用一下 `pytest.mark.parametrize` 这个装饰器。 例子 https://docs.pytest.org/en/7.3.x/how-to/parametrize.html
linshan added 1 commit 2024-04-16 08:46:11 +08:00
linshan commented 2024-04-16 08:47:48 +08:00
Poster

@mrlan
已经删除test_json.file，并且使用pytest.mark.parametrize进行测试。

@mrlan 已经删除`test_json.file`，并且使用`pytest.mark.parametrize`进行测试。
linshan added 1 commit 2024-04-16 08:48:33 +08:00
mrlan added 2 commits 2024-04-18 20:06:24 +08:00
mrlan commented 2024-04-18 20:09:50 +08:00
Owner

@linshan

谢谢。我精简了一下你的测试代码。请按照 README.md 中的 End-to-end testing 那节安装好配置好 Edge 浏览器的 web driver，就可以运行测试了。

@linshan 谢谢。我精简了一下你的测试代码。请按照 README.md 中的 [End-to-end testing](http://118.25.96.118:3000/mrlan/EnglishPal/src/branch/Bug553_LinShan/README.md#end-to-end-testing) 那节安装好配置好 Edge 浏览器的 web driver，就可以运行测试了。
fanwenqi changed title from WIP: Bug553_LinShan to WIP: Bug553_LinShan 2024-04-22 12:28:31 +08:00
fengcheng commented 2024-04-22 12:37:43 +08:00

建议在代码的最后添加driver.quit()，因为打开浏览器后还需要关闭浏览器。 @linshan

建议在代码的最后添加`driver.quit()`，因为打开浏览器后还需要关闭浏览器。 @linshan
fengcheng commented 2024-04-22 12:38:28 +08:00

测试的用例也可以写成数组嵌套在函数中，这样测试的时候只需要打开和关闭一次浏览器。 @linshan

测试的用例也可以写成数组嵌套在函数中，这样测试的时候只需要打开和关闭一次浏览器。 @linshan
linshan added 3 commits 2024-04-22 12:48:05 +08:00
linshan commented 2024-04-22 12:49:15 +08:00
Poster

建议在代码的最后添加driver.quit()，因为打开浏览器后还需要关闭浏览器。 @linshan

接受建议，已修改 @fengcheng

> 建议在代码的最后添加`driver.quit()`，因为打开浏览器后还需要关闭浏览器。 @linshan 接受建议，已修改 @fengcheng
linshan commented 2024-04-22 12:50:52 +08:00
Poster

测试的用例也可以写成数组嵌套在函数中，这样测试的时候只需要打开和关闭一次浏览器。 @linshan

有道理，虽然我们已经使用pytest.mark.parametrize方法，但是你的建议仍然有价值。 @fengcheng

> 测试的用例也可以写成数组嵌套在函数中，这样测试的时候只需要打开和关闭一次浏览器。 @linshan 有道理，虽然我们已经使用`pytest.mark.parametrize`方法，但是你的建议仍然有价值。 @fengcheng
mrlan commented 2024-04-24 15:27:34 +08:00
Owner

@linshan

Alpha-snapshot20230621 已经更新， 需要与 Alpha-snapshot20230621 同步一下。

@linshan Alpha-snapshot20230621 已经更新， 需要与 Alpha-snapshot20230621 同步一下。
linshan added 1 commit 2024-04-24 18:15:32 +08:00
linshan commented 2024-04-24 18:16:14 +08:00
Poster

@linshan

Alpha-snapshot20230621 已经更新， 需要与 Alpha-snapshot20230621 同步一下。

已同步。 @mrlan

> @linshan > > Alpha-snapshot20230621 已经更新， 需要与 Alpha-snapshot20230621 同步一下。 已同步。 @mrlan
fengcheng reviewed 2024-04-24 18:17:56 +08:00
fengcheng left a comment

code review完毕

code review完毕
linshan changed title from WIP: Bug553_LinShan to Bug553_LinShan 2024-04-24 18:21:11 +08:00
mrlan commented 2024-04-25 07:58:27 +08:00
Owner

APPROVED

APPROVED
linshan merged commit df34770af0 into Alpha-snapshot20230621 2024-04-25 15:12:55 +08:00
Sign in to join this conversation.
Reviewers
No reviewers
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#104
There is no content yet.
Delete Branch "Bug553_LinShan"

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?