Just document the locale thing for now
commitc7fde7e7e0e7d3ab10b887671cb77f2e3270d527
authorAlbert Astals <albert.astals@canonical.com>
dateFri 10 Jan 2014, 09:47 (13 years ago)
parent29a72ae
CODING+4
@@ -166,3 +166,7 @@ $ make tryDragHandle
166166NB: Manual test or interation doesn't make sense for all components or classes,
167167particularly the non-graphical ones. In such cases no "tryFoo" make target is
168168provided.
169+
170+NB: The tests expect the locale to be the C locale, they may fail if run
171+under a different locale, if you get failures others can not reproduce
172+prepend 'LC_ALL=C' to the 'make test' command
tests/plugins/Utils/timeformattertest.cpp−11
@@ -21,15 +21,6 @@
2121#include <QtTest>
2222#include <QDebug>
2323
24−class LCALLSetter
25−{
26−public:
27− LCALLSetter()
28− {
29− setenv("LC_ALL", "C", 1);
30− }
31−};
32−
3324class TimeFormatterTest : public QObject
3425{
3526 Q_OBJECT
@@ -72,8 +63,6 @@ private Q_SLOTS:
7263 }
7364};
7465
75−static LCALLSetter setter;
76−
7766QTEST_GUILESS_MAIN(TimeFormatterTest)
7867
7968#include "timeformattertest.moc"