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

Version lock indirect dependencies of our test suites #2622

Merged
merged 2 commits into from May 24, 2023

Conversation

lunkwill42
Copy link
Member

A couple of the libraries we utilize as part of our test suite have seen their dependencies be released in new versions that are no longer compatible.

This PR locks those indirect dependencies to versions that are know to work for us.

Fixes #2617

`snmpsim` depends on `pyasn1`, but does not lock the dependent
version. `pyasn1` version 0.5.0 was recently released, and it seems
`snmpsim` is incompatible with the new version, causing integration
tests that use snmpsim to fail.

This locks pyasn1 to a version that should still work with the latest
version of snmpsim.
`selenium` and `requests` depend on `urllib3`, which recently released
version 2.0.

The version of selenium we are using for driving our functional tests
doesn't appear to be compatible with this new major version, causing all
selenium-based tests to fail.
@lunkwill42 lunkwill42 requested a review from hmpf May 24, 2023 13:10
@lunkwill42 lunkwill42 self-assigned this May 24, 2023
@sonarcloud
Copy link

sonarcloud bot commented May 24, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codecov
Copy link

codecov bot commented May 24, 2023

Codecov Report

Merging #2622 (eea73ef) into master (72a94db) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2622      +/-   ##
==========================================
+ Coverage   54.20%   54.25%   +0.05%     
==========================================
  Files         558      558              
  Lines       40627    40722      +95     
==========================================
+ Hits        22020    22092      +72     
- Misses      18607    18630      +23     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link

Test results

     12 files       12 suites   12m 43s ⏱️
3 238 tests 3 142 ✔️   96 💤 0
9 189 runs  8 901 ✔️ 288 💤 0

Results for commit eea73ef.

@lunkwill42 lunkwill42 merged commit 7e26ce2 into Uninett:master May 24, 2023
20 of 21 checks passed
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.

Fix failing SNMP integration tests
2 participants