Actions
Patch #31941
closedThemesTest may fail if a third-party theme with a favicon is installed
Description
Steps to reproduce:
1. Add a theme with the name 'xyz'.
cp -r public/themes/alternate public/themes/xyz mkdir public/themes/xyz/favicon cp public/favicon.ico public/themes/xyz/favicon/
2. Run the test.
$ bin/rails test test/integration/lib/redmine/themes_test.rb Run options: --seed 6694 # Running: .F Failure: ThemesTest#test_use_only_one_theme_favicon_if_theme_provides_many [/Users/maeda/redmines/redmine-trunk/test/integration/lib/redmine/themes_test.rb:85]: Expected at least 1 element matching "link[rel="shortcut icon"][href^="/themes/xyz-theme/favicon/b.ico"]", found 0.. Expected 0 to be >= 1. bin/rails test test/integration/lib/redmine/themes_test.rb:79 .F Failure: ThemesTest#test_use_default_favicon_if_theme_provides_none [/Users/maeda/redmines/redmine-trunk/test/integration/lib/redmine/themes_test.rb:65]: Expected at least 1 element matching "link[rel="shortcut icon"][href^="/favicon.ico"]", found 0.. Expected 0 to be >= 1. bin/rails test test/integration/lib/redmine/themes_test.rb:61 F Failure: ThemesTest#test_use_theme_favicon_if_theme_provides_one [/Users/maeda/redmines/redmine-trunk/test/integration/lib/redmine/themes_test.rb:74]: Expected at least 1 element matching "link[rel="shortcut icon"][href^="/themes/xyz-theme/favicon/a.ico"]", found 0.. Expected 0 to be >= 1. bin/rails test test/integration/lib/redmine/themes_test.rb:68 F Failure: ThemesTest#test_with_sub_uri [/Users/maeda/redmines/redmine-trunk/test/integration/lib/redmine/themes_test.rb:100]: Expected at least 1 element matching "link[rel="shortcut icon"][href^="/foo/themes/xyz-theme/favicon/a.ico"]", found 0.. Expected 0 to be >= 1. bin/rails test test/integration/lib/redmine/themes_test.rb:91 . Finished in 2.641204s, 2.6503 runs/s, 6.4365 assertions/s. 7 runs, 17 assertions, 4 failures, 0 errors, 0 skips
Files
Related issues
Updated by Go MAEDA over 5 years ago
- File themes_test_fix.patch themes_test_fix.patch added
- Target version set to Candidate for next major release
Here is a patch to fix this issue.
Updated by Go MAEDA over 5 years ago
- Related to Defect #25118: ThemesTest#test_without_theme_js may fail if third-party theme is installed added
Updated by Go MAEDA over 5 years ago
- File 0001-ThemesTest-fails-if-a-third-party-theme-with-a-favic.patch 0001-ThemesTest-fails-if-a-third-party-theme-with-a-favic.patch added
Removed a hard-coded theme name from the previous patch.
Updated by Go MAEDA over 5 years ago
- Subject changed from ThemesTest may fail if a third-party theme is installed to ThemesTest may fail if a third-party theme with a favicon is installed
- Target version changed from Candidate for next major release to 4.1.0
Setting the target version to 4.1.0.
Updated by Go MAEDA over 5 years ago
- Tracker changed from Defect to Patch
- Status changed from New to Closed
- Assignee set to Go MAEDA
Committed.
Actions