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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Unit test failing on AqlQueryBuilder #79

Open
arn-the-long-beard opened this issue Sep 6, 2021 · 5 comments 路 Fixed by #80
Open

Unit test failing on AqlQueryBuilder #79

arn-the-long-beard opened this issue Sep 6, 2021 · 5 comments 路 Fixed by #80

Comments

@arn-the-long-beard
Copy link
Contributor

arn-the-long-beard commented Sep 6, 2021 •
edited

Hey @fMeow , how are you doing ? 馃槃

I am still playing with Rust 馃槅

But I got an issue today and I am not sure why. I did updated arangor to 0.5.0 since I am using the latest actix-web now.

Here is the error when I want to use arangors 0.5.0 or unit test it :

image

Did somebody sae that 0.5.0 tests were failing ?
@mrtnzlml
Copy link
Contributor

mrtnzlml commented Sep 6, 2021

I am not using arangors directly so this might not be much of a help, but, it's most likely because typed-builder dependency got upgraded, see: adeira/universe@5250232

Maybe you can pin the dependency?

@ManevilleF ManevilleF mentioned this issue Sep 7, 2021
Merged
@ManevilleF
Copy link
Contributor

Does anyone have any idea why #80 is failing actions?

@fMeow fMeow closed this as completed in #80 Sep 9, 2021
@fMeow
Copy link
Owner

fMeow commented Sep 9, 2021 •
edited

I will look into the CI issue this weekend. It's related to behavior change of arangoDB.

@fMeow fMeow reopened this Sep 9, 2021
@arn-the-long-beard
Copy link
Contributor Author

Thank you guys for having worked on the issue 馃憤

@ManevilleF
Copy link
Contributor

I will look into the CI issue this weekend. It's related to behavior change of arangoDB.

I tried to find in ArangoDB 3.8 changelog, but I saw nothing. So I just removed the line from my latest PR #81

@ManevilleF ManevilleF mentioned this issue Sep 21, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@mrtnzlml @fMeow @ManevilleF @arn-the-long-beard