Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add tcl test 2 #2497

Merged
merged 6 commits into from
Mar 12, 2024
Merged

feat: add tcl test 2 #2497

merged 6 commits into from
Mar 12, 2024

Conversation

Mixficsol
Copy link
Collaborator

@Mixficsol Mixficsol commented Mar 12, 2024

fix: #2046

在这个 PR 中,我们新加了之前注释掉的 basic, scan, expire, multi, quit, list, pubsub, slowlog, protocol, maxmemory, bitops, hyperloglog的 TCL 测试,将 RedisTCL 单测在 Pika 上几乎全部进行了覆盖,至于剩下的一部分单测类型没有解掉注释的原因在于这些类型中部分命令 Pika 暂不支持,例如 Rename, Debug, Move, Save, Sort, Eval, Dump, Command 等命令

In this PR, We let go of the previous comments on basic, scan, expire, multi, quit, list, pubsub, slowlog, protocol, maxmemory, bitops, The TCL test of hyperloglog covers almost all the TCL single test of Redis on Pika. As for the remaining part of the single test types, the reason why the comments are not removed is that some commands of these types are not supported by Pika. For example, Rename, Debug, Move, Save, Sort, Eval, Dump, command, and so on

截屏2024-03-04 16 36 14

@github-actions github-actions bot added ☢️ Bug Something isn't working 🧹 Updates This will not be worked on labels Mar 12, 2024
@Mixficsol Mixficsol changed the title fix: Tcl test all feat: add tcl test Mar 12, 2024
@Mixficsol Mixficsol removed the ☢️ Bug Something isn't working label Mar 12, 2024
@github-actions github-actions bot added the ✏️ Feature New feature or request label Mar 12, 2024
@Mixficsol Mixficsol requested a review from machinly March 12, 2024 09:46
@Mixficsol Mixficsol added 3.5.3 test and removed ✏️ Feature New feature or request labels Mar 12, 2024
@github-actions github-actions bot added the ✏️ Feature New feature or request label Mar 12, 2024
test {AUTH succeeds when the right password is given} {
r auth foobar
} {OK}
# test {AUTH fails when a wrong password is given} {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

注释的这些用例,pika不兼容是吗

@AlexStocks AlexStocks merged commit b83ad79 into OpenAtomFoundation:unstable Mar 12, 2024
11 checks passed
luky116 pushed a commit to luky116/pika that referenced this pull request Mar 14, 2024
* add tcl test

* Update default.conf

---------

Co-authored-by: wuxianrong <wuxianrong@360.cn>
Co-authored-by: Xin.Zh <dragoncharlie@foxmail.com>
luky116 pushed a commit to luky116/pika that referenced this pull request Mar 14, 2024
* add tcl test

* Update default.conf

---------

Co-authored-by: wuxianrong <wuxianrong@360.cn>
Co-authored-by: Xin.Zh <dragoncharlie@foxmail.com>
@Mixficsol Mixficsol deleted the tcl-test-all branch March 14, 2024 12:16
luky116 pushed a commit that referenced this pull request Mar 18, 2024
* add tcl test

* Update default.conf

---------

Co-authored-by: wuxianrong <wuxianrong@360.cn>
Co-authored-by: Xin.Zh <dragoncharlie@foxmail.com>
@AlexStocks AlexStocks changed the title feat: add tcl test feat: add tcl test 2 Mar 21, 2024
bigdaronlee163 pushed a commit to bigdaronlee163/pika that referenced this pull request Jun 8, 2024
* add tcl test

* Update default.conf

---------

Co-authored-by: wuxianrong <wuxianrong@360.cn>
Co-authored-by: Xin.Zh <dragoncharlie@foxmail.com>
cheniujh pushed a commit to cheniujh/pika that referenced this pull request Sep 24, 2024
* add tcl test

* Update default.conf

---------

Co-authored-by: wuxianrong <wuxianrong@360.cn>
Co-authored-by: Xin.Zh <dragoncharlie@foxmail.com>
cheniujh pushed a commit to cheniujh/pika that referenced this pull request Sep 24, 2024
* add tcl test

* Update default.conf

---------

Co-authored-by: wuxianrong <wuxianrong@360.cn>
Co-authored-by: Xin.Zh <dragoncharlie@foxmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.5.3 ✏️ Feature New feature or request test 🧹 Updates This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pika 的 TCL 测试完善计划
3 participants