diff options
author | Marius Balteanu <marius.balteanu@zitec.com> | 2025-03-10 22:29:02 +0000 |
---|---|---|
committer | Marius Balteanu <marius.balteanu@zitec.com> | 2025-03-10 22:29:02 +0000 |
commit | f2e02c60cc9012cebeebc4bc71ae9ce738a4b937 (patch) | |
tree | 5dc7878b312812d434cf10937dec65a40836cb8c | |
parent | bddc7b8c84d2aa4a84c5de5b683bdaa3fc9983de (diff) | |
download | redmine-f2e02c60cc9012cebeebc4bc71ae9ce738a4b937.tar.gz redmine-f2e02c60cc9012cebeebc4bc71ae9ce738a4b937.zip |
Enable GitHub Actions for testing and linting on 6.0-stable branch (#42135).
Patch by Katsuya HIDAKA (user:hidakatsuya).
git-svn-id: https://svn.redmine.org/redmine/branches/6.0-stable@23536 e93f8b46-1217-0410-a6f0-8f06a7374b81
-rw-r--r-- | .github/workflows/linters.yml | 55 | ||||
-rw-r--r-- | .github/workflows/rubyonrails.yml | 17 | ||||
-rw-r--r-- | .github/workflows/tests.yml | 112 | ||||
-rw-r--r-- | .stylelintrc | 5 | ||||
-rw-r--r-- | Gemfile | 1 | ||||
-rw-r--r-- | test/unit/repository_bazaar_test.rb | 25 |
6 files changed, 197 insertions, 18 deletions
diff --git a/.github/workflows/linters.yml b/.github/workflows/linters.yml new file mode 100644 index 000000000..54ffd08df --- /dev/null +++ b/.github/workflows/linters.yml @@ -0,0 +1,55 @@ +name: Lint + +on: + push: + +jobs: + lint: + runs-on: ubuntu-latest + + steps: + - name: Checkout code + uses: actions/checkout@v4 + + - name: Set up Ruby + uses: ruby/setup-ruby@v1 + with: + ruby-version: 3.2 + bundler-cache: true + + - name: Lint code for consistent style + run: bundle exec rubocop --parallel + + stylelint: + runs-on: ubuntu-latest + + steps: + - name: Checkout code + uses: actions/checkout@v4 + + - name: Set up Node.js + uses: actions/setup-node@v2 + with: + node-version: '20' + + - name: Install dependencies + run: yarn install + + - name: Lint CSS and SCSS files + run: npx stylelint "app/assets/stylesheets/**/*.css" + + bundle-audit: + runs-on: ubuntu-latest + + steps: + - name: Checkout code + uses: actions/checkout@v4 + + - name: Set up Ruby + uses: ruby/setup-ruby@v1 + with: + ruby-version: '3.2' + bundler-cache: true + + - name: Run bundle-audit + run: bundle exec bundle audit check --update diff --git a/.github/workflows/rubyonrails.yml b/.github/workflows/rubyonrails.yml index d147851b2..e69de29bb 100644 --- a/.github/workflows/rubyonrails.yml +++ b/.github/workflows/rubyonrails.yml @@ -1,17 +0,0 @@ -name: "Ruby on Rails CI" -on: - push: - -jobs: - lint: - runs-on: ubuntu-latest - steps: - - name: Checkout code - uses: actions/checkout@v4 - - name: Install Ruby and gems - uses: ruby/setup-ruby@v1 - with: - ruby-version: 3.2 - bundler-cache: true - - name: Lint Ruby files - run: bundle exec rubocop --parallel diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml new file mode 100644 index 000000000..12b507b0e --- /dev/null +++ b/.github/workflows/tests.yml @@ -0,0 +1,112 @@ +name: Tests + +on: + push: + +jobs: + tests: + name: test ${{matrix.db}} ruby-${{ matrix.ruby }} + runs-on: ubuntu-latest + + strategy: + matrix: + ruby: ['3.1', '3.2', '3.3'] + db: ['postgresql', 'mysql2', 'sqlite3'] + fail-fast: false + + services: + postgres: + image: postgres:13 + env: + POSTGRES_DB: redmine_test + POSTGRES_USER: root + POSTGRES_PASSWORD: root + ports: + - 5432:5432 + options: >- + --health-cmd pg_isready + --health-interval 10s + --health-timeout 5s + --health-retries 5 + + mysql: + image: mysql:8.0 + env: + MYSQL_DATABASE: redmine_test + MYSQL_ROOT_PASSWORD: 'root' + ports: + - 3306:3306 + options: >- + --health-cmd="mysqladmin ping" + --health-interval=10s + --health-timeout=5s + --health-retries=3 + + steps: + - name: Checkout code + uses: actions/checkout@v4 + + - name: Install dependencies and configure environment + run: | + sudo apt-get update + sudo apt-get install --yes --quiet ghostscript gsfonts locales bzr cvs + sudo locale-gen en_US # for bazaar non ascii test + + - name: Allow imagemagick to read PDF files + run: | + echo '<policymap>' > policy.xml + echo '<policy domain="coder" rights="read | write" pattern="PDF" />' >> policy.xml + echo '</policymap>' >> policy.xml + sudo rm /etc/ImageMagick-6/policy.xml + sudo mv policy.xml /etc/ImageMagick-6/policy.xml + + - if: ${{ matrix.db == 'sqlite3' }} + name: Prepare test database for sqlite3 + run: | + cat > config/database.yml <<EOF + test: + adapter: sqlite3 + database: db/test.sqlite3 + EOF + + - if: ${{ matrix.db == 'mysql2' || matrix.db == 'postgresql' }} + name: Prepare test database for mysql2 and postgresql + run: | + cat > config/database.yml <<EOF + test: + adapter: ${{ matrix.db }} + database: redmine_test + username: root + password: root + host: 127.0.0.1 + EOF + + - name: Install Ruby and gems + uses: ruby/setup-ruby@v1 + with: + ruby-version: ${{ matrix.ruby }} + bundler-cache: true + + - name: Run prepare test environment + env: + RAILS_ENV: test + SCMS: subversion,git,git_utf8,filesystem,bazaar,cvs + run: | + bundle exec rake ci:about + bundle exec rake ci:setup + bundle exec rake db:environment:set + + - name: Run tests + run: | + bin/rails test + + - name: Run bazaar non ascii test + env: + LANG: en_US.ISO8859-1 + LC_ALL: en_US.ISO8859-1 + run: | + bin/rails test test/unit/repository_bazaar_test.rb + + - name: Run autoload test + run: | + bin/rails test:autoload diff --git a/.stylelintrc b/.stylelintrc index ea5c7df0f..6ee7b505e 100644 --- a/.stylelintrc +++ b/.stylelintrc @@ -22,5 +22,8 @@ "no-duplicate-selectors": true, "no-empty-source": true, "no-invalid-double-slash-comments": true, - } + }, + "ignoreFiles": [ + "app/assets/stylesheets/jquery/*.css" + ] } @@ -113,6 +113,7 @@ group :test do gem 'rubocop', '~> 1.68.0', require: false gem 'rubocop-performance', '~> 1.22.0', require: false gem 'rubocop-rails', '~> 2.27.0', require: false + gem 'bundle-audit', require: false end local_gemfile = File.join(File.dirname(__FILE__), "Gemfile.local") diff --git a/test/unit/repository_bazaar_test.rb b/test/unit/repository_bazaar_test.rb index 5af9e7f13..23f3ce48f 100644 --- a/test/unit/repository_bazaar_test.rb +++ b/test/unit/repository_bazaar_test.rb @@ -165,7 +165,22 @@ class RepositoryBazaarTest < ActiveSupport::TestCase end if File.directory?(REPOSITORY_PATH_NON_ASCII) && RUN_LATIN1_OUTPUT_TEST + # https://www.redmine.org/issues/42024 + def skip_bzr_failure_on_ubuntu24 + return unless File.exist?('/etc/os-release') + + os_release = File.read('/etc/os-release') + name = os_release[/^NAME="(.+?)"$/, 1] + version = os_release[/^VERSION_ID="(.+?)"$/, 1] + + if name == 'Ubuntu' && version == '24.04' + skip 'bzr command fails on Ubuntu 24.04, causing this test to fail' + end + end + def test_cat_latin1_path + skip_bzr_failure_on_ubuntu24 + latin1_repo = create_latin1_repo buf = latin1_repo.cat( @@ -186,6 +201,8 @@ class RepositoryBazaarTest < ActiveSupport::TestCase end def test_annotate_latin1_path + skip_bzr_failure_on_ubuntu24 + latin1_repo = create_latin1_repo ann1 = latin1_repo.annotate( @@ -206,6 +223,8 @@ class RepositoryBazaarTest < ActiveSupport::TestCase end def test_diff_latin1_path + skip_bzr_failure_on_ubuntu24 + latin1_repo = create_latin1_repo diff1 = latin1_repo.diff( @@ -217,6 +236,8 @@ class RepositoryBazaarTest < ActiveSupport::TestCase end def test_entries_latin1_path + skip_bzr_failure_on_ubuntu24 + latin1_repo = create_latin1_repo entries = latin1_repo.entries("test-#{CHAR_1_UTF8_HEX}-dir", 2) assert_kind_of Redmine::Scm::Adapters::Entries, entries @@ -227,6 +248,8 @@ class RepositoryBazaarTest < ActiveSupport::TestCase end def test_entry_latin1_path + skip_bzr_failure_on_ubuntu24 + latin1_repo = create_latin1_repo ["test-#{CHAR_1_UTF8_HEX}-dir", "/test-#{CHAR_1_UTF8_HEX}-dir", @@ -245,6 +268,8 @@ class RepositoryBazaarTest < ActiveSupport::TestCase end def test_changeset_latin1_path + skip_bzr_failure_on_ubuntu24 + latin1_repo = create_latin1_repo assert_equal 0, latin1_repo.changesets.count latin1_repo.fetch_changesets |