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

Skip tests if database and client versions do not match #2389

Conversation

yahonda
Copy link
Collaborator

@yahonda yahonda commented Aug 13, 2024

This pull request skips specs if the Oracle server and client versions do not match

https://github.com/rsim/oracle-enhanced/actions/runs/10371897254/job/28713269701?pr=2387

ActiveRecord::StatementInvalid:
OCIError: ORA-01805: possible error in date/time operation

This commit skip specs if it would raise the following error:

https://github.com/rsim/oracle-enhanced/actions/runs/10371897254/job/28713269701?pr=2387

```ruby
ActiveRecord::StatementInvalid:
OCIError: ORA-01805: possible error in date/time operation
```
@yahonda yahonda force-pushed the skip_tests_if_database_and_client_versions_do_not_match branch from 0283a91 to 3ffb0cb Compare August 14, 2024 00:34
@yahonda yahonda merged commit f4a06ae into rsim:master Aug 14, 2024
7 of 13 checks passed
@yahonda yahonda deleted the skip_tests_if_database_and_client_versions_do_not_match branch August 14, 2024 00:55
yahonda added a commit to yahonda/oracle-enhanced that referenced this pull request Sep 22, 2024
…TCH` set to true

This commit skips failing specs correctly if `DATABASE_SERVER_AND_CLIENT_VERSION_DO_NOT_MATCH` set to true

Follow up rsim#2389

```ruby
$
export DATABASE_SERVER_AND_CLIENT_VERSION_DO_NOT_MATCH="true"
bundle exec rspec ./spec/active_record/oracle_enhanced/type/timestamp_spec.rb
==> Loading config from ENV or use default
==> Running specs with ruby version 3.3.5
/home/yahonda/.rbenv/versions/3.3.5/lib/ruby/gems/3.3.0/bundler/gems/rails-cefd9e9942c5/activesupport/lib/active_support/logger_silence.rb:5: warning: logger was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.
You can add logger to your Gemfile or gemspec to silence this warning.
==> Effective ActiveRecord version 7.1.4
**

Pending: (Failures listed here are expected and do not affect your suite's status)

  1) OracleEnhancedAdapter timestamp with timezone support / TIMESTAMP WITH TIME ZONE values from ActiveRecord model should return Time value from TIMESTAMP columns
     # No reason given
     # ./spec/active_record/oracle_enhanced/type/timestamp_spec.rb:49

  2) OracleEnhancedAdapter timestamp with timezone support / TIMESTAMP WITH TIME ZONE values from ActiveRecord model should return Time value with fractional seconds from TIMESTAMP columns
     # No reason given
     # ./spec/active_record/oracle_enhanced/type/timestamp_spec.rb:63

Finished in 0.00178 seconds (files took 0.53574 seconds to load)
2 examples, 0 failures, 2 pending

$
```
yahonda added a commit to yahonda/oracle-enhanced that referenced this pull request Sep 22, 2024
…TCH` set to true

This commit skips failing specs correctly if `DATABASE_SERVER_AND_CLIENT_VERSION_DO_NOT_MATCH` set to true

Follow up rsim#2389

```ruby
$
export DATABASE_SERVER_AND_CLIENT_VERSION_DO_NOT_MATCH="true"
bundle exec rspec ./spec/active_record/oracle_enhanced/type/timestamp_spec.rb
==> Loading config from ENV or use default
==> Running specs with ruby version 3.3.5
/home/yahonda/.rbenv/versions/3.3.5/lib/ruby/gems/3.3.0/bundler/gems/rails-cefd9e9942c5/activesupport/lib/active_support/logger_silence.rb:5: warning: logger was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.
You can add logger to your Gemfile or gemspec to silence this warning.
==> Effective ActiveRecord version 7.1.4
**

Pending: (Failures listed here are expected and do not affect your suite's status)

  1) OracleEnhancedAdapter timestamp with timezone support / TIMESTAMP WITH TIME ZONE values from ActiveRecord model should return Time value from TIMESTAMP columns
     # No reason given
     # ./spec/active_record/oracle_enhanced/type/timestamp_spec.rb:49

  2) OracleEnhancedAdapter timestamp with timezone support / TIMESTAMP WITH TIME ZONE values from ActiveRecord model should return Time value with fractional seconds from TIMESTAMP columns
     # No reason given
     # ./spec/active_record/oracle_enhanced/type/timestamp_spec.rb:63

Finished in 0.00178 seconds (files took 0.53574 seconds to load)
2 examples, 0 failures, 2 pending

$
```
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.

1 participant