Add tests for quality assurance [#1172124]

We should create a bunch of tests to verify i18nviews.

Comment	File	Size	Author
#7	i18nviews-test-1172124-7.patch	5.46 KB	webflo
#6	i18nviews-test-1172124-6.patch	3.76 KB	webflo
#5	i18nviews-test-1172124-5.patch	3.73 KB	webflo
#4	i18nviews-test-1172124-4.patch	4.89 KB	webflo
#3	i18nviews_test2.patch	8.9 KB	Jose Reyero
#1	i18nviews_test.patch	3.27 KB	Jose Reyero

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Jose Reyero CreditAttribution: Jose Reyero commented 26 December 2011 at 14:14

Priority:	Normal	» Critical
Status:	Active	» Needs work

File	Size
i18nviews_test.patch	3.27 KB

I agree tests are a must to get somwhere close to a good stable release. If you don't mind, upgraded to 'critical' because this would really speed up further development on the module.

This is a first try. It is a test that gets some strings translated but eventually fails. And that's because I haven't been able to figure out how to really display a View from a test. :-(

Anyway I guess having this is better than starting from scratch so I'm posting this not working test.

Comment #2

webflo CreditAttribution: webflo commented 26 December 2011 at 19:40

The tracker default view is outdated and causes this error. see #1386372: Tracker default view needs update. for further explanation.

Comment #3

Jose Reyero CreditAttribution: Jose Reyero commented 27 December 2011 at 11:27

File	Size
i18nviews_test2.patch	8.9 KB

Tried a lot of things and coundn't get the view properly displayed. It seems the only way is to have a view in a module so added a module for testing that creates the view.

This still fails (caching issues I guess) but much better.