psyinfra / HTCAnalyze

MIT License
0 stars 3 forks source link

tests failing #121

Closed aqw closed 3 years ago

aqw commented 3 years ago

I think CI wasn't working for awhile due to the Travis switchover.

I recently pushed changes to move from Travis to GitHub Actions. The machinery seems to work, but a single test fails.

>       assert ram_history_dict == {
            'Dates': [datetime.datetime(2020, 7, 11, 20, 40, 3),
                      datetime.datetime(2020, 7, 11, 20, 45, 4)],
            'Image size updates': [448, 1052936],
            'Memory usages': [1, 922],
            'Resident Set Sizes': [448, 943244]}
E       AssertionError: assert {'Dates': [da...[448, 943244]} == {'Dates': [da...[448, 943244]}
E         Omitting 3 identical items, use -vv to show
E         Differing items:
E         {'Dates': [datetime.datetime(2021, 7, 11, 20, 40, 3), datetime.datetime(2021, 7, 11, 20, 45, 4)]} != {'Dates': [datetime.datetime(2020, 7, 11, 20, 40, 3), datetime.datetime(2020, 7, 11, 20, 45, 4)]}
E         Use -v to get the full diff

tests/htcanalyze_test.py:119: AssertionError
=============================== warnings summary ===============================
../../../../../opt/hostedtoolcache/Python/3.6.13/x64/lib/python3.6/site-packages/htcondor/__init__.py:48
  /opt/hostedtoolcache/Python/3.6.13/x64/lib/python3.6/site-packages/htcondor/__init__.py:48: UserWarning: Neither the environment variable CONDOR_CONFIG, /etc/condor/, /usr/local/etc/, nor ~condor/ contain a condor_config source. Therefore, we are using a null condor_config.
    _warnings.warn(message)

-- Docs: https://docs.pytest.org/en/stable/warnings.html

---------- coverage: platform linux, python 3.6.13-final-0 -----------
Coverage XML written to file coverage.xml

=========================== short test summary info ============================
FAILED tests/htcanalyze_test.py::test_log_to_dict - AssertionError: assert {'...
=================== 1 failed, 32 passed, 1 warning in 36.75s ===================
Error: Process completed with exit code 1.
mathisloevenich commented 3 years ago

I will have a look at this as soon as I have time to do so.

aqw commented 3 years ago

It's mostly a note so that we don't forget. No worries. Keep your focus on your thesis. :-)