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

Support :uuid column type for validate_absence_of matcher #1518

Merged
merged 5 commits into from
Nov 14, 2022

Conversation

dlupu
Copy link
Contributor

@dlupu dlupu commented Oct 31, 2022

Hello,

I run into a case where I needed to make sure that an uuid column was empty but could not use validate_absence_of matcher.

It tried to set the column with "an arbitrary value" which is not a valid uuid value.

This Pull Requests attempts to fix the situation.

vsppedro
vsppedro previously approved these changes Nov 5, 2022
@vsppedro
Copy link
Collaborator

vsppedro commented Nov 5, 2022

Hi, @dlupu, thank you very much for your contribution!

There is only one detail: rubocop detected trailing whitespace. Would be possible to fix that, please?

About the error with 3.1.2, rails_6_0 and PostgreSQL I'll take a closer look as soon as I have some time.

@vsppedro vsppedro self-requested a review November 5, 2022 13:34
@vsppedro vsppedro dismissed their stale review November 5, 2022 13:41

I did not notice the error on Test / build (3.1.2, rails_6_0, postgresql)

Remove trailing whitespace
@dlupu
Copy link
Contributor Author

dlupu commented Nov 13, 2022

@vsppedro sorry for the late reply.

I fixed the rubocop issue.

However, I'm surprised about the error that appears only on Test / build (3.1.2, rails_6_0, postgresql). I've cloned the code locally and run the failing test on my machine using

  • ruby 3.1.2
  • appraisal rails_6_0
  • postgresql db adapter

The tests passes ....

@dlupu
Copy link
Contributor Author

dlupu commented Nov 14, 2022

@vsppedro it seems that Test / build (3.1.2, rails_6_0, postgresql) is green not. I also fixed the last rubocop issue, please trigger the CI again.

@vsppedro
Copy link
Collaborator

Hi @dlupu, thanks for working on this. The failure we had earlier looks like an intermittent test. It was probably not caused by your changes. I'll keep an eye on that. Thank you again!

@vsppedro vsppedro merged commit 481712b into thoughtbot:main Nov 14, 2022
@vsppedro vsppedro mentioned this pull request Jan 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants