Merge pull request #264 from solidusio/gh-actions

CI: Run tests on GH actions

Author: tvdeyen

.circleci/config.yml

@@ -0,0 +1,63 @@
+name: Test
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+  schedule:
+    - cron: "0 0 * * 4" # every Thursday
+
+concurrency:
+  group: test-${{ github.ref_name }}
+  cancel-in-progress: ${{ github.ref_name != 'main' }}
+
+permissions:
+  contents: read
+
+jobs:
+  rspec:
+    name: Solidus ${{ matrix.solidus-branch }}, Rails ${{ matrix.rails-version }} and Ruby ${{ matrix.ruby-version }} on ${{ matrix.database }}
+    runs-on: ubuntu-24.04
+    strategy:
+      fail-fast: false
+      matrix:
+        rails-version:
+          - "7.2"
+          - "8.0"
+        ruby-version:
+          - "3.1"
+          - "3.4"
+        solidus-branch:
+          - "v4.4"
+          - "v4.5"
+          - "v4.6"
+        database:
+          - "postgresql"
+          - "mysql"
+          - "sqlite"
+        exclude:
+          - solidus-branch: "v4.5"
+            ruby-version: "3.1"
+          - solidus-branch: "v4.6"
+            ruby-version: "3.1"
+          - solidus-branch: "v4.4"
+            rails-version: "8.0"
+    env:
+      TEST_RESULTS_PATH: coverage/coverage.xml
+      CAPYBARA_JAVASCRIPT_DRIVER: selenium_headless
+    steps:
+      - uses: actions/checkout@v4
+      - name: Run extension tests
+        uses: solidusio/test-solidus-extension@main
+        with:
+          database: ${{ matrix.database }}
+          rails-version: ${{ matrix.rails-version }}
+          ruby-version: ${{ matrix.ruby-version }}
+          solidus-branch: ${{ matrix.solidus-branch }}
+      - name: Upload coverage reports to Codecov
+        uses: codecov/codecov-action@v5
+        continue-on-error: true
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          files: ${{ env.TEST_RESULTS_PATH }}

.github/workflows/test.yml

@@ -1,6 +1,6 @@
 # Solidus Auth Devise
 
-[![CircleCI](https://circleci.com/gh/solidusio/solidus_auth_devise.svg?style=shield)](https://circleci.com/gh/solidusio/solidus_auth_devise)
+[![Test](https://github.com/solidusio/solidus_auth_devise/actions/workflows/test.yml/badge.svg)](https://github.com/solidusio/solidus_auth_devise/actions/workflows/test.yml)
 [![codecov](https://codecov.io/gh/solidusio/solidus_auth_devise/branch/main/graph/badge.svg)](https://codecov.io/gh/solidusio/solidus_auth_devise)
 
 Provides authentication services for Solidus, using the Devise gem.

README.md

@@ -9,6 +9,7 @@
       fill_in "Email", with: user.email
       fill_in "Password", with: user.password
       click_button "Login"
+      expect(page).to have_text "Logged in successfully"
 
       click_link "My Account"
       expect(page).to have_text "admin@person.com"
@@ -22,6 +23,7 @@
       fill_in "Password", with: "password"
       fill_in "Password Confirmation", with: "password"
       click_button "Create"
+      expect(page).to have_text "You have signed up successfully"
 
       click_link "My Account"
       expect(page).to have_text "email@person.com"
@@ -43,6 +45,7 @@
       fill_in "Email", with: user.email
       fill_in "Password", with: user.password
       click_button "Login"
+      expect(page).to have_text "Logged in successfully"
 
       click_link "My Account"
       expect(page).to have_text "email@person.com"

spec/features/account_spec.rb

@@ -2,7 +2,7 @@
 
 RSpec.feature "Admin - Sign Out", type: :feature, js: true do
   given!(:user) do
-    create :user, email: "email@person.com"
+    create :admin_user, email: "email@person.com"
   end
 
   background do
@@ -12,6 +12,7 @@
     # Regression test for #1257
     check "Remember me"
     click_button "Login"
+    expect(page).to have_text "Logged in successfully"
   end
 
   scenario "allows a signed in user to logout" do

spec/features/admin/sign_out_spec.rb

@@ -10,6 +10,7 @@
       fill_in "Email", with: user.email
       fill_in "Password", with: user.password
       click_button "Login"
+      expect(page).to have_text "Logged in successfully"
     end
 
     context "admin is restricted from accessing orders" do

spec/features/admin_permissions_spec.rb

@@ -11,6 +11,7 @@
     fill_in "spree_user[email]", with: user.email
     fill_in "spree_user[password]", with: "secret"
     click_button "Login"
+    expect(page).to have_text "Logged in successfully"
 
     visit spree.edit_account_path
   end

spec/features/change_email_spec.rb

@@ -79,11 +79,13 @@
       user = create(:user, email: "email@person.com", password: "password", password_confirmation: "password")
       click_link "RoR Mug"
       click_button "Add To Cart"
+      within("h1") { expect(page).to have_text "Shopping Cart" }
 
       click_link "Login"
       fill_in "Email", with: user.email
       fill_in "Password", with: user.password
       click_button "Login"
+      expect(page).to have_text "Logged in successfully"
       click_link "Cart"
 
       expect(page).to have_text "RoR Mug"
@@ -112,6 +114,7 @@
       create(:user, email: "email@person.com", password: "password", password_confirmation: "password")
       click_link "RoR Mug"
       click_button "Add To Cart"
+      within("h1") { expect(page).to have_text "Shopping Cart" }
 
       click_link "Login"
       click_link "Forgot Password?"
@@ -142,6 +145,7 @@
     scenario "allow a user to register during checkout" do
       click_link "RoR Mug"
       click_button "Add To Cart"
+      within("h1") { expect(page).to have_text "Shopping Cart" }
       click_button "Checkout"
 
       expect(page).to have_text "Registration"

spec/features/checkout_spec.rb

@@ -40,6 +40,7 @@
       fill_in "Email", with: user.email
       fill_in "Password", with: user.password
       click_button "Login"
+      expect(page).to have_text "Logged in successfully"
 
       # Order should have been merged with first session
       click_link "Cart"

spec/features/order_spec.rb

@@ -48,6 +48,7 @@
     fill_in "Email", with: @user.email
     fill_in "Password", with: @user.password
     click_button "Login"
+    expect(page).to have_text "Logged in successfully"
     expect(current_path).to eq "/account"
   end
 end