Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test on Django 5.0 #3269

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Test on Django 5.0 #3269

wants to merge 7 commits into from

Conversation

hmpf
Copy link
Contributor

@hmpf hmpf commented Jan 29, 2025

Depends on #2850

Copy link

@hmpf
Copy link
Contributor Author

hmpf commented Jan 29, 2025

Integration tests fail:

Probably caused by naive date times

FAILED tests/integration/eventengine/juniper_alert_count_test.py::TestHandleStart::test_handle_start_resolves_existing_alert_with_different_count - AssertionError: assert datetime.datetime(2025, 1, 29, 12, 19, 12, 136714) == datetime...
FAILED tests/integration/web/maintenance/views_test.py::TestAddMaintenanceTask::test_valid_data_without_end_time_should_suceed - TypeError: can't compare offset-naive and offset-aware datetimes
FAILED tests/integration/web/maintenance/views_test.py::TestAddMaintenanceTask::test_valid_data_with_end_time_should_suceed - TypeError: can't compare offset-naive and offset-aware datetimes

Other

FAILED tests/integration/l2trace_test.py::DownlinkTests::test_employee1_downlink_should_be_foo_sw1_gi_0_10 - AssertionError: False is not true
FAILED tests/integration/l2trace_test.py::PathTests::test_path_for_employee1_should_be_3_long - AssertionError: 2 != 3
FAILED tests/integration/l2trace_test.py::PathTests::test_path_for_employee1_should_be_on_vlan_20 - AttributeError: 'NoneType' object has no attribute 'vlan'
FAILED tests/integration/l2trace_test.py::PathTests::test_path_for_employee1_should_end_with_foo_gw - AttributeError: 'NoneType' object has no attribute 'host'
FAILED tests/integration/l2trace_test.py::PathTests::test_path_for_employee2_should_be_3_long - AssertionError: 2 != 3 : [PathNode(<Vlan: 20 (employeevlan)>, None, <Host('10.0.20.90...
FAILED tests/integration/l2trace_test.py::TraceTests::test_employee_path_passes_through_foo_sw1 - AssertionError: 5 != 3 : [<nav.web.l2trace.ResultRow object at 0x74de2075e810>, <nav....
FAILED tests/integration/web/crawler_test.py::test_all_links_should_be_reachable - AssertionError: 1 unreachable pages:
FAILED tests/integration/web/machinetracker_test.py::test_get_netbios_query_should_not_fail - assert 0 == 1

Copy link

codecov bot commented Jan 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.35%. Comparing base (35b0f71) to head (b28aa42).
Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #3269       +/-   ##
===========================================
+ Coverage   60.58%   84.35%   +23.76%     
===========================================
  Files         606        7      -599     
  Lines       43733      262    -43471     
  Branches       48       48               
===========================================
- Hits        26494      221    -26273     
+ Misses      17227       29    -17198     
  Partials       12       12               

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hmpf
Copy link
Contributor Author

hmpf commented Jan 29, 2025

DeprecationWarning, deprecated packages

.tox/integration-py311-django50/lib/python3.11/site-packages/napalm/init.py:1
/source/.tox/integration-py311-django50/lib/python3.11/site-packages/napalm/init.py:1: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
import pkg_resources

.tox/integration-py311-django50/lib/python3.11/site-packages/netmiko/base_connection.py:12
/source/.tox/integration-py311-django50/lib/python3.11/site-packages/netmiko/base_connection.py:12: DeprecationWarning: 'telnetlib' is deprecated and slated for removal in Python 3.13
import telnetlib

.tox/integration-py311-django50/lib/python3.11/site-packages/twisted/web/http.py:103
/source/.tox/integration-py311-django50/lib/python3.11/site-packages/twisted/web/http.py:103: DeprecationWarning: 'cgi' is deprecated and slated for removal in Python 3.13
import cgi

@hmpf
Copy link
Contributor Author

hmpf commented Jan 29, 2025

DeprecationWarning: old-style namespace packages

.tox/integration-py311-django50/lib/python3.11/site-packages/pkg_resources/init.py:2846
/source/.tox/integration-py311-django50/lib/python3.11/site-packages/pkg_resources/init.py:2846: DeprecationWarning: Deprecated call to pkg_resources.declare_namespace('jnpr').
Implementing implicit namespace packages (as specified in PEP 420) is preferred to pkg_resources.declare_namespace. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
declare_namespace(pkg)

.tox/integration-py311-django50/lib/python3.11/site-packages/pkg_resources/init.py:2846
.tox/integration-py311-django50/lib/python3.11/site-packages/pkg_resources/init.py:2846
/source/.tox/integration-py311-django50/lib/python3.11/site-packages/pkg_resources/init.py:2846: DeprecationWarning: Deprecated call to pkg_resources.declare_namespace('sphinxcontrib').
Implementing implicit namespace packages (as specified in PEP 420) is preferred to pkg_resources.declare_namespace. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
declare_namespace(pkg)

.tox/integration-py311-django50/lib/python3.11/site-packages/pkg_resources/init.py:2846
/source/.tox/integration-py311-django50/lib/python3.11/site-packages/pkg_resources/init.py:2846: DeprecationWarning: Deprecated call to pkg_resources.declare_namespace('zope').
Implementing implicit namespace packages (as specified in PEP 420) is preferred to pkg_resources.declare_namespace. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
declare_namespace(pkg)

@hmpf
Copy link
Contributor Author

hmpf commented Jan 29, 2025

RuntimeWarning: DateTimeField received a naive datetime while time zone support is active.

147 tests with warnings.

naive-datetimes.txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants