-
Notifications
You must be signed in to change notification settings - Fork 721
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 tests to github.com/go-quicktest/qt #1251
Labels
good first issue
Good for newcomers
Comments
sayboras
added a commit
to sayboras/ebpf
that referenced
this issue
Dec 5, 2023
sayboras
added a commit
to sayboras/ebpf
that referenced
this issue
Dec 6, 2023
Fixes: cilium#1251 Signed-off-by: Tam Mach <[email protected]>
sayboras
added a commit
to sayboras/ebpf
that referenced
this issue
Dec 6, 2023
Fixes: cilium#1251 Signed-off-by: Tam Mach <[email protected]>
sayboras
added a commit
to sayboras/ebpf
that referenced
this issue
Dec 6, 2023
Fixes: cilium#1251 Signed-off-by: Tam Mach <[email protected]>
sayboras
added a commit
to sayboras/ebpf
that referenced
this issue
Dec 6, 2023
Fixes: cilium#1251 Signed-off-by: Tam Mach <[email protected]>
lmb
pushed a commit
that referenced
this issue
Dec 6, 2023
Fixes: #1251 Signed-off-by: Tam Mach <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We currently use https://github.com/frankban/quicktest in our test suite. The API is full of
any
parameters and therefore needs some annoying work arounds when comparing non-int
values:We also rename the import from
quicktest
toqt
.There is a new "version" of the package which uses generics instead: https://github.com/go-quicktest/qt Migrate our testsuite to use the new
qt
package.The text was updated successfully, but these errors were encountered: