LoginSignup
3
2

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 1 year has passed since last update.

@sanak(Ko Nagase)

redmine.tokyo第23回LT「Redmineプラグインのテスト自動化を頑張っている話」の補足と続き

Last updated at Posted at 2022-12-25

この記事は、Redmine Advent Calendar 2022 の22日目の内容になります。(が、遅れてしまって申し訳ありません...:bow:)

はじめに

2022/11/05(土)のredmine.tokyo 第23回勉強会で「Redmineプラグインのテスト自動化を頑張っている話」というLTをしましたが、時間配分が全くできてませんでした...。
特に後半の説明箇所を飛ばしてしまいましたので、説明できなかった箇所の補足と、続きで少し試してみたことを記載しようと思います。

GitHub Actionsでのテスト自動化設定

redminepdf/9

test.yml ファイルのリンクは以下で、右上の Raw ボタン上で右クリックし、リンク先を別名で保存すればファイルをダウンロードできます。
https://github.com/sanak/redmine_text_blocks/blob/main/.github/workflows/test.yml

redminepdf/10
GitHubコードリンク

冒頭の env:↓ PLUGIN_NAME の箇所は、ワークフロー全体での環境変数で、ここではプラグインのフォルダ名を指定します。
環境変数 - GitHub Docs

また、 on 配下はワークフローのトリガーを指定している箇所で、指定したブランチ(main と master)へのプッシュ・プルリクエスト時のみ、ワークフローが起動されることを意味しています。
ワークフローのトリガー - GitHub Docs
ワークフローのトリガーには、他にも スケジュールされた時刻(cron形式) や 手動トリガー も可能なので、状況に応じて使い分けることができるかもしれません。

redminepdf/11
GitHubリンク

テストのマトリックス(組み合わせ)を指定している箇所です。
ジョブにマトリックスを使用する - GitHub Docs

Redmineのみで、RedMicaのサポートが不要であれば、 redmine: [...] の箇所は redmine_version: [4.2-stable, 5.0-stable, master] などと変更可能と思いますが、RedMicaはRedmineにとっても両輪のような形で引き続き重要なように思います。
なお、RedMicaについては、先月末に2.2がリリースされましたので、
RedMica 2.2 バージョンアップのお知らせ |プロジェクト管理・タスク管理はMy Redmine
今だと以下の行を追加することも可能のはずです。

{ repo: redmica, version: stable-2.2 },

Rubyも12/25(日)のクリスマスに3.2がリリースされましたので、Redmine本家側での対応を待ってから、以下のように ruby_version の配列末尾に追加することでテスト可能と思います。
Feature #38099: Ruby 3.2 support - Redmine

ruby_version: ['2.6', '2.7', '3.0', '3.1', '3.2']

redminepdf/12

こちらでは、先ほどのマトリックスから除外する組み合わせを exclude 配下で指定しています。
ジョブにマトリックスを使用する - マトリックス構成の除外 - GitHub Docs

例えば、先ほどのRuby3.2が追加された状態の場合、Ruby3.2のビルドがサポートされるのはRedmine・RedMicaの各masterブランチになると思われますので、その他を除外する場合は以下を exclude 配下に追加する形になると思います。

        exclude:
          :
          - redmine: { repo: redmine, version: 4.2-stable }
            ruby_version: '3.2'
          - redmine: { repo: redmine, version: 5.0-stable }
            ruby_version: '3.2'
          - redmine: { repo: redmica, version: stable-2.0 }
            ruby_version: '3.2'
          - redmine: { repo: redmica, version: stable-2.1 }
            ruby_version: '3.2'
          - redmine: { repo: redmica, version: stable-2.2 }
            ruby_version: '3.2'

redminepdf/13

赤字箇所は、プラグインのアップストリームにテスト目的のプルリクエストを投げないようにしましょうという注意書きです。
(GitHubのデフォルトのターゲットリポジトリ・ブランチが、自分のフォークリポジトリ・ブランチになっていると、誤りがなくて助かるところではあります...:sweat_smile:)

redminepdf/14

テスト自動実行の確認手順を記載しています。私のリポジトリでのリンクも貼っておきます。

redminepdf/15

こちらも当時のリンクが残ってましたので、貼っておきます。

ちなみに、上記のエラーの原因は、確認の結果、Redmine本体側の Gemfile にあることが分かり、修正パッチを作成して取り込んで頂きました。:smile:

Patch #37883: Limit mocha version to < 2.0.0 when Ruby version is < 2.7 to avoid test error - Redmine

redminepdf/16

こちらもGitHubのリンクを貼っておきます。

redminepdf/17

  • Redmineプラグインでテストを記述してなくても、DBマイグレーションやZeitwerkチェックとしてだけでも有効

DBマイグレーションに関しては、私の場合PostgreSQLは日常的に利用していても、MySQLはほぼ使ってないので、その点で役に立ちました。

また、Zeitwerkチェック(rake zeitwerk:check)も、普段から意識して実行していないと忘れがちになるので、注意が必要です。

  • システムテスト関連のコメントアウト箇所を外せば、システムテスト(E2E、ブラウザテスト)の実行も可能 (ただし、2〜3倍時間がかかります)

こちらは、以下の37行目〜43行目のコメントアウト箇所を外せば実行可能です。
(ただし、時間がかかるので、組み合わせを1つのみ(Redmineのmasterブランチ、Ruby3.1、MySQL)に絞る形にしています。)
https://github.com/sanak/redmine_text_blocks/blob/main/.github/workflows/test.yml#L37-L43

        # # System test takes 2~3 times longer, so limit to specific matrix combinations
        # # See: https://docs.github.com/en/actions/using-jobs/using-a-matrix-for-your-jobs#expanding-or-adding-matrix-configurations
        # include:
        #   - system_test: true
        #     redmine: { repo: redmine, version: master }
        #     ruby_version: '3.1'
        #     db: mysql
  • Redmine本体のコメントアウト箇所を外せば、プラグインがRedmine本体の挙動に影響を与えないことの確認も可能 (こちらはさらに時間がかかります)

こちらは、最後の方の185行目〜198行目のコメントアウト箇所を外せば実行可能です。
https://github.com/sanak/redmine_text_blocks/blob/main/.github/workflows/test.yml#L185-L198

      # - name: Run core tests
      #   env:
      #     RAILS_ENV: test
      #     PARALLEL_WORKERS: 1
      #   working-directory: redmine
      #   run: bundle exec rake test

      # - name: Run core system tests
      #   if: matrix.system_test == true
      #   env:
      #     RAILS_ENV: test
      #     GOOGLE_CHROME_OPTS_ARGS: "headless,disable-gpu,no-sandbox,disable-dev-shm-usage"
      #   working-directory: redmine
      #   run: bundle exec rake test:system

こちらも、テスト実行に時間がかかる点はご注意ください。

redminepdf/18

  • ジョブが並列実行されるGitHub Actionsでも、初期状態のtest.yml(組み合わせ数:60)で7〜11分ほどかかるので、テスト実行時間の短縮化を図る
    • GitHub Actions自体の機能が年々向上してきているので、そちらに期待したいというのもあったり...

現状、 test.yml の可読性を上げ、重複を避けたいという思いから、PostgreSQLのテスト時にもMySQLサービスも同時に立ち上がる作りになっていて、そのこともテスト実行に時間がかかる一因となっているかもしれません。

redmine.tokyo第23回の後、Redmine本体側のGitHub Actionsテストでは、複合アクションを利用することで、
複合アクションを作成する - GitHub Docs

重複箇所を別のアクションとして切り出し、ワークフロー側ではマトリックス定義だけに絞ることも可能そうに思われましたが、様々なケースが想定されるRedmineプラグインの場合のGitHub Actionsの最適解がどのような形になるかはまだ見いだせてません...。

システムテスト(E2E、ブラウザテスト)はChromeのインストールなどでさらに時間がかかるので、良い方法を検討する

システムテストに関しては、まだプラグインに組み込めてないのと、Redmine本体側の方でもランダムに失敗していると思われる箇所がありますので、インストールもさることながら、引き続き調査・検討を行っていく予定です。

redminepdf/20

LT発表時に流してしまいましたが、発表にあった test.yml は上記にある様々な先行事例を参考に組み合わせたものになります。
Redmineパッチ会の方々や、Redmineプラグイン・関連プロジェクト開発者の方々には改めてお礼申し上げます。

おわりに

本年もコミュニティの方には大変お世話になりました。
redmine.tokyo第23回も当日オフライン参加できて本当に良かったです。

来年も引き続き、よろしくお願いいたします！

また、Redmineパッチ会も月1回ペースで開催しておりますので、ご興味ある方おられましたら、是非ご参加ください。

それでは、良いお年を！

3
2
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
Sign upLogin
3
2

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?