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
Migrate unit tests to new framework #428
Labels
Comments
madolson
added
help wanted
Extra attention is needed
good first issue
Good for newcomers
labels
May 3, 2024
This was referenced May 7, 2024
madolson
pushed a commit
that referenced
this issue
May 7, 2024
This PR fixes an error in the unit/READMED.md (see #428) in order to correct the steps for running single unit test file. Signed-off-by: Karthick Ariyaratnam <[email protected]>
This was referenced May 7, 2024
madolson
pushed a commit
that referenced
this issue
May 7, 2024
This PR migrates all tests related to util into new test framework as part of the parent issue #428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]>
madolson
added a commit
that referenced
this issue
May 7, 2024
This PR migrates all tests related to kvstore into new test framework as part of the parent issue #428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]>
This was referenced May 8, 2024
madolson
added a commit
that referenced
this issue
May 9, 2024
This patch migrates all tests in sds.c into new test framework as part of the parent issue #428. --------- Signed-off-by: Lipeng Zhu <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]>
madolson
added a commit
that referenced
this issue
May 10, 2024
This migrates unit tests related to sha1 to new framework, ref: #428. --------- Signed-off-by: Shivshankar-Reddy <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]>
madolson
added a commit
that referenced
this issue
May 12, 2024
This PR migrates all tests related to endianconv into new test framework as part of the parent issue #428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]>
karthyuom
added a commit
to karthyuom/valkey
that referenced
this issue
May 13, 2024
This PR migrates all tests related to util into new test framework as part of the parent issue valkey-io#428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: Karthick Ariyaratnam <[email protected]>
karthyuom
added a commit
to karthyuom/valkey
that referenced
this issue
May 13, 2024
This PR migrates all tests related to kvstore into new test framework as part of the parent issue valkey-io#428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]> Signed-off-by: Karthick Ariyaratnam <[email protected]>
karthyuom
pushed a commit
to karthyuom/valkey
that referenced
this issue
May 13, 2024
This patch migrates all tests in sds.c into new test framework as part of the parent issue valkey-io#428. --------- Signed-off-by: Lipeng Zhu <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]> Signed-off-by: Karthick Ariyaratnam <[email protected]>
karthyuom
pushed a commit
to karthyuom/valkey
that referenced
this issue
May 13, 2024
This migrates unit tests related to sha1 to new framework, ref: valkey-io#428. --------- Signed-off-by: Shivshankar-Reddy <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]> Signed-off-by: Karthick Ariyaratnam <[email protected]>
madolson
pushed a commit
that referenced
this issue
May 14, 2024
This is the actual PR which is created to migrate all tests related to zmalloc into new test framework as part of the parent issue #428. Signed-off-by: Karthick Ariyaratnam <[email protected]>
adetunjii
pushed a commit
to adetunjii/valkey
that referenced
this issue
May 15, 2024
This PR migrates all tests related to endianconv into new test framework as part of the parent issue valkey-io#428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]>
adetunjii
pushed a commit
to adetunjii/valkey
that referenced
this issue
May 15, 2024
) This is the actual PR which is created to migrate all tests related to zmalloc into new test framework as part of the parent issue valkey-io#428. Signed-off-by: Karthick Ariyaratnam <[email protected]>
adetunjii
pushed a commit
to adetunjii/valkey
that referenced
this issue
May 15, 2024
This PR migrates all tests related to endianconv into new test framework as part of the parent issue valkey-io#428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]> Signed-off-by: adetunjii <[email protected]>
adetunjii
pushed a commit
to adetunjii/valkey
that referenced
this issue
May 15, 2024
) This is the actual PR which is created to migrate all tests related to zmalloc into new test framework as part of the parent issue valkey-io#428. Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: adetunjii <[email protected]>
adetunjii
pushed a commit
to adetunjii/valkey
that referenced
this issue
May 15, 2024
This PR migrates all tests related to endianconv into new test framework as part of the parent issue valkey-io#428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]> Signed-off-by: adetunjii <[email protected]>
adetunjii
pushed a commit
to adetunjii/valkey
that referenced
this issue
May 15, 2024
) This is the actual PR which is created to migrate all tests related to zmalloc into new test framework as part of the parent issue valkey-io#428. Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: adetunjii <[email protected]>
hallmason17
pushed a commit
to hallmason17/valkey
that referenced
this issue
May 15, 2024
This PR migrates all tests related to endianconv into new test framework as part of the parent issue valkey-io#428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]>
hallmason17
pushed a commit
to hallmason17/valkey
that referenced
this issue
May 15, 2024
) This is the actual PR which is created to migrate all tests related to zmalloc into new test framework as part of the parent issue valkey-io#428. Signed-off-by: Karthick Ariyaratnam <[email protected]>
hallmason17
pushed a commit
to hallmason17/valkey
that referenced
this issue
May 18, 2024
This PR migrates all tests related to endianconv into new test framework as part of the parent issue valkey-io#428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]>
hallmason17
pushed a commit
to hallmason17/valkey
that referenced
this issue
May 18, 2024
) This is the actual PR which is created to migrate all tests related to zmalloc into new test framework as part of the parent issue valkey-io#428. Signed-off-by: Karthick Ariyaratnam <[email protected]>
Is there a part of this not WIP currently that I can pick up @madolson ? |
@SoulPancake The only one not picked up right now is dict.c. |
srgsanky
pushed a commit
to srgsanky/valkey
that referenced
this issue
May 19, 2024
This patch migrates all tests in sds.c into new test framework as part of the parent issue valkey-io#428. --------- Signed-off-by: Lipeng Zhu <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]>
srgsanky
pushed a commit
to srgsanky/valkey
that referenced
this issue
May 19, 2024
This migrates unit tests related to sha1 to new framework, ref: valkey-io#428. --------- Signed-off-by: Shivshankar-Reddy <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]>
srgsanky
pushed a commit
to srgsanky/valkey
that referenced
this issue
May 19, 2024
This PR migrates all tests related to endianconv into new test framework as part of the parent issue valkey-io#428. --------- Signed-off-by: Karthick Ariyaratnam <[email protected]> Signed-off-by: Madelyn Olson <[email protected]> Co-authored-by: Madelyn Olson <[email protected]>
srgsanky
pushed a commit
to srgsanky/valkey
that referenced
this issue
May 19, 2024
) This is the actual PR which is created to migrate all tests related to zmalloc into new test framework as part of the parent issue valkey-io#428. Signed-off-by: Karthick Ariyaratnam <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The legacy unit tests were a collection of distinct tests stored at the bottom of various files, that required rebuilding the server with a special flag and starting the server. We're moving to a new framework that can always be built and are running on a unified infrastructure. This issue is keeping track of migration from the old to new framework.
To migrate the tests, remove the custom testing code in the testing file lists listed below, create a new test like src/unit/test_*.c, and remove the reference in
valkey/src/server.c
Line 6890 in 5b1fd22
The text was updated successfully, but these errors were encountered: