Skip to content

test: Move VDF validation test to gunit#428

Merged
malone-at-work merged 2 commits intomainfrom
maloney/testing/1
May 5, 2026
Merged

test: Move VDF validation test to gunit#428
malone-at-work merged 2 commits intomainfrom
maloney/testing/1

Conversation

@malone-at-work
Copy link
Copy Markdown
Member

@malone-at-work malone-at-work commented May 5, 2026

Drop the system tests which restart the server causing slower tests.

@malone-at-work malone-at-work force-pushed the maloney/testing/1 branch 2 times, most recently from d06bb6e to 6c1f70c Compare May 5, 2026 13:39
Comment thread unittest/gunit/villagesql/validate-t.cc Outdated
"MYTYPE::transform", // unrecognised method suffix
"WRONGTYPE::from_string", // prefix does not match type name
"::MYTPE::transform",
"MYTPE::::transform",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should probably be pairs of bad_name, expected error string?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done. But the string right now is pretty boring!

@malone-at-work malone-at-work merged commit 488465c into main May 5, 2026
3 checks passed
@malone-at-work malone-at-work deleted the maloney/testing/1 branch May 5, 2026 18:18
@github-actions github-actions Bot locked and limited conversation to collaborators May 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants