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

test: enable skipped tests #788

Merged
merged 4 commits into from
Dec 6, 2022
Merged

Conversation

nodece
Copy link
Contributor

@nodece nodece commented Jul 19, 2022

Signed-off-by: Zixuan Liu [email protected]

Motivation

Enable skipped tests.

Modifications

  • Enable skipped tests
  • Refactor some tests based on gomega

Documentation

Check the box below.

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@nodece nodece requested review from zymap and a team as code owners July 19, 2022 09:27
@github-actions github-actions bot added the no-need-doc This pr does not need any document label Jul 19, 2022
@nodece nodece force-pushed the enable-skipped-tests branch 4 times, most recently from 8c2e478 to 898ed15 Compare July 19, 2022 10:22
@nodece
Copy link
Contributor Author

nodece commented Jul 21, 2022

This PR has been blocked by apache/pulsar#16355.

Signed-off-by: Zixuan Liu <[email protected]>
@nodece nodece merged commit 9b6a9d0 into streamnative:master Dec 6, 2022
nodece added a commit that referenced this pull request Dec 6, 2022
* test: enable skipped tests

Signed-off-by: Zixuan Liu <[email protected]>

* test: enable skipped tests

Signed-off-by: Zixuan Liu <[email protected]>

* ci: upgrade Pulsar version

Signed-off-by: Zixuan Liu <[email protected]>

* Update pulsar version

Signed-off-by: Zixuan Liu <[email protected]>
(cherry picked from commit 9b6a9d0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-need-doc This pr does not need any document
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants