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

fix(tests): core24 classic linter test expectations #4600

Merged
merged 2 commits into from
Feb 22, 2024

Conversation

sergiusens
Copy link
Collaborator

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox run -m lint?
  • Have you successfully run tox run -e test-py310? (supported versions: py39, py310, py311, py312)

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
@sergiusens sergiusens requested review from tigarmo and lengau February 21, 2024 21:55
Copy link

codecov bot commented Feb 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6ff3e0b) 88.83% compared to head (97f1d46) 88.88%.
Report is 11 commits behind head on feature/craft-application.

❗ Current head 97f1d46 differs from pull request most recent head 1e10549. Consider uploading reports for the commit 1e10549 to get more accurate results

Additional details and impacted files
@@                      Coverage Diff                      @@
##           feature/craft-application    #4600      +/-   ##
=============================================================
+ Coverage                      88.83%   88.88%   +0.05%     
=============================================================
  Files                            327      330       +3     
  Lines                          22033    22066      +33     
=============================================================
+ Hits                           19573    19614      +41     
+ Misses                          2460     2452       -8     

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

root@evolved-marten:~# sed -n -e 's/VERSION=\(.*\)/\1/p' /etc/os-release
"24.04 (Noble Numbat)"
root@evolved-marten:~# apt download libc6
Get:1 http://archive.ubuntu.com/ubuntu noble/main amd64 libc6 amd64 2.38-3ubuntu1 [3248 kB]
Fetched 3248 kB in 4s (850 kB/s)
root@evolved-marten:~# dpkg-deb -x libc6_2.38-3ubuntu1_amd64.deb x
root@evolved-marten:~# find x | grep usr/lib/x86_64-linux-gnu/audit/sotruss-lib.so
root@evolved-marten:~# echo $?
1

root@divine-piranha:~# sed -n -e 's/VERSION=\(.*\)/\1/p' /etc/os-release
"22.04.4 LTS (Jammy Jellyfish)"
root@divine-piranha:~# apt download libc6
Get:1 http://archive.ubuntu.com/ubuntu jammy-updates/main amd64 libc6 amd64 2.35-0ubuntu3.6 [3236 kB]
Fetched 3236 kB in 3s (1146 kB/s)
root@divine-piranha:~# dpkg-deb -x libc6_2.35-0ubuntu3.6_amd64.deb x
root@divine-piranha:~# find x | grep usr/lib/x86_64-linux-gnu/audit/sotruss-lib.so
x/usr/lib/x86_64-linux-gnu/audit/sotruss-lib.so

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
@sergiusens sergiusens changed the title fix(tests): expect the proper caca versions core24 classic linter test fix(tests): core24 classic linter test expectations Feb 21, 2024
@sergiusens
Copy link
Collaborator Author

seems we now need #4601 for the unit tests to pass

@sergiusens sergiusens merged commit ee5e89c into feature/craft-application Feb 22, 2024
4 of 8 checks passed
@sergiusens sergiusens deleted the CRAFT-2471-classic-linter branch February 22, 2024 10:55
@tigarmo
Copy link
Contributor

tigarmo commented Feb 22, 2024

I think we have a problem now with using the same Project class for core22 and core24 projects. This is a core22 spread failure:

2024-02-21T23:19:49.4297186Z 2024-02-21 23:19:49 Error executing google:ubuntu-22.04-64:tests/spread/general/appstream-desktop (feb212306-729302) : 
2024-02-21T23:19:49.4298762Z -----
2024-02-21T23:19:49.4299642Z + unset SNAPCRAFT_BUILD_ENVIRONMENT
2024-02-21T23:19:49.4300614Z + cd ../snaps/appstream-desktop
2024-02-21T23:19:49.4302256Z + snapcraft --use-lxd
2024-02-21T23:19:49.4303121Z Launching instance...
2024-02-21T23:19:49.4304117Z Creating new instance from remote
2024-02-21T23:19:49.4305231Z Creating new base instance from remote
2024-02-21T23:19:49.4306415Z Creating new instance from base instance
2024-02-21T23:19:49.4307902Z Starting instance
2024-02-21T23:19:49.4308839Z Initializing parts lifecycle
2024-02-21T23:19:49.4309888Z Installing build-packages
2024-02-21T23:19:49.4310932Z Installing build-snaps
2024-02-21T23:19:49.4311628Z Pulling appstream-desktop
2024-02-21T23:19:49.4312472Z Building appstream-desktop
2024-02-21T23:19:49.4314577Z :: + install -D -m 0644 io.snapcraft.appstream.desktop /root/parts/appstream-desktop/install/usr/share/applications/io.snapcraft.appstream.desktop
2024-02-21T23:19:49.4317163Z :: + install -D -m 0644 io.snapcraft.appstream.metainfo.xml /root/parts/appstream-desktop/install/usr/share/metainfo/io.snapcraft.appstream.metainfo.xml
2024-02-21T23:19:49.4319306Z :: + install -D -m 0644 snapcraft.svg /root/parts/appstream-desktop/install/usr/share/icons/hicolor/scalable/apps/snapcraft.svg
2024-02-21T23:19:49.4321126Z :: + install -D -m 0755 appstream-desktop /root/parts/appstream-desktop/install/usr/bin/appstream-desktop
2024-02-21T23:19:49.4322328Z Staging appstream-desktop
2024-02-21T23:19:49.4323400Z Priming appstream-desktop
2024-02-21T23:19:49.4324470Z Extracting and updating metadata...
2024-02-21T23:19:49.4326265Z error setting __root__: Required field 'version' is not set and 'adopt-info' not used.

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

Successfully merging this pull request may close these issues.

3 participants