arooo icon indicating copy to clipboard operation
arooo copied to clipboard

Upgrade ruby from 2.7.7 to 3.x latest

Open compwron opened this issue 2 years ago • 6 comments

Check .ruby-version for what ruby we are using. Look at https://www.ruby-lang.org/en/downloads/releases/ and upgrade one version per PR until we are at the latest mainstream supported version (currently 3.2.1).

compwron avatar Feb 18 '23 15:02 compwron

Hey @compwron, can i work on this issue?

fchatterji avatar Feb 19 '23 21:02 fchatterji

@fchatterji go for it! :)

compwron avatar Feb 20 '23 16:02 compwron

Hey, I tried to look at upgrading ruby, it creates a lot of errors in the test, that with my current knowledge of the codebase will be too long to fix. I just did the rails upgrade, will unassign myself from this issue.

fchatterji avatar Feb 20 '23 20:02 fchatterji

I tried looking into upgrading Ruby and found that while the app ran locally fine, all of the tests failed. I added this gem to make the tests work "gem 'net-smtp', require: false" and then there were 45 failures. I think the stripe gem needs to be upgraded first because many of the failures were around the stripe tests.

sylphiae avatar May 27 '23 22:05 sylphiae

Failures:

  1. StripeEventHelper ChargeFailed with no Stripe customer ID #call sends an email Failure/Error: let(:event) { StripeMock.mock_webhook_event("charge.failed") }

    NoMethodError: undefined method `exists?' for File:Class

    /home/jenny/.rvm/gems/ruby-3.2.2/gems/stripe-ruby-mock-2.5.8/lib/stripe_mock/api/webhooks.rb:7:in `mock_webhook_payload'

    /home/jenny/.rvm/gems/ruby-3.2.2/gems/stripe-ruby-mock-2.5.8/lib/stripe_mock/api/webhooks.rb:34:in `mock_webhook_event'

    ./spec/helpers/stripe_event_helper_spec.rb:27:in `block (3 levels) in <top (required)>'

    ./spec/helpers/stripe_event_helper_spec.rb:49:in `block (4 levels) in <top (required)>'

sylphiae avatar May 27 '23 22:05 sylphiae

Related? #489

ItsElaineTime avatar May 27 '23 22:05 ItsElaineTime