diff --git a/app/assets/javascripts/admin/addon/components/plugin-commit-hash.hbs b/app/assets/javascripts/admin/addon/components/plugin-commit-hash.hbs index 441febf757f..f9dabfbfba6 100644 --- a/app/assets/javascripts/admin/addon/components/plugin-commit-hash.hbs +++ b/app/assets/javascripts/admin/addon/components/plugin-commit-hash.hbs @@ -1,6 +1,8 @@ {{#if this.commitHash}} {{this.shortCommitHash}} diff --git a/app/assets/javascripts/discourse/tests/acceptance/admin-plugins-test.js b/app/assets/javascripts/discourse/tests/acceptance/admin-plugins-test.js index 179425999e5..3a326ce86d3 100644 --- a/app/assets/javascripts/discourse/tests/acceptance/admin-plugins-test.js +++ b/app/assets/javascripts/discourse/tests/acceptance/admin-plugins-test.js @@ -28,6 +28,9 @@ acceptance("Admin - Plugins", function (needs) { enabled_setting: "testplugin_enabled", has_settings: true, is_official: true, + commit_hash: "1234567890abcdef", + commit_url: + "https://github.com/username/some-test-plugin/commit/1234567890abcdef", }, ], }) @@ -47,5 +50,11 @@ acceptance("Admin - Plugins", function (needs) { exists(".admin-plugins .admin-container .alert-error"), "displays an error for unknown routes" ); + + assert.strictEqual( + table.querySelector("tr .version a.commit-hash").href, + "https://github.com/username/some-test-plugin/commit/1234567890abcdef", + "displays a commit hash with a link to commit url" + ); }); });