Opened 18 years ago

Closed 14 years ago

Last modified 14 years ago

#1926 closed defect (worksforme)

[MapServer-msautotest]Some tests run only on Linux

Reported by: nsavard@… Owned by: warmerdam
Priority: high Milestone:
Component: msautotest Version: 4.10
Severity: normal Keywords:
Cc:

Description (last modified by warmerdam)

There are actually five tests in wms_time_pattern.map that run only on Linux
because of a different behaviour of strptime function on Windows.  These tests
are:  wms_time_pattern1a.png, wms_time_pattern1b.png, wms_time_pattern1c.png,
wms_time_pattern1d.png and wms_time_pattern2d.png.

How should msautotest handle this case?

This is a note done by Assefa:
Pattern "%Y%m%d"  : does not work (on windows at least) : problem with strptime
(parsing of the year is done like this (line 408) ret = strtol (buf, &s, 10)
(where buffer =20040101 so the return year is 20040101

Change History (6)

comment:1 by nsavard@…, 18 years ago

See bug 970.

comment:2 by fwarmerdam, 18 years ago

Normand, 

Is there another date format we could use that would work everywhere?
If so, that might be better, and we could add a note in bug 970 to add
a YYYYMMDD casein msautotest once the bug is fixed.


comment:3 by nsavard@…, 18 years ago

YYYYMMDD is just a special case that I wanted to test because it was available
to the user.  But I agree that for now we could remove these five tests. I could
keep these tests in the map file and remove the " RUN_PARMS:" to avoid their
execution.  There are these two existing format already:  "%Y-%m-%d" and
"%Y-%m-%dZ".

I would add a note to bug 970.

comment:4 by warmerdam, 14 years ago

Description: modified (diff)

Normand,did you disable these tests? If so, can we close this ticket?

comment:5 by warmerdam, 14 years ago

Resolution: worksforme
Status: newclosed

I'm seeing all the wms_time_pattern*.map tests succeed on windows and linux, so I assume the problem tests were disabled.

comment:6 by nsavard, 14 years ago

Frank sorry for the delay. I'll check that to be sure but It sounds like a good assumption.

Note: See TracTickets for help on using tickets.