Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 293239

Split out a couple more config translation UI tests

$
0
0

Problem/Motivation

Follow-up from #3389668: Split up ConfigTranslationUiTest - we have a whole test class for date formats with one method, and a huge translation ui test that also has a date formats test. The date format tests could live together in one class.

This requires two very minor changes to the test method that gets moved:

1. Don't try to log in - it already happens in the constructor
2. The big ConfigTranslationUi test uses French, the date formats test uses Spanish and German, switched to testing against German, only about three lines change.

I think this will help test performance marginally because the huge test class is still one of the slowest at well over 6 minutes but it also seems like a better place for it to live.

Also there are two site information tests methods, if we move those out to their own class, I think that will save potentially over a minute.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet


Viewing all articles
Browse latest Browse all 293239

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>