Maikel Linke
2cf8a6dcb9
Skip haml-up spec with newer versions
2024-02-22 15:01:14 +11:00
Maikel Linke
4516d90ede
Add script to upgrade HAML syntax
...
[skip-ci]
2024-02-22 15:01:13 +11:00
Maikel Linke
008e384f14
Simplify oidc_user factory
...
Now you can build in memory only as well.
2024-02-22 10:22:00 +11:00
Maikel Linke
4d8bb25f86
Allow enterprise users to disconnect their OIDC account
2024-02-22 10:21:58 +11:00
Maikel Linke
0813f43b49
Better wording for OIDC connection
2024-02-22 10:15:07 +11:00
Maikel Linke
6d9c5a9c66
Allow user to get new OIDC refresh token
...
The refresh token is usually valid for a year but it can be revoked at
any time. When we try to use it and it's expired, we should remove it
from the account record and notify the user. They can then refresh the
authorisation.
2024-02-22 10:15:07 +11:00
Maikel Linke
a89b22e397
Allow user to disconnect OIDC account
...
This makes testing much easier. But probably also good for users to
revoke any access via OIDC apps. It also enables users to then connect
to a different account, or just renew the current connection.
2024-02-22 10:15:07 +11:00
Maikel Linke
4f3ae4f2a4
Spec OIDC setup flow
2024-02-22 10:15:07 +11:00
Maikel Linke
07a8617143
Store OIDC account data in new model
2024-02-22 10:15:07 +11:00
Maikel Linke
6c0d15b6f9
Migrate existing OIDC account data
2024-02-22 10:15:07 +11:00
Maikel Linke
b4ee24368c
Add model for OIDC accounts
...
The provider name and uid are currently stored on the user model but
it's better to move them to their own table. They are only needed in
certain situations, only some users have an account and we are now
storing a lot more.
2024-02-22 10:15:07 +11:00
Maikel Linke
4d680e5fd1
Use recorded auth hash including all tokens
...
We want to store the access and refresh token as well.
2024-02-22 10:15:07 +11:00
Maikel Linke
60dc710760
Refactor OIDC callback spec
...
* Clarify that it's a request spec, not testing a controller directly.
* Use `before` block to avoid side effects changing config at load time.
* Better name the test action as request instead of plain "subject".
* Move assignments into `before` block instead of variable.
2024-02-22 10:15:07 +11:00
Maikel Linke
26f4ebc8f9
Remove unnecessary test code
2024-02-22 10:15:07 +11:00
David Cook
cbb1e41bbc
Merge pull request #12174 from mkllnk/user-factory
...
Simplify user factory around admin role
2024-02-22 10:08:28 +11:00
filipefurtad0
89848efd23
Replaces stubs for VCR calls
...
Remove calling StripeStubs helper
2024-02-21 10:48:53 +00:00
filipefurtad0
b91719d63c
Removes reference to STRIPE_ACCOUNT
...
THe connected account is created a few lines below, and this env variable is not needed
2024-02-20 15:15:54 +00:00
Maikel Linke
e423015f0a
Treat enterprises as normal user attribute in factory
2024-02-20 14:00:19 +11:00
Maikel Linke
af085c223d
Simplify user factory around admin role
...
The original Spree user factory was adding the admin role by default.
Since we don't do that anymore, we don't need to remove it for normal
users. We also need to add it only once to admin users.
2024-02-20 12:58:04 +11:00
Konrad
2477a63f36
Merge pull request #12103 from basilawwad/selecting-OC-should-redirect-to-shop
...
Modify default hash tabs controller to redirect users to shopping panel when an OC is selected.
2024-02-19 21:51:19 +01:00
Gaetan Craig-Riou
cb83ad688e
Remove old vcr stripe cassettes
2024-02-19 15:44:20 +11:00
Gaetan Craig-Riou
02019f7cae
Re recorde vcr cassette after rebase
2024-02-19 11:01:20 +11:00
Gaetan Craig-Riou
848767d50f
Per review, clean up test set up
2024-02-19 10:58:31 +11:00
Gaetan Craig-Riou
89236f4c2f
Per review, create a Stripe account
...
It's better to avoid hard coded stripe entities
2024-02-19 10:58:31 +11:00
Gaetan Craig-Riou
ef1d2fdb4d
Clean up cassettes
...
* remove old cassettes left over from a rebase
* re recorded cassettes with the lastest Stripe version
2024-02-19 10:58:31 +11:00
Gaetan Craig-Riou
657f71c357
Further refactor payment specs
...
- remove any stub_request
- use intance_double of ActiveMerchant::Billing::Response
2024-02-19 10:58:31 +11:00
Gaetan Craig-Riou
8832f2fef6
Refactor #credit spec
...
Model spec shouldn't know about the undelying call to stripe. Replaced
request stubs by payment method stubs.
Consolidate #credit spec in one `describe` block
2024-02-19 10:58:31 +11:00
Gaetan Craig-Riou
0af9ccd17a
Add test for #credit
...
it uses ActiveMerchant::Billing::StripePaymentIntentsGateway#refund,
we covering the last scenario that previously used the decorator.
2024-02-19 10:58:31 +11:00
Gaetan Craig-Riou
549610bc35
Finally remove StripePaymentIntentsGateway decorator
...
Add spec for Spree::Gateway::StripeSCA#void
It also partially fix 11670, refund of Stripe payment should now work
for complete order
2024-02-19 10:58:31 +11:00
Gaetan Craig-Riou
8a0c32eec0
Improve Stripe::ProfileStorer specs
...
* Use `with_stripe_setup` helper
* add the request failing scenario
2024-02-19 10:53:53 +11:00
Gaetan Craig-Riou
62fbaa8a6e
Update Stripe payment intents decorator
...
- copied the relevant code from the new Active Merchant version
- Add spec to cover the scenario of saving a card when paying by card
- Fix Stripe stub.
I used stripe stubs for the new scenario because storing a card on
stripe depends on some client side interaction with Stripe. We can't
capture that with VCR.
2024-02-19 10:53:53 +11:00
Maikel
4bb1afcc8f
Merge pull request #12166 from openfoodfoundation/dependabot/bundler/stripe-10.9.0
...
chore(deps): bump stripe from 10.8.0 to 10.9.0
2024-02-19 10:22:53 +11:00
Maikel Linke
7eccce138b
Update Stripe API recordings for new version
2024-02-19 10:02:05 +11:00
Rachel Arnould
39b83eae19
Merge pull request #12133 from dacook/buu/new-variant-11066
...
[BUU] Create a new variant inline
2024-02-16 14:32:17 +01:00
Rachel Arnould
a3af15dbae
Merge pull request #12162 from isidzukuri/12003_improve_error_messageg_when_no_products_are_changed
...
Improve error message by not displaying the first part of the second …
2024-02-16 11:24:52 +01:00
Filipe
49ebcc23b3
Merge pull request #12159 from mkllnk/capybara-wait-time
...
Reduce Capybara wait time during development
2024-02-16 09:37:28 +00:00
basilawwad
08111f9a2c
added shop target and activateshoppanel function
...
Remove Pending Message
Update shopping_spec.rb
Update shopping_spec.rb
2024-02-16 09:27:21 +03:00
David Cook
7164cbd0bd
Don't validate numericality if blank
...
This avoids unnecessary second message when left blank:
> can't be blank
> is not a number
Ok this is a little confusing. Why is there a separate presence check above, and why is it only for measurable units, when we still require a number for _all_ units? Because, for 'items', we allow a blank value then auto-set it to 1.
I don't know if it's really necessary, but that's how it currently works...
2024-02-16 08:51:53 +11:00
David Cook
915a5ce66b
Listen for changes on added fields
...
Finally, we can add new variants!
Requires updated stimulus-rails-nested-form
2024-02-16 08:51:49 +11:00
isidzukuri
ae9229ec7a
Merge branch 'master' into 12003_improve_error_messageg_when_no_products_are_changed
2024-02-15 09:36:53 +02:00
isidzukuri
0c02b5d9bd
fix spec
2024-02-15 09:24:02 +02:00
isidzukuri
b9eea88f5c
more precise spec for error message wording
2024-02-15 09:20:32 +02:00
isidzukuri
c0e820e447
Improve error message by not displaying the first part of the second sentence if no changes were counted
2024-02-15 09:14:52 +02:00
David Cook
1df0eca4d1
Click button to add form
...
Now we can fill out the form, but in most cases we can't submit it because it's not marked as 'changed'.
2024-02-15 17:32:08 +11:00
David Cook
5253aeab17
Add unit_value so we can create new variants for producuts by weight/volume
2024-02-15 17:31:50 +11:00
David Cook
eff92b076f
Add new variant form
...
And with the benefit of time travel, a feature spec from the future which will show us where we're heading.
2024-02-15 17:20:38 +11:00
David Cook
831d786d44
Fix existing specs
...
Gotta wait for server response before making expectations. This reveals that one test was incomplete, oops.
2024-02-15 17:04:14 +11:00
Maikel Linke
78bfd47ad3
Add rake task to reset queued jobs
...
Please note that the spec is using Redis as Sidekiq does. This revealed
that Sidekiq wasn't set up for jobs in the test environment properly.
2024-02-15 16:51:10 +11:00
Maikel
c15bf61099
Merge pull request #12156 from isidzukuri/issue_12062_fix_flacky_tax_rate_finder_spec
...
Improve random name generation in factories related to tax_rate_factory
2024-02-15 15:43:05 +11:00
David Cook
ec0e8b80eb
Add specs for creating variants
...
This already works, but we want to make sure it stays that way!
2024-02-15 14:57:13 +11:00
