Bug 968255 - MIMEUtilTest fails if performed in wrong timezone (east to UTC)
Summary: MIMEUtilTest fails if performed in wrong timezone (east to UTC)
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: libdap
Version: rawhide
Hardware: x86_64
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: Orion Poplawski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-05-29 10:18 UTC by Eugene Syromyatnikov
Modified: 2013-05-29 21:31 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-05-29 21:31:51 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch for the TZ-sensitive test (2.17 KB, patch)
2013-05-29 10:19 UTC, Eugene Syromyatnikov
no flags Details | Diff

Description Eugene Syromyatnikov 2013-05-29 10:18:04 UTC
Description of problem:
Tests in unit-tests/MIMEUtilTest.cc (namely, set_mime_text_test() and rfc822_date_test()) fail if performed in eastern (related to UTC) timezones since mktime()

Version-Release number of selected component (if applicable):
Tested with 3.11.0-2, but quick look at 3.11.3-3 reveals no differences in this part of code.

How reproducible:
Always.

Steps to Reproduce:
1. Set tomezone to UTC+smth (Europe/Moscow is fine).
2. Try to build package.

Actual results:
Tests set_mime_text_test() and rfc822_date_test() fail which fails the build.

Expected results:
Tests set_mime_text_test() and rfc822_date_test() should pass.

Additional info:
Some quick patch (against 3.11.0, but should apply also against 3.11.3 since difference between those two versions of this file is negligible) which should likely solve the problem.

Comment 1 Eugene Syromyatnikov 2013-05-29 10:19:10 UTC
Created attachment 754280 [details]
Patch for the TZ-sensitive test

Comment 2 Orion Poplawski 2013-05-29 21:31:51 UTC
I would suggest sending this to support, they are pretty responsive.  I'm afraid I'm not really interested in maintaining a patch that just fixes a testcase.


Note You need to log in before you can comment on or make changes to this bug.