entrouvert
/
wcs-olap
16
0
Fork 0
Code Issues Pull Requests Releases Activity

tests: adapt to wcs switch to timestamptz (#86887) #7

Merged
fpeters merged 1 commits from wip/86887-timestamptz into main 2024-02-19 17:38:25 +01:00
Conversation 0 Commits 1 Files Changed 3 +6 -3
3 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
tests/conftest.py
View File

@ -88,6 +88,7 @@ account.store()
import datetime
import random
from quixote import get_publisher
from django.utils.timezone import make_aware
from wcs.categories import Category
from wcs.formdef import FormDef
@ -140,7 +141,7 @@ agent.store()
for i in range(50):
formdata = formdef.data_class()()
formdata.just_created()
formdata.receipt_time = datetime.datetime(2018, random.randrange(1, 13), random.randrange(1, 29)).timetuple()
formdata.receipt_time = make_aware(datetime.datetime(2018, random.randrange(1, 13), random.randrange(1, 29)))
formdata.data = {'1': 'FOO BAR é %d' % i}
if i%4 == 0:
formdata.data['2'] = 'foo'

4
tests/test_wcs.py
View File

@ -24,6 +24,7 @@ import httmock
import pytest
import requests
import utils
from django.utils.timezone import make_aware
from psycopg2.extensions import ISOLATION_LEVEL_AUTOCOMMIT
@ -251,6 +252,7 @@ def test_dimension_stability(wcs, wcs_dir, postgres_db, tmpdir, olap_cmd, caplog
script = """
import datetime
import random
from django.utils.timezone import make_aware
from quixote import get_publisher
from wcs.formdef import FormDef
formdef = FormDef.get_by_urlname('demande')
@ -267,7 +269,7 @@ user = get_publisher().user_class.select()[0]
formdata = formdef.data_class()()
formdata.just_created()
formdata.receipt_time = datetime.datetime(2018, random.randrange(1, 13), random.randrange(1, 29)).timetuple()
formdata.receipt_time = make_aware(datetime.datetime(2018, random.randrange(1, 13), random.randrange(1, 29)))
formdata.data = {'1': 'FOO BAR 1'}
formdata.data['2'] = 'bazouka'
formdata.data['2_display'] = 'bazouka'

2
wcs_olap/wcs_api.py
View File

@ -101,7 +101,7 @@ class Evolution(BaseObject):
def __init__(self, wcs_api, **kwargs):
super().__init__(wcs_api, **kwargs)
self.time = isodate.parse_datetime(self.time)
self.time = isodate.parse_datetime(self.time[:19]) # skip microseconds and timezone
fpeters commented 2024-02-12 18:50:44 +01:00
Review
Copy Link

Cette partie est hors des tests, isodate.parse_datetime() obtient ici une timezone, je raccourci la chaine reçue pour considérer uniquement la portion avec l'heure locale. Pour le receipt_time principal c'est quelque chose qui est dans wcs même, pour les evolution/time ça me va de les laisser complets dans wcs, en misant sur le fait que la seule utilisation était ici.

Cette partie est hors des tests, isodate.parse_datetime() obtient ici une timezone, je raccourci la chaine reçue pour considérer uniquement la portion avec l'heure locale. Pour le receipt_time principal c'est quelque chose qui est dans wcs même, pour les evolution/time ça me va de les laisser complets dans wcs, en misant sur le fait que la seule utilisation était ici.
self.parts = [BaseObject(wcs_api, **part) for part in self.parts or []]
self.who = EvolutionUser(wcs_api, **self.who) if self.who else None