Skip to content

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
@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

$
```
yahonda added a commit to yahonda/oracle-enhanced that referenced this pull request Jun 23, 2025
…iveRecord model"

This test has been disabled at test_11g.yml via rsim#2389
Once this test can be run with GitHub Actions, we can disable Travis CI.
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