跳到内容
新发行

对这个项目有疑问吗?注册一个免费的GitHub帐户以打开一个问题,并联系其维护者和社区。

单击“注册GitHub”,表示您同意我们的服务条款隐私声明。我们偶尔会向您发送与帐户相关的电子邮件。

已经在GitHub上了?登录到您的帐户

删除重命名rn UI测试 #39042个

对话

约斯特内斯
复制链接
贡献者

@约斯特内斯 乔斯特内斯 评论2022年2月24日

描述

本PR删除了其他测试中涉及的Gutenberg Mobile的1 React Native E2E UI测试:应该能够添加新的段落块

这个PR还包括重命名更改,我最初认为这3个功能是相同的,并准备将它们组合起来,但发现有3个不同的块有错别字,修复了这个PR中的错别字。

测试说明

确保React Native UI测试仍然通过gutenberg-mobile上的CI.PR:wordpress-mobile/gutenberg-mobile#4608

屏幕截图

不适用

更改的类型

删除了冗余测试并修复了打字错误

检查清单:

  • 我的代码已经过测试。
  • 我的代码遵循WordPress代码风格。
  • 我的代码遵循可访问性标准。
  • 我已经用键盘和屏幕阅读器测试了我的更改。
  • 我的代码有适当的内联文档。
  • 如果合适的话,我已经包括了开发人员文档。
  • 我已经更新了所有受此PR中任何重构/重命名影响的React Native文件(请手动搜索所有*.nature.js需要重命名或删除的术语的文件)。
  • 如果合适的话,我已经更新了相关的模式。

@github-操作 github-操作 机器人程序补充这个首次贡献者 Gutenberg存储库的首次贡献者打开的拉取请求标签2022年2月24日
@github-操作
复制链接

github-操作 机器人程序 评论2022年2月24日

👋 感谢您的第一次Pull Request,感谢您帮助建立古腾堡和WordPress的未来,@约斯特内斯! 如果你错过了,我们希望你能加入我们的懒散社区,我们在哪里举行每周例会向任何人开放,以相互协调。

如果你想了解更多关于WordPress开发的一般信息,请查看核心手册充满了有用的信息。

@杰瑞克斯 杰瑞克斯补充移动应用程序-即Android或iOS 块编辑器的本地移动实现。(注意:在脚本中使用,ping移动用户进行更改) 移动应用程序-自动化 用于启动移动应用程序公关自动化的标签标签2022年2月24日
@大肆宣传
复制链接
贡献者

感谢您打开此公关@约斯特内斯!

应该能够将块插入柱中测试涵盖于在2个现有块之间插入测试

也许第一个测试的标题不太准确,但似乎比从插入器中添加块要做得更多。一个特别“敏感”的动作是将文本放在段落块中,这是一个非常关键的流程。这个在两个现有块之间插入测试没有涵盖这一点。如果该操作在其他地方也有涉及,那么这可能仍然是一个多余的测试,但你能确定吗?谢谢!

应该能够从标题开始在帖子的开头插入块覆盖于应该能够创建带有标题和段落块的帖子

嗯,第一个文本的关键区别在于,在标题中时从插入器添加一个新块(例如,希望新块位于内容顶部而不是末尾)。我的印象是第二次测试没有涵盖这一点🤔 . 也许我遗漏了一些细节?请让我知道🙇

应该能够添加新的段落块

同意,在这个阶段,许多测试实际上都在执行这个测试的操作。

应该能够创建包含多个段落块的帖子测试包含在多项测试中

我的理解是,该测试所涵盖的关键位(粘贴创建多个块的多行文本)仅由应该能够将块插入柱中测试,但这也被删除在本公关中,所以,不确定这是否真的涵盖在其他地方,或者我们可以重命名测试以使其更准确?

@约斯特内斯
复制链接
贡献者 作者

你好@炒作,谢谢你的评论!所有问题都是有根据的,感谢您对公关的深入了解🙇

我的理解是,此测试所涵盖的关键位(粘贴创建多个块的多行文本)仅包含在应该能够将块插入到后测试中的中,但在本PR中也删除了该关键位,因此,不确定这是否确实包含在其他地方,或者我们可以重命名测试以使其更准确?

我比较的测试用例是标题和段落块测试,看起来它确实涵盖了正在创建的多个多行块。虽然区别是一个创建了2个段落,但我建议删除的那个创建了3个段落。此处记录测试以进行比较,并确保涵盖:

  1. 应该能够创建带有标题和段落块的帖子
  2. 应该能够创建包含多个段落块的帖子

一个特别“敏感”的动作是将文本放在段落块中,这是一个非常关键的流程。
你说得对,其他测试没有涵盖这一点。

第一个文本的关键区别在于,在插入标题时从插入器中添加一个新块(例如,希望新块位于内容顶部而不是末尾)

好主意!其他两个测试虽然涵盖了测试所提及的内容,但并没有涵盖您提到的特定案例。我可能有点急于删除这些测试😅 我将恢复对此所做的更改,稍微重命名一个测试,并暂时将步骤保留为-is。

@大肆宣传
复制链接
贡献者

我将其与之进行比较的测试用例是标题和段落块测试,看起来它确实涵盖了正在创建的多个块以及多行。虽然区别是一个创建了2个段落,但我建议删除的那个创建了3个段落。此处记录测试以进行比较,并确保涵盖:

应该能够创建带有标题和段落块的帖子
应该能够创建包含多个段落块的帖子

啊,我想我明白了不同之处,抱歉,我之前没有明确指出:第一个测试通过UI(Inserter)插入不同的段落来创建不同的段落,而第二个测试是在键入时点击“Enter/Return”键来创建它们。换句话说,第二个测试是测试多段文本的编写流程,而第一个测试是通过UI“手动”完成。这些流的代码路径实际上是不同的,所以我认为测试并不完全涵盖相同的情况。WDYT?

@约斯特内斯
复制链接
贡献者 作者

啊,我想我明白了不同之处,抱歉,我之前没有明确指出:第一个测试通过UI(Inserter)插入不同的段落来创建不同的段落,而第二个测试是在键入时点击“Enter/Return”键来创建它们。

啊,好吧,我现在明白你的意思了。通过将文本发送到段落块功能。但现在我考虑一下,在重构其他测试时,它可能会被删除/覆盖,所以最好暂时保留它,因为这个测试专门测试了一个流。我会还原更改。

@大肆宣传
复制链接
贡献者

所以,最好现在就把它留在这里,因为这个测试专门测试了一个流。我会还原更改。

啊,谢谢你确认Jos!在这个阶段,你能不能更新公关描述,以便更好地反映公关的当前状态,包括它所改变/引入的内容?我的理解是,在这一点上,只有一个测试被删除了,但是的,很高兴在更新描述后再次通过。谢谢!

@约斯特内斯
复制链接
贡献者 作者

你能不能更新公关描述,以便更好地反映公关的当前状态,包括它所改变/引入的内容?

很抱歉没有提前更新,我已经更新了描述以更好地反映现在的更改。再次感谢您的复习!

@大肆宣传
复制链接
贡献者

我注意到公关检查表中有许多未检查的项目Jos,你能把它们全部检查一遍并标记为已完成吗?这意味着公关作者确实已经通过了检查。谢谢!

复制链接
贡献者

@大肆宣传 大肆宣传 留下了评论

选择隐藏此评论的原因

将显示原因,以便向其他人描述此评论。了解更多信息.

LGTM!感谢您对本次公关的反复关注@约斯特内斯!

请随意合并,我建议的过程是首先合并古腾堡PR,更新古腾堡移动PR以指向“古腾堡主干上的合并提交”,然后在CI上变绿时也合并它。谢谢!

@大肆宣传 大肆宣传合并提交c64e6b3号机组进入之内 WordPress(文字出版社):大旅行箱 2022年3月9日
@github-操作 github-操作 机器人程序将此添加到古腾堡12.8里程碑2022年3月9日
@约斯特内斯 约斯特内斯删除了 删除重命名rn-ui测试 分支 2022年3月15日08:26
免费注册 在GitHub上加入此对话.已经有帐户了吗?登录以发表评论
标签
首次贡献者 Gutenberg存储库的首次贡献者打开的Pull请求 移动应用程序-自动化 用于启动移动应用程序公关自动化的标签 移动应用程序-即Android或iOS 块编辑器的本地移动实现。(注意:在脚本中使用,ping移动用户进行更改)
项目
还没有
开发

成功合并此请求可能会解决这些问题。

还没有

3名参与者